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

[Plots] Toggling Y-Axis visible doesn't take into account zoom level #7928

Open
3 of 7 tasks
jvigliotta opened this issue Nov 13, 2024 · 1 comment
Open
3 of 7 tasks

Comments

@jvigliotta
Copy link
Contributor

jvigliotta commented Nov 13, 2024

Related to #7929

Summary

Clicking the eye icon (toggle y-axis visibility) will query data at the time conductor range and will not take into account the current zoom level of the plot. This is especially noticeable when zoomed in on the plot when the time conductor has a large time range. This can cause the data in the zoom level to be less accurate due to min/maxing.

Expected vs Current Behavior

The range of data in the zoomed in section of the plot, should be queried at that range.

Steps to Reproduce

  1. View a large time range of data on a plot.
  2. Zoom in to one section of the plot, zoom in pretty far.
  3. Notice the plot will display the points correctly as you zoom in
  4. Click the eye icon to toggle y-axis
  5. Notice the zoomed in section doesn't display the same data, it displays the data at the range of the time conductor and ignores the zoom level.

Environment

  • Open MCT Version:
  • Deployment Type:
  • OS:
  • Browser:

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? zoom out and zoom in again, not desirable
  • 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

Eye icon in the bottom left corner of the plot
Screenshot 2024-11-13 at 10 53 35 AM

@mudinthewater
Copy link

Note this only occurs if you've cleared the data.

View a large time range of data on a plot.
Zoom in to one section of the plot, zoom in pretty far.
Notice the plot will display the points correctly as you zoom in
Clear data in the UI menu
Click the eye icon to toggle y-axis
Notice the zoomed in section doesn't display the same data, it displays the data at the range of the time conductor and ignores the zoom level.

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

No branches or pull requests

2 participants