Drop flow_caps from the sample config - #60
Merged
Conversation
The key has had no reader anywhere in the stack since cotmetrics 734bbc6 deleted the intra-week gap estimator (an ADR-0006 prerequisite). cotmetrics stops loading it in its own change; this is the matching removal from the sample config the app falls back to when COT_VIZ_CONFIG is unset. viz_config reads params with safe_load and callers use .get(), so nothing here required the key. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
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.
flow_capsinconfig/params.yamlhas had no reader anywhere in the stack since 2026-07-24, when cotmetrics734bbc6deleted the intra-week gap estimatorestimate_current_gap_positionsas an ADR-0006 prerequisite.This is the sample-config half of a cross-repo cleanup.
viz_configreads params withsafe_loadand every consumer uses.get(), so nothing here depended on the key. A comment pointing at734bbc6replaces it.Companion changes
cotmetrics#29 — stops loading the key intoCotIndexer.flow_capscotmetrics-config— real tuned block removed, its measured bind rates preserved in a private doc (merged)npf— same key intests/fixtures/params.yamlVerification
437 passed.
🤖 Generated with Claude Code