Repository navigation
feat: a crosshair with the values under the mouse in Point mode - #14
Merged
Merged
Conversation
In Point mode, a crosshair follows the mouse in the panel under it. A label beside it lists the values of the curves at its x, with dots where it meets them: of a time trace at that time, of a profile at that coordinate. Over bars it lists the bars of the element, and over a map or a time-space image the value of the image at that place. - The values are along the lines of the curves, on log axes too; a curve that does not reach that x is left out. At most ten are listed. - The label keeps to the side with more room, stays within the plot, and is drawn over the legend and the title. It follows the view when it changes, and its values follow the time when it is stepped with the keys. - It goes away when the mouse leaves the plot, and in the other modes. - The coordinates, in the label and in the status bar, are rounded to what the view resolves (0, not 2.2e-16), and are values on log axes, not logarithms. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_018aVekYzCRLPE71Mm6UH8Zp
# Conflicts: # tests/test_gui.py
The crosshair and the status bar round the coordinates of the mouse to what the view resolves. A tiny negative coordinate, such as the mapping of the scene gives for 0, was rounded to -0.0 and shown as "-0". Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_018aVekYzCRLPE71Mm6UH8Zp
4 tasks done
icyspirit
pushed a commit
that referenced
this pull request
Sep 29, 2026
Main now has the crosshair (#14) and the device template (#15). The crosshair gives the value of the image through Panel._image_value_at, so it also says the value of the cell of an unstructured grid under it, and the status bar keeps the coordinates rounded to the view. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_018aVekYzCRLPE71Mm6UH8Zp
icyspirit
pushed a commit
that referenced
this pull request
Sep 29, 2026
Main now has the crosshair (#14) and the device template (#15). The crosshair reads the curves a panel draws, so with sources relative to the first it gives B − A or B / A; a test checks it. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_018aVekYzCRLPE71Mm6UH8Zp
4 tasks done
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
A crosshair in Point mode. It follows the mouse in the panel under it. A label beside it gives the values there, and dots mark where it meets the curves.
t = 5 s, then one row per curve with its color, e.g.■ Ip (equilibrium) [A] = 1.rho_tor_norm = 0.3, then one row per curve.PF1), then its bars, one per source.Values:
The label:
,.while the mouse rests on a profile.Coordinates in the label and in the status bar:
Notes
sigTransformChanged. A guard skips the callback that the mapping itself can trigger.tests/test_gui.py, and both are kept.Test plan
QT_QPA_PLATFORM=offscreen pytest: 190 passed in the dev environment, a clean PySide6 6.11 environment and a PyQt5 5.15 environment, after merging main.pyflakes src testsreports nothing.🤖 Generated with Claude Code
https://claude.ai/code/session_018aVekYzCRLPE71Mm6UH8Zp