Repository navigation
Remove the daily memory enqueue cap - #252
Merged
Merged
Conversation
With hourly enqueue the 2000-run daily cap ran out around 04:00 UTC and stopped all memory work for the rest of the day; on 2026-10-12 it skipped 4,652 qualifying chat hours. The active-queue bound MEMORY_MAX_QUEUED_RUNS is now the only enqueue limit.
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.
Summary
This removes the daily memory enqueue cap (
MEMORY_MAX_DAILY_ENQUEUED_RUNS, default 2000). Since memory runs are enqueued hourly (#251), the cap ran out at about 04:00 UTC. After that, no chat hour got a run for the rest of the UTC day, and GPU2 sat idle. On 2026-10-12 the cap dropped 4,652 qualifying chat hours. The active-queue boundMEMORY_MAX_QUEUED_RUNSstays and is now the only enqueue limit.Changes
MemoryRunEnqueuePolicylosesmax_daily_enqueued_runs. The enqueue capacity ismax_queued_runs − active queue.SQL_COUNT_BASE_MEMORY_RUNS_CREATED_SINCEis removed.daily_capdecision becomesqueue_cap. Themax_daily_enqueued_runsmetric is dropped, andeffective_daily_limitbecomeseffective_limit. The admin UI does not read these fields./admin/api/memory/runsdropspolicy.max_daily_enqueued_runs..env.exampleand the design spec no longer mention the cap. A leftoverMEMORY_MAX_DAILY_ENQUEUED_RUNSin an env file is ignored.Verification
cargo fmt --all,cargo clippy --workspace --all-targets -- -D warnings: clean.cargo test -p openplotva-memory -p openplotva-storage -p openplotva-config -p openplotva-app --lib: 1,680 passed, 0 failed.live_memory_runs_are_enqueued_through_the_last_complete_hourruns the changed enqueue and rollup SQL in CI.Deployment effect
MEMORY_MAX_QUEUED_RUNS(5000 in production).