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

[FEAT] smart viewer bounds #2741

Open
havok2063 opened this issue Mar 7, 2024 · 0 comments
Open

[FEAT] smart viewer bounds #2741

havok2063 opened this issue Mar 7, 2024 · 0 comments
Labels

Comments

@havok2063
Copy link
Collaborator

havok2063 commented Mar 7, 2024

Jdaviz component

Specviz

What is the problem this feature will solve?

I have spectra with lots of noise at the far blue edge, which blows up the y min/max values in the full zoom view. Cases e.g. where reasonable y min/max values are [0, 800] 1e-17 erg/Ang/s/cm^2 and the real y min/max values are [-18620000, 18620000] 1e-17 erg/Ang/s/cm^2 . I'm trying to resize the viewer window easily without fiddling with multiple zooms or manually trying viewer settings. On initial spectrum load, I'd like the viewer display to be reasonably sized around the spectrum.

Right now it's very difficult to get the spectrum to display nicely in the viewer, especially when other spectra are loaded. I have to play around with toggling spectra on/off and resetting the zoom window to get the spectra to overlay nicely.

Describe the desired outcome

I'd like a UI option for selecting or setting smart viewer bounds. Something as simple as sizing around the 99 percentile of my data would work. There could be a new right-click option under the "Reset Zoom" for "Smart Zoom" which resets the bounds to some percentile of the data. Or if I manually set the bounds under "Viewer Bounds", I'd like the option to save those bounds to the "Smart Zoom" button.

Additional context

🐱

@havok2063 havok2063 added feature Feature request needs-triage Issue opened via template and needs triaging labels Mar 7, 2024
@pllim pllim added specviz cubeviz and removed needs-triage Issue opened via template and needs triaging labels Mar 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants