[ENH]: Introduce _ApplyWarper #266

Merged
synchon merged 12 commits from feat/apply-warper into main 2023-10-24 12:30:51 +00:00
synchon commented 2023-10-17 17:07:48 +00:00 (Migrated from github.com)
  • description of feature/fix
  • tests added/passed
  • add an entry for the latest changes

This PR introduces _ApplyWarper as a generic FSL applywarp wrapper class for internal use (as of now).

* [x] description of feature/fix * [x] tests added/passed * [x] add an entry for the latest changes This PR introduces `_ApplyWarper` as a generic FSL `applywarp` wrapper class for internal use (as of now).
codecov[bot] commented 2023-10-17 17:13:54 +00:00 (Migrated from github.com)

Codecov Report

Merging #266 (3d91e84) into main (6c1e04e) will decrease coverage by 0.72%.
The diff coverage is 38.46%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #266      +/-   ##
==========================================
- Coverage   92.35%   91.63%   -0.72%     
==========================================
  Files          86       87       +1     
  Lines        3846     3898      +52     
  Branches      753      757       +4     
==========================================
+ Hits         3552     3572      +20     
- Misses        188      219      +31     
- Partials      106      107       +1     
Flag Coverage Δ
docs 100.00% <ø> (ø)
junifer 91.63% <38.46%> (-0.72%) ⬇️

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

Files Coverage Δ
junifer/preprocess/__init__.py 100.00% <ø> (ø)
junifer/preprocess/fsl/apply_warper.py 38.46% <38.46%> (ø)
## [Codecov](https://app.codecov.io/gh/juaml/junifer/pull/266?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml) Report > Merging [#266](https://app.codecov.io/gh/juaml/junifer/pull/266?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml) (3d91e84) into [main](https://app.codecov.io/gh/juaml/junifer/commit/6c1e04ee276ec32c3e3d606be83441c3add54649?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml) (6c1e04e) will **decrease** coverage by `0.72%`. > The diff coverage is `38.46%`. [![Impacted file tree graph](https://app.codecov.io/gh/juaml/junifer/pull/266/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/266?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml) ```diff @@ Coverage Diff @@ ## main #266 +/- ## ========================================== - Coverage 92.35% 91.63% -0.72% ========================================== Files 86 87 +1 Lines 3846 3898 +52 Branches 753 757 +4 ========================================== + Hits 3552 3572 +20 - Misses 188 219 +31 - Partials 106 107 +1 ``` | [Flag](https://app.codecov.io/gh/juaml/junifer/pull/266/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/266/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/266/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml) | `91.63% <38.46%> (-0.72%)` | :arrow_down: | 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/266?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml) | Coverage Δ | | |---|---|---| | [junifer/preprocess/\_\_init\_\_.py](https://app.codecov.io/gh/juaml/junifer/pull/266?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml#diff-anVuaWZlci9wcmVwcm9jZXNzL19faW5pdF9fLnB5) | `100.00% <ø> (ø)` | | | [junifer/preprocess/fsl/apply\_warper.py](https://app.codecov.io/gh/juaml/junifer/pull/266?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml#diff-anVuaWZlci9wcmVwcm9jZXNzL2ZzbC9hcHBseV93YXJwZXIucHk=) | `38.46% <38.46%> (ø)` | |
github-actions[bot] commented 2023-10-17 17:13:54 +00:00 (Migrated from github.com)
PR Preview Action v1.4.4
Preview removed because the pull request was closed.
2023-10-24 12:36 UTC
[PR Preview Action](https://github.com/rossjrw/pr-preview-action) v1.4.4 :---: Preview removed because the pull request was closed. 2023-10-24 12:36 UTC <!-- Sticky Pull Request Commentpr-preview -->
fraimondo (Migrated from github.com) approved these changes 2023-10-18 09:16:17 +00:00
Sign in to join this conversation.
No reviewers
No milestone
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
juaml/junifer!266
No description provided.