Skip to content

Conversation

kelle
Copy link
Collaborator

@kelle kelle commented Jul 31, 2025

This will make it possible to read more JWST spectra.

  • raise minimum specutils to 2.0
  • change class from Spectrum1D to Spectrum

@kelle kelle changed the title upgrade to specutils2.1 upgrade to specutils>2.0 Jul 31, 2025
@kelle kelle requested review from dr-rodriguez and Copilot July 31, 2025 15:16
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR upgrades the specutils dependency from version 1.0+ to 2.0+ to enable reading more JWST spectra formats. The change involves updating the dependency constraint and replacing all references to the deprecated Spectrum1D class with the new Spectrum class from specutils 2.0.

  • Update minimum specutils version requirement from 1.0 to 2.0
  • Replace Spectrum1D class references with Spectrum throughout the codebase
  • Update documentation and comments to reflect the class name change

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
pyproject.toml Updates minimum specutils dependency from 1.0 to 2.0
docs/index.rst Updates documentation references from Spectrum1D to Spectrum
astrodbkit/tests/test_spectra.py Updates test mock to use Spectrum instead of Spectrum1D
astrodbkit/spectra.py Updates all code references from Spectrum1D to Spectrum class

Co-authored-by: Copilot <[email protected]>
@dr-rodriguez
Copy link
Collaborator

Looks OK to me, and the tests are all passing.

@kelle kelle merged commit 237bf8d into astrodbtoolkit:main Aug 13, 2025
3 checks passed
@kelle kelle deleted the upgrade-specutils branch August 13, 2025 18:46
@kelle
Copy link
Collaborator Author

kelle commented Aug 13, 2025

Now we need a new release. :)

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