search

nipreps / mriqc

Automated Quality Control and visual reports for Quality Assessment of structural (T1w, T2w) and functional MRI of the brain
http://mriqc.readthedocs.io
Apache License 2.0
299 stars 132 forks source link

CI: Simplify GitHub actions checks and update action versions #1141

Closed effigies closed 1 year ago

effigies commented 1 year ago

This drops old pip and use of venv for build, and uses pipx for tools to eliminate some pip installs altogether.

It also drops the pip cache. This is generally not a bottleneck, and I'm guessing it's the cause of the space limitations @celprov mentioned in #1137.

effigies commented 1 year ago

I was wrong, the cache was not the bottleneck, the 3 venvs with massive wheels unpacked into them were.

effigies commented 1 year ago

On the other hand, the majority of the time of pip install does not seem to be downloading. Installation seems to take 2-3 minutes regardless of caching, and caching itself takes about 2 minutes. I'm a little inclined to re-disable it.

effigies commented 1 year ago

Canceling CircleCI since that seems broken and irrelevant to this PR.