Repository navigation
Feed incremental engine streams instead of recomputing the buffer - #100
Merged
Merged
Conversation
… the buffer Each event now feeds a RunDistanceStream, a SessionSegmenter and the goal stream in lockstep, so a recompute reads their state instead of re-parsing every run and re-segmenting the whole buffer. Run indices count from the worker's first event, so they stay joined to the goal stream after the rolling buffer wraps. The unused per-event list is dropped from the episodes payload, and stored goal runs are released except the latest. Renderer tests are updated for the explicit hat list and the not connected section.
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
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.
Each event now feeds a RunDistanceStream, a SessionSegmenter and the goal stream together, in one place, so a recompute reads their state instead of redoing the whole buffer. The output is the same as the batch calls give over the buffer until it wraps. A test checks exactly that.
What changes
This bumps the engine to f03612b, which is main again with the scaling fixes from agent-lm-packages #8. The humanize and hat tests are updated for the engine's explicit hat list and its not connected section.
All 267 tests pass, including test_run_identity and test_audit_fixes.
Closes #99