fix: o padrão de num_ctx passa a valer onde importa - #17
Merged
Conversation
Eu troquei 2048 por 4096 no campo da dataclass e nao no `from_env` — e quem
monta a configuracao de verdade e o `from_env`. O robo continuou com 2048, e
quem denunciou foi o aviso que eu tinha acabado de escrever:
a persona ocupa ~1945 dos 2048 tokens de contexto ... e ela nao cabe
junto com a resposta
O valor era repetido em dois lugares, entao mudar um so mudou pela metade.
Agora e uma constante, `DEFAULT_NUM_CTX`, e os dois a usam.
Dois testes cercam isso: um confere que os dois caminhos dao o mesmo numero, o
outro que a persona deste robo cabe junto com a resposta no padrao — que era
exatamente o que 2048 nao permitia.
760 testes passam.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_0173mr3mDsAKQuPUNaPghe9v
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Troquei 2048 por 4096 no campo da dataclass e não no
from_env— e quem monta a configuração de verdade é ofrom_env. O robô continuou com 2048.Quem denunciou foi o aviso que eu tinha acabado de escrever, no arranque seguinte:
O valor estava repetido em dois lugares, então mudar um só mudou pela metade. Agora é uma constante,
DEFAULT_NUM_CTX, e os dois a usam.Dois testes cercam isso: um confere que os dois caminhos dão o mesmo número, o outro que a persona deste robô cabe junto com a resposta no padrão — que era exatamente o que 2048 não permitia.
760 testes passam.
🤖 Generated with Claude Code
https://claude.ai/code/session_0173mr3mDsAKQuPUNaPghe9v