Skip to content

Pass max output tokens into transcript prompts - #305

Draft
Mirochill wants to merge 1 commit into
souzatharsis:mainfrom
Mirochill:fix-258-max-output-tokens
Draft

Pass max output tokens into transcript prompts#305
Mirochill wants to merge 1 commit into
souzatharsis:mainfrom
Mirochill:fix-258-max-output-tokens

Conversation

@Mirochill

@Mirochill Mirochill commented May 27, 2026

Copy link
Copy Markdown

Fixes #258

Summary:

  • include max_output_tokens in standard and long-form prompt parameters
  • add the configured output-token budget to the composed transcript-generation prompt
  • format the long-form instruction with the configured token count
  • add focused tests for prompt parameter composition and the composed prompt variables

Validation:

  • git diff --check
  • git diff --check HEAD~1..HEAD
  • git show --check --oneline HEAD -- podcastfy/content_generator.py tests/test_genai_podcast.py
  • Static inspection of podcastfy/content_generator.py and tests/test_genai_podcast.py
  • Not run locally

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

max_output_tokens not passed to prompt

1 participant