Skip to content

[ENH] Spectral Series: visual parameter adjustment#830

Open
borondics wants to merge 2 commits intoQuasars:masterfrom
borondics:spectralseries_parameters
Open

[ENH] Spectral Series: visual parameter adjustment#830
borondics wants to merge 2 commits intoQuasars:masterfrom
borondics:spectralseries_parameters

Conversation

@borondics
Copy link
Member

This has been missing since now - now I copy-pasted it from Spectra. I think we probably said that we will make it work at some point.

  • Do we want to use InteractiveViewBox from the Spectra widget to provide more interactive zooming?

@markotoplak markotoplak changed the title Adding visual parameter adjustment possibility in Spectral Series [ENH] Spectral Series: visual parameter adjustment Dec 1, 2025
@markotoplak
Copy link
Collaborator

Why didn't you reuse ImageParameterSetter from owhyper or ParameterSetter from owspectra? If you said you copy-pasted it, why not just import it? What is specific for this implementation?

@borondics
Copy link
Member Author

This is yet another different type plot, so I am not sure whether to reuse either one of those now and change it in the future or make a new one and change it according to the plot requirements. It would be nice to have only one and reuse it, but I am not sure if we can generalize everything.

What do you think?

@markotoplak
Copy link
Collaborator

If current functionality can be supported by importing, I'd import. And then worry about extensions later.

@borondics
Copy link
Member Author

If #839 is merged, we can delete this.

@markotoplak
Copy link
Collaborator

Feri, this is a different PR, this is some other plot.

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.

2 participants