MAINT: Adopt tox for testing environment configuration
#234
Conversation
jhlegarreta
force-pushed
the
AdoptToxTestingEnvConfig
branch
from
f3341ab to
17a4546
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #234 +/- ##
=======================================
Coverage 63.97% 63.97%
=======================================
Files 18 18
Lines 905 905
Branches 154 164 +10
=======================================
Hits 579 579
Misses 282 282
Partials 44 44 ☔ View full report in Codecov by Sentry. |
jhlegarreta
force-pushed
the
AdoptToxTestingEnvConfig
branch
8 times, most recently
from
b93c5ce to
77f2b97
Compare
|
@oesteban I am missing something with ANTs: looks like ANTs is not being searched/I am not passing the I have checked where miniconda is downloading it: https://github.com/nipreps/eddymotion/actions/runs/11197535087/job/31127651103#step:6:1 And I believe I am trying to pass its value to Do you happen to know off the top of your head how to fix this? Thanks. Edit: Fixed the issues with Ready to be merged. |
jhlegarreta
force-pushed
the
AdoptToxTestingEnvConfig
branch
4 times, most recently
from
87b7873 to
0303064
Compare
Adopt `tox` for testing environment configuration as per decision at the Oct 1 2024 Technical Monitoring meeting. Add the necessary `tox.ini` file and edit the GHA `test.yml` workflow file to use `tox`, to use `ants`, to use `DataLad` to download the testing data, and to cache the data across the build matrix. Pin `nitransforms` to <24 following commit d26541b. The decision on pinning to <24 was made following consistent failures on `test_proximity_estimator_trivial_model`, e.g. https://github.com/nipreps/eddymotion/actions/runs/11123784605/job/30907972182#step:8:196 and https://github.com/nipreps/eddymotion/actions/runs/11203922080/job/31141711676?pr=234#step:12:342 The version was pinned for the conda `env.yml` config file, but not `pyproject.toml`.
Increase consistency in dependency pinning specification: remove spaces and adopt compact notation for `nipype` in `pyproject.toml` as used for the rest of the dependencies.
jhlegarreta
force-pushed
the
AdoptToxTestingEnvConfig
branch
from
0303064 to
342c7fc
Compare
Adopt
toxfor testing environment configuration as per decision at the Oct 1 2024 Technical Monitoring meeting.Add the necessary
tox.inifile and edit the GHAtest.ymlworkflow file to usetox, to useants, to useDataLadto download the testing data, and to cache the data across the build matrix.Pin
nitransformsto <24 following commit d26541b. The decision on pinning to <24 was made following consistent failures ontest_proximity_estimator_trivial_model, e.g.https://github.com/nipreps/eddymotion/actions/runs/11123784605/job/30907972182#step:8:196
and
https://github.com/nipreps/eddymotion/actions/runs/11203922080/job/31141711676?pr=234#step:12:342
The version was pinned for the conda
env.ymlconfig file, but notpyproject.toml.