search

spacetelescope / stdatamodels

https://stdatamodels.readthedocs.io
Other
5 stars 24 forks source link

enable devdeps testing #208

Closed braingram closed 11 months ago

braingram commented 12 months ago

This PR enables devdeps testing in github actions allowing us to run tests against numpy 2.0 and other development versions of some key dependencies.

requirements-dev.txt is added, listing several key dependencies (asdf, astropy, asdf-astropy numpy) and configuring pip to install development versions (either from git or from available built development versions). scipy is not a direct dependency of stdatmodels but is installed in requirements-dev.txt as it's unlikely the development version of numpy will work with the released scipy.

tox.ini required minor updates to use the new requirements-dev.txt (an existing devdeps environment was already configured).

ci.yml was updated to run the devdeps tox environment which appears as test-devdeps-xdist. See a run triggered by this PR: https://github.com/spacetelescope/stdatamodels/actions/runs/6150774039/job/16689450474?pr=208

test-jwst-devdeps-xdist is also run (jwst tests are run in the CI and contribute to stdatamodels coverage due to a large part of stdatamodels.jwst coming directly from jwst). However these tests are more likely to fail due to extra dependencies included for jwst.

Checklist

codecov[bot] commented 12 months ago

Codecov Report

Patch has no changes to coverable lines.

:loudspeaker: Thoughts on this report? Let us know!.

braingram commented 12 months ago

As this doesn't change any code or documentation (only CI configuration) I'm inclined to skip regtests to not take up time on the server.

braingram commented 11 months ago

As this doesn't change any code or documentation (only CI configuration) I'm inclined to skip regtests to not take up time on the server.

@hbushouse any objection merging this without regtests since it doesn't change code or docs (only CI)?

hbushouse commented 11 months ago

As this doesn't change any code or documentation (only CI configuration) I'm inclined to skip regtests to not take up time on the server.

@hbushouse any objection merging this without regtests since it doesn't change code or docs (only CI)?

Oh sure, "transparent to the user". We've heard that before ... ;-)