-
Notifications
You must be signed in to change notification settings - Fork 63
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
Add option of using self derived interpolated profile for horne extract in specviz2d/spectral extraction plugin #2845
base: main
Are you sure you want to change the base?
Add option of using self derived interpolated profile for horne extract in specviz2d/spectral extraction plugin #2845
Conversation
jdaviz/configs/specviz2d/plugins/spectral_extraction/spectral_extraction.py
Show resolved
Hide resolved
jdaviz/configs/specviz2d/plugins/spectral_extraction/spectral_extraction.vue
Show resolved
Hide resolved
jdaviz/configs/specviz2d/plugins/spectral_extraction/spectral_extraction.vue
Outdated
Show resolved
Hide resolved
pyproject.toml
Outdated
"specreduce>=1.3.0,<1.4.0", | ||
"specreduce@git+https://github.com/astropy/specreduce", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should release of specreduce and pinning to it block merge here? Alternatively, we could have the feature dependent on a version check of specreduce without having to bump the min-pin?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm +1 for release specreduce.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left some +1's on @kecnry's review comments.
pyproject.toml
Outdated
"specreduce>=1.3.0,<1.4.0", | ||
"specreduce@git+https://github.com/astropy/specreduce", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm +1 for release specreduce.
jdaviz/configs/specviz2d/plugins/spectral_extraction/tests/test_spectral_extraction.py
Outdated
Show resolved
Hide resolved
This comment was marked as resolved.
This comment was marked as resolved.
e5c898d
to
bb683bd
Compare
@@ -24,7 +24,7 @@ dependencies = [ | |||
"voila>=0.4,<0.5", | |||
"pyyaml>=5.4.1", | |||
"specutils>=1.15", | |||
"specreduce>=1.3.0,<1.4.0", | |||
"specreduce>=1.4.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we exclude 1.5 now since we're allowing breaking changes still in specreduce?
"specreduce>=1.4.0", | |
"specreduce==1.4.*", |
Add the option of using a 'self profile' for Horne extraction to the spectral extraction plugin in Specviz2D.
This PR also bumps up specreduce to the most recent 1.4.0 which contains the relevant changes there.