nipreps / fmripost-aroma

Run ICA-AROMA on fMRIPrep derivatives
https://fmripost-aroma.readthedocs.io/
Apache License 2.0
7 stars 1 forks source link

Draft CircleCI configuration #43

Closed tsalo closed 1 month ago

tsalo commented 1 month ago

Closes #42.

Pending issues:

Changes proposed in this pull request

codecov[bot] commented 1 month ago

Codecov Report

Attention: Patch coverage is 15.50388% with 109 lines in your changes missing coverage. Please review.

Project coverage is 30.54%. Comparing base (4bd4e0c) to head (550be7f). Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
src/fmripost_aroma/tests/utils.py 18.51% 88 Missing :warning:
src/fmripost_aroma/workflows/base.py 0.00% 12 Missing :warning:
src/fmripost_aroma/cli/run.py 0.00% 3 Missing :warning:
src/fmripost_aroma/cli/workflow.py 0.00% 2 Missing :warning:
src/fmripost_aroma/utils/bids.py 0.00% 1 Missing and 1 partial :warning:
src/fmripost_aroma/cli/parser.py 0.00% 1 Missing :warning:
src/fmripost_aroma/workflows/resampling.py 0.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #43 +/- ## ========================================== - Coverage 31.18% 30.54% -0.64% ========================================== Files 26 26 Lines 2155 2259 +104 Branches 327 358 +31 ========================================== + Hits 672 690 +18 - Misses 1463 1549 +86 Partials 20 20 ```

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

tsalo commented 1 month ago

@effigies would you be willing to activate fmripost-aroma on CircleCI?

effigies commented 1 month ago

Unfortunately, we need to hear back about a no-cost extension before committing more money to CircleCI. Would you be willing to run it on your fork?

tsalo commented 1 month ago

I can do that, although I'm not sure how the Docker deployment would work.

tsalo commented 1 month ago

I can just have my fork deploy I guess. I am worried about deploying from tags, but I can just push to unstable for now.

effigies commented 1 month ago

For docker, we could deploy via GitHub actions.

tsalo commented 1 month ago

That works for me. Can you either add me to the nipreps org on DockerHub (username tsalo) or add your credentials as secrets on this repo? The action I'm adding in #46 needs DOCKER_USERNAME and DOCKER_PASSWORD.