search

pypeit / PypeIt

The Python Spectroscopic Data Reduction Pipeline
BSD 3-Clause "New" or "Revised" License
160 stars 102 forks source link

Package dependency updates #1854

Closed kbwestfall closed 3 weeks ago

kbwestfall commented 3 weeks ago

Deprecates support for python <= 3.10. I also updated other package dependencies, trying to at least synchronize them to when 3.11 was released.

This also includes some doc fixes.

We'll need to run tests.

kbwestfall commented 3 weeks ago

PS - This also includes some updates to the installing doc.

codecov-commenter commented 3 weeks ago

:warning: Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

Attention: Patch coverage is 0% with 4 lines in your changes missing coverage. Please review.

Project coverage is 38.03%. Comparing base (745f36c) to head (9ef8b39). Report is 55 commits behind head on develop.

Files with missing lines Patch % Lines
pypeit/spectrographs/vlt_xshooter.py 0.00% 4 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #1854 +/- ## =========================================== + Coverage 38.01% 38.03% +0.02% =========================================== Files 210 211 +1 Lines 48850 48975 +125 =========================================== + Hits 18569 18629 +60 - Misses 30281 30346 +65 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

kbwestfall commented 3 weeks ago

Do you also need to change tox.ini? Otherwise, this all looks great to me - thanks for doing this!

Can you say more? The PR has changes to tox.ini, but did I miss something?

rcooke-ast commented 3 weeks ago

Do you also need to change tox.ini? Otherwise, this all looks great to me - thanks for doing this!

Can you say more? The PR has changes to tox.ini, but did I miss something?

Ahhh... sorry - I didn't see it at the bottom of the PR. All looks good to me!!

kbwestfall commented 3 weeks ago

Do you also need to change tox.ini? Otherwise, this all looks great to me - thanks for doing this!

Can you say more? The PR has changes to tox.ini, but did I miss something?

Ahhh... sorry - I didn't see it at the bottom of the PR. All looks good to me!!

Phew! Thanks for checking.