Skip to content
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.

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

Plot annotations aren't creatable #6157

Closed
1 of 7 tasks
scottbell opened this issue Jan 23, 2023 · 7 comments · Fixed by #6161
Closed
1 of 7 tasks

Plot annotations aren't creatable #6157

scottbell opened this issue Jan 23, 2023 · 7 comments · Fixed by #6161
Labels
Milestone

Comments

@scottbell
Copy link
Contributor

scottbell commented Jan 23, 2023

Summary

When you marquee plot points, they aren't being selected properly for annotation creation.

Expected vs Current Behavior

When using the marquee for annotations, plot points should glow.
Expected behavior:
image

Current behavior:
image

Steps to Reproduce

  1. Create a Sine Wave Generator
  2. Put it into Fixed Time Mode
  3. Hold Option+Shift and drag a marquee around some plot points
  4. Ensure plot points are glowing and annotation inspector allows adding a tag

Impact Check List

  • Data loss or misrepresented data?
  • Regression? Did this used to work or has it always been broken?
  • Is there a workaround available?
  • Does this impact a critical component?
  • Is this just a visual bug with no functional impact?
  • Does this block the execution of e2e tests?
  • Does this have an impact on Performance?

Additional Information

@scottbell
Copy link
Contributor Author

scottbell commented Jan 23, 2023

Tested and seems to be due to this merge. Lots of overlapping code, so expected.

@scottbell
Copy link
Contributor Author

This issue is due to the marquee y axis now being an array.

@shefalijoshi
Copy link
Contributor

Testing instructions:

  1. Create several sine wave generators with different offsets and ranges.
  2. Add all but one of these sine wave generators to an overlay plot so that they are all on one y axis
  3. Add annotations to points on the plots and confirm that they're saved and can be recalled.
  4. Move some of the sine wave generators to different y axes
  5. Ensure that the existing annotations correctly show up and scale to the y axis they're on.
  6. Add the last sine wave generator to the overlay plot (this should have no annotations)
  7. Ensure that the last sine wave generator does not show any annotations
  8. Remove a sine wave generator from the overlay plot and ensure that the other sine wave generator annotations still show up and work correctly.

@ozyx
Copy link
Member

ozyx commented Jan 24, 2023

Testathon 1/24/23

Mostly verified, was not able to confirm adding tags since no tags were configured on testathon. Dragging the marquee does highlight the area and points correctly

@charlesh88
Copy link
Contributor

Verified fixed Testathon 2023-01-26.

@ozyx
Copy link
Member

ozyx commented Jan 26, 2023

Verified fixed -- Testathon 2023-01-26

@rukmini-bose
Copy link
Contributor

Verified 3/20/2024

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants