[ENH]: Enable `compute_brain_mask` to use subject-specific probseg files #404

Merged

fraimondo merged 4 commits from enh/compute_subject_gm_mask into main

2024-12-06 11:39:34 +00:00

fraimondo commented

2024-11-25 18:00:28 +00:00

(Migrated from github.com)

This PR implements #400

github-actions[bot] commented

2024-11-25 18:15:45 +00:00

(Migrated from github.com)

PR Preview Action v1.4.8
Preview removed because the pull request was closed.
2024-12-06 11:52 UTC

[PR Preview Action](https://github.com/rossjrw/pr-preview-action) v1.4.8 :---: Preview removed because the pull request was closed. 2024-12-06 11:52 UTC

codecov[bot] commented

2024-11-25 18:35:53 +00:00

(Migrated from github.com)

Attention: Patch coverage is 0% with 19 lines in your changes missing coverage. Please review.

Project coverage is 0.01%. Comparing base (ebe985e) to head (1ea8ce4).
Report is 5 commits behind head on main.

Files with missing lines	Patch %	Lines
junifer/data/masks/_masks.py	0.00%	19 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff            @@
##            main    #404      +/-   ##
========================================
- Coverage   0.01%   0.01%   -0.01%     
========================================
  Files        133     133              
  Lines       5698    5714      +16     
========================================
  Hits           1       1              
- Misses      5697    5713      +16

Flag	Coverage Δ
docs	`100.00% <ø> (ø)`
junifer	`0.00% <0.00%> (ø)`

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines	Coverage Δ
junifer/data/masks/_masks.py	`0.00% <0.00%> (ø)`

## [Codecov](https://app.codecov.io/gh/juaml/junifer/pull/404?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml) Report Attention: Patch coverage is `0%` with `19 lines` in your changes missing coverage. Please review. > Project coverage is 0.01%. Comparing base [(`ebe985e`)](https://app.codecov.io/gh/juaml/junifer/commit/ebe985ed95ec0d2b15ddd54fb952bcd2882d8fb3?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml) to head [(`1ea8ce4`)](https://app.codecov.io/gh/juaml/junifer/commit/1ea8ce42eceb7e5aa0a0ad6344b51490b37e6fe3?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml). > Report is 5 commits behind head on main. | [Files with missing lines](https://app.codecov.io/gh/juaml/junifer/pull/404?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml) | Patch % | Lines | |---|---|---| | [junifer/data/masks/\_masks.py](https://app.codecov.io/gh/juaml/junifer/pull/404?src=pr&el=tree&filepath=junifer%2Fdata%2Fmasks%2F_masks.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml#diff-anVuaWZlci9kYXRhL21hc2tzL19tYXNrcy5weQ==) | 0.00% | [19 Missing :warning: ](https://app.codecov.io/gh/juaml/junifer/pull/404?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml) | <details><summary>Additional details and impacted files</summary> [![Impacted file tree graph](https://app.codecov.io/gh/juaml/junifer/pull/404/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/404?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml) ```diff @@ Coverage Diff @@ ## main #404 +/- ## ======================================== - Coverage 0.01% 0.01% -0.01% ======================================== Files 133 133 Lines 5698 5714 +16 ======================================== Hits 1 1 - Misses 5697 5713 +16 ``` | [Flag](https://app.codecov.io/gh/juaml/junifer/pull/404/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/404/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml) | `100.00% <ø> (ø)` | | | [junifer](https://app.codecov.io/gh/juaml/junifer/pull/404/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml) | `0.00% <0.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 with missing lines](https://app.codecov.io/gh/juaml/junifer/pull/404?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml) | Coverage Δ | | |---|---|---| | [junifer/data/masks/\_masks.py](https://app.codecov.io/gh/juaml/junifer/pull/404?src=pr&el=tree&filepath=junifer%2Fdata%2Fmasks%2F_masks.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml#diff-anVuaWZlci9kYXRhL21hc2tzL19tYXNrcy5weQ==) | `0.00% <0.00%> (ø)` | | </details>

synchon (Migrated from github.com) approved these changes

2024-12-03 12:22:59 +00:00