Specutils 2 compatibility #2922
Draft
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
github-actions
bot
added
documentation
rosteen
force-pushed
the
specutils-2-compatibility
branch
3 times, most recently
from
83b16e0 to
dabf631
Compare
Fix coords mouseover when SkyCoord is before SpectralCoord More debugging Fix slice with different spectral_axis_index, add specutils_format kwarg to cubeviz parser Remove unused import Remove ESA parser from commissioning [skip ci] Generalizing coords_info to handle spectral axis first or last [ci skip] Working on debugging aperture photometry Need to pull meta from data not comp Fix aperture photometry and wrong coord order in coords_info Fix codestyle, delete defunct case in parser Change Spectrum1D to Spectrum Change class name in recent code additions Get spectral extraction working again for cylinder case Debugging wavelength dependent case Make sure spectral extraction result is linked when added to data collection Moved fix to correct place Fix indents Remove stray code from rebase Update missed Spectrum1D Debugging cubeviz parser First start on fixing testt Fix bad rebase in spectral extraction Starting to work through test failures Working through test failures Debugging spectral extraction
rosteen
force-pushed
the
specutils-2-compatibility
branch
from
dabf631 to
9fcb2bc
Compare
Finish fixing codestyle, probably still need functional fixes
rosteen
force-pushed
the
specutils-2-compatibility
branch
from
787141a to
f9d26cf
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Replaces #2762 after a significant rebase, with a better title and branch name that are more accurate to the scope of the PR. Still in draft, need to update tests