Skip to content

Fix scale on right side of pane for channel configuration dialogs #499

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Fix scale on right side of pane for channel configuration dialogs #499

wants to merge 3 commits into from

Conversation

bparks13
Copy link
Member

@bparks13 bparks13 commented Sep 16, 2025 •
edited
Loading

This PR updates how scales are rendered and manipulated in channel configuration dialogs. Previously, they were drawn as objects in the same GraphPane as the ProbeInterface objects. Now, there is a secondary pane that is created to the right of the main pane that overrides the default Y-Axis labels and tics to simplify the logic required to maintain these scales over time.

The scale pane and the main pane are tied together so that they pan/zoom whenever the user pans/zooms in the main pane. Due to the separation in panes, all panning and zooming are disabled in the scale pane to avoid any issues.

Devices that utilize this new feature:

  • NeuropixelsV1e
  • NeuropixelsV1f
  • NeuropixelsV2e
  • NeuropixelsV2eBeta

For devices that do not have a scale, such as the Rhs2116, the original behavior is retained with a single pane to draw the probe object.

Fixes #339
Fixes #313

NeuropixelsV1e GIF

npxv1e-scale

NeuropixelsV2e GIF

npxv2e-scale

Rhs2116 GIF

rhs2116-probe

@bparks13
Update probe configuration dialogs
6677a31 
- Add a column to separate the track bar for the channel configuration dialog
@bparks13 bparks13 added this to the 0.7.0 milestone Sep 16, 2025
@bparks13 bparks13 requested a review from jonnew September 16, 2025 14:25
@bparks13 bparks13 self-assigned this Sep 16, 2025
@bparks13
Move scale to second pane
9ea1b94 
- Only applies for dialogs that need to draw a scale (such as NeuropixelsV1e/V1f and NeuropixelsV2e/V2eBeta)
- Updated the logic for constraining pan/zoom in all dialogs so that the probe is always in view
- Standardized dialog sizes for consistency
@bparks13
Address review comments
0599a37 
- Change background color to WhiteSmoke to provide visual difference between background and probe
- Decrease scale line thickness, and remove boldness from text
- Round the X/Y values when resizing panes to avoid a visual flickering effect
@bparks13
Copy link
Member Author

@jonnew This PR is ready for review again after our meeting today.

@bparks13
Copy link
Member Author

Screenshots of the new color change and scale line/text updates.

Screenshot 2025-09-24 085722 Screenshot 2025-09-24 085835

@bparks13
Copy link
Member Author

During testing, this update solves issue #313 as well. Now, the ChannelConfiguration dialog does not reset the zoom status when resizing.

@jonnew
Copy link
Member

jonnew commented Sep 29, 2025 •
edited
Loading

Please remove ChannelConfigurationDialog.CheckProbeIsInView. It is ad-hoc and its jumpy behavior during use is very annoying. I would much rather have the probe disappear from view rather and simply have a button, or some type of mouse click shortcut, to reset the view. Additionally, its fine to have zoom boudaries, but make them more extreme please. I simply commented out their redefinition away from "20" in the constructors (not sure what that means) and was pretty happy with the result.

jonnew
jonnew reviewed Sep 29, 2025
View reviewed changes
Copy link
Member

@jonnew jonnew left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Almost, please see comment. Also because there will no longer be pan boundaries, the scale bar can keep going into the negative, it should display correctly for the entire area it exists over:

Image

@jonnew jonnew self-requested a review September 29, 2025 23:22
jonnew
jonnew requested changes Sep 29, 2025
View reviewed changes
Copy link
Member

@jonnew jonnew left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See comments.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jonnew jonnew jonnew requested changes

Requested changes must be addressed to merge this pull request.

Assignees

@bparks13 bparks13

Labels
None yet
Projects
None yet
Milestone
0.7.0
Development

Successfully merging this pull request may close these issues.

Ruler in neuropixels GUI should be afixed to right Y axis Save zoom state when resizing dialogs
2 participants
@bparks13 @jonnew