Repository navigation
Conversation
Under hovermode "x unified" the flow-z strip reports only when the cursor is on the strip, so hovering the lines showed the level and never the week's move. The Speculator line's hover now reads "68 · flow z -0.65 (net -5,727 contracts)": index, then that week's flow z and net contracts. Warm-up weeks stay silent; a week with an index but no readable z shows the index alone. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Owner
Author
|
Superseded by the Commercial flow strip PR: the Speculator line is removed, and the same hover now rides on the Commercial line. |
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.
Follow-up to #142. Under hovermode "x unified" the flow-z strip on the Positioning Index panel reports only when the cursor is on the strip itself, so hovering the lines gave the level and never the week's move.
The Speculator line's hover now carries both,
Speculator (Managed Money) : 68 · flow z -0.65 (net -5,727 contracts), beside the Legacy lines' values in the same box. Warm-up weeks stay silent, and a week with an index but no readable z (masked or zero-sd) shows the index alone. The strip's own hover is unchanged.Verification: suite 910 passed, one new store-free test pinning the text per week; ruff clean. Looked at in the running app on Gold: the unified box reads
2025-04-08 | Commercial : 100 | Non-Commercial : 0 | Non-Reportable : 100 | Speculator (Managed Money) : 0 · flow z -0.82 (net -11,013 contracts).🤖 Generated with Claude Code