Skip to content

Remove token_count from AgentState#771

Closed
mateusz834 wants to merge 2 commits into
dev/MP/remove-total_stepsfrom
dev/MP/remove-token_count
Closed

Remove token_count from AgentState#771
mateusz834 wants to merge 2 commits into
dev/MP/remove-total_stepsfrom
dev/MP/remove-token_count

Conversation

@mateusz834
Copy link
Copy Markdown
Member

No description provided.

@mateusz834 mateusz834 force-pushed the dev/MP/remove-token_count branch from cb990de to 2b1c5dc Compare May 8, 2026 10:04
@mateusz834 mateusz834 force-pushed the dev/MP/remove-total_steps branch from 8a0d9bd to 26468f3 Compare May 8, 2026 10:04
@mateusz834 mateusz834 force-pushed the dev/MP/remove-token_count branch from 2b1c5dc to 96eec84 Compare May 8, 2026 11:16
@mateusz834 mateusz834 force-pushed the dev/MP/remove-total_steps branch from 26468f3 to 3b63421 Compare May 8, 2026 11:17
mateusz834 added 2 commits May 8, 2026 13:24
Steps can be inferred from len(messages). Additionally
this fixes a bug, where model middleware overrides messages, without
updating the total_steps field.
@mateusz834 mateusz834 force-pushed the dev/MP/remove-total_steps branch from 3b63421 to d3e0ca5 Compare May 8, 2026 11:24
@mateusz834 mateusz834 force-pushed the dev/MP/remove-token_count branch from 96eec84 to ee9d591 Compare May 8, 2026 11:24
@mateusz834 mateusz834 force-pushed the dev/MP/remove-total_steps branch from d3e0ca5 to 451c178 Compare May 8, 2026 11:31
@mateusz834 mateusz834 closed this May 8, 2026
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.

1 participant