Skip to content

Conversation

@jellybean2004
Copy link
Contributor

@jellybean2004 jellybean2004 commented Sep 11, 2025

Description

It gives the wedge slicer plot a unique ID so it does not use the same plot as the sector slicer.

The plots no longer affect each other's scales either.

Fixes #3223

How Has This Been Tested?

Followed the reproduction steps and printed IDs to confirm the fix.

Review Checklist:

Documentation (check at least one)

  • There is nothing that needs documenting
  • Documentation changes are in this PR
  • There is an issue open for the documentation (link?)

Installers

  • There is a chance this will affect the installers, if so
    • Windows installer (GH artifact) has been tested (installed and worked)
    • MacOSX installer (GH artifact) has been tested (installed and worked)
    • Wheels installer (GH artifact) has been tested (installed and worked)

Licensing (untick if necessary)

  • The introduced changes comply with SasView license (BSD 3-Clause)

Copy link
Contributor

@DrPaulSharp DrPaulSharp left a comment

Choose a reason for hiding this comment

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

Looks good, and does the job nicely.

@jellybean2004 jellybean2004 merged commit 3e3f268 into main Sep 12, 2025
32 checks passed
@jellybean2004 jellybean2004 deleted the slicer-bugfix branch September 12, 2025 16:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

The Wedge Slicer in Q slicer and the Sector slicer seem to be using the same plot object

3 participants