juaml / junifer

Forschungszentrum Jülich Neuroimaging Feature Extractor
https://juaml.github.io/junifer
GNU Affero General Public License v3.0
14 stars 13 forks source link

[CI]: Improve CI #318

Closed synchon closed 2 months ago

synchon commented 5 months ago

The CI is acting up due to AFNI tests failing (ReHo and (f)ALFF) and FSL not being installed correctly. Although the latter is being avoided now, the former is not and thus coverage is not being reported. We need to come up with a flexible solution to handle situations like this.

github-actions[bot] commented 2 months ago

PR Preview Action v1.4.7 :---: Preview removed because the pull request was closed. 2024-06-21 09:17 UTC

codecov[bot] commented 2 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 86.64%. Comparing base (b890e54) to head (8d0956c).

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/juaml/junifer/pull/318/graphs/tree.svg?width=650&height=150&src=pr&token=5H21JuZXMw&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml)](https://app.codecov.io/gh/juaml/junifer/pull/318?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml) ```diff @@ Coverage Diff @@ ## main #318 +/- ## ============================================ - Coverage 100.00% 86.64% -13.36% ============================================ Files 1 114 +113 Lines 1 5137 +5136 Branches 0 999 +999 ============================================ + Hits 1 4451 +4450 - Misses 0 528 +528 - Partials 0 158 +158 ``` | [Flag](https://app.codecov.io/gh/juaml/junifer/pull/318/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml) | Coverage Δ | | |---|---|---| | [docs](https://app.codecov.io/gh/juaml/junifer/pull/318/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml) | `100.00% <ø> (ø)` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml#carryforward-flags-in-the-pull-request-comment) to find out more. | [Files](https://app.codecov.io/gh/juaml/junifer/pull/318?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml) | Coverage Δ | | |---|---|---| | [junifer/markers/brainprint.py](https://app.codecov.io/gh/juaml/junifer/pull/318?src=pr&el=tree&filepath=junifer%2Fmarkers%2Fbrainprint.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml#diff-anVuaWZlci9tYXJrZXJzL2JyYWlucHJpbnQucHk=) | `40.94% <100.00%> (ø)` | | | [junifer/markers/falff/\_afni\_falff.py](https://app.codecov.io/gh/juaml/junifer/pull/318?src=pr&el=tree&filepath=junifer%2Fmarkers%2Ffalff%2F_afni_falff.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml#diff-anVuaWZlci9tYXJrZXJzL2ZhbGZmL19hZm5pX2ZhbGZmLnB5) | `37.83% <ø> (ø)` | | | [junifer/markers/reho/\_afni\_reho.py](https://app.codecov.io/gh/juaml/junifer/pull/318?src=pr&el=tree&filepath=junifer%2Fmarkers%2Freho%2F_afni_reho.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml#diff-anVuaWZlci9tYXJrZXJzL3JlaG8vX2FmbmlfcmVoby5weQ==) | `35.89% <ø> (ø)` | | | [junifer/pipeline/pipeline\_step\_mixin.py](https://app.codecov.io/gh/juaml/junifer/pull/318?src=pr&el=tree&filepath=junifer%2Fpipeline%2Fpipeline_step_mixin.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml#diff-anVuaWZlci9waXBlbGluZS9waXBlbGluZV9zdGVwX21peGluLnB5) | `97.72% <100.00%> (ø)` | | | [junifer/preprocess/smoothing/\_afni\_smoothing.py](https://app.codecov.io/gh/juaml/junifer/pull/318?src=pr&el=tree&filepath=junifer%2Fpreprocess%2Fsmoothing%2F_afni_smoothing.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml#diff-anVuaWZlci9wcmVwcm9jZXNzL3Ntb290aGluZy9fYWZuaV9zbW9vdGhpbmcucHk=) | `39.13% <ø> (ø)` | | | [junifer/utils/logging.py](https://app.codecov.io/gh/juaml/junifer/pull/318?src=pr&el=tree&filepath=junifer%2Futils%2Flogging.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml#diff-anVuaWZlci91dGlscy9sb2dnaW5nLnB5) | `72.00% <100.00%> (ø)` | | ... and [107 files with indirect coverage changes](https://app.codecov.io/gh/juaml/junifer/pull/318/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml)