feat(compact): refresh token usage after summarizing context#4502
feat(compact): refresh token usage after summarizing context#45020xdeafbeef wants to merge 1 commit into
Conversation
|
All contributors have signed the CLA ✍️ ✅ |
|
I have read the CLA Document and I hereby sign the CLA |
c2eb014 to
1a6ac23
Compare
|
@pakrym-oai hi, can you please take a look? |
|
Hi, any feedback? |
|
Now i can use tiktoken for more precise calc. Waiting for comments. |
|
Thanks for the contribution, and apologies for the slow response. We've received many PRs, and we don't have the bandwidth on the codex team to review all of them. We've updated our contribution guidelines to clarify that we're currently accepting contributions for bugs and security fixes, but we're not generally accepting new features at this time. We need to make sure that all new features compose well with both existing and upcoming features and fit into our roadmap. If you would like to propose a new feature, please file or upvote an enhancement request in the issue tracker. We will generally prioritize new features based on community feedback. |
I have read the CLA Document and I hereby sign the CLA.
This fixes that we still see prev
n% context leftafter running compaction and it's unclear, if compaction really worked.