Conversation

effigies

Summary

CI is broken because nipy is not being updated to deal with numpy deprecations. This removes nipy tests generally. I'm seeing if I can add a requirement config that will cap numpy for now so we don't skip these tests entirely.

@codecov

Codecov Report

Base: 47.56% // Head: 63.63% // Increases project coverage by +16.07% 🎉

Coverage data is based on head (e4c0fe6) compared to base (f98ce46).
has no changes to coverable lines.

Additional details and impacted files
@@             Coverage Diff             @@
##           master    #3532       +/-   ##
===========================================
+ Coverage   47.56%   63.63%   +16.07%     
===========================================
  Files         309      309               
  Lines       40844    40873       +29     
  Branches     5381     5381               
===========================================
+ Hits        19427    26011     +6584     
+ Misses      21299    13838     -7461     
- Partials      118     1024      +906     
Impacted FilesCoverage Δ
nipype/pipeline/plugins/pbs.py16.90% <0.00%> (-0.25%)⬇️
nipype/pipeline/plugins/condor.py15.94% <0.00%> (-0.24%)⬇️
nipype/pipeline/plugins/lsf.py16.21% <0.00%> (-0.23%)⬇️
nipype/pipeline/plugins/oar.py18.60% <0.00%> (-0.22%)⬇️
nipype/pipeline/plugins/slurm.py13.25% <0.00%> (-0.17%)⬇️
nipype/pipeline/plugins/sgegraph.py11.36% <0.00%> (-0.14%)⬇️
nipype/pipeline/plugins/slurmgraph.py10.98% <0.00%> (-0.13%)⬇️
nipype/interfaces/mrtrix/convert.py15.73% <0.00%> (-0.09%)⬇️
nipype/interfaces/minc/minc.py85.20% <0.00%> (+0.01%)⬆️
nipype/algorithms/mesh.py29.70% <0.00%> (+0.49%)⬆️
... and 118 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@effigieseffigies merged commit 7382b3d into nipy:master Dec 27, 2022
Sign up for free to join this conversation on . Already have an account? Sign in to comment
None yet
None yet

Successfully merging this pull request may close these issues.

@effigies