Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ENH]: Add support for on-the-fly template space transformation #299

Merged
merged 47 commits into from
Apr 3, 2024

Conversation

synchon
Copy link
Member

@synchon synchon commented Feb 2, 2024

  • description of feature/fix
  • tests added/passed
  • add an entry for the latest changes

This PR adds support for on-the-fly template space transformation in junifer.data.get_parcellation() and junifer.data.get_mask() enabling merging of multi-space parcellation and mask. The target space is set as per the datagrabber's specification and all of the other assets are transformed to it and then applied.

Copy link

codecov bot commented Feb 2, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (e82603f) to head (78d63d6).

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##             main      #299       +/-   ##
============================================
+ Coverage   88.15%   100.00%   +11.84%     
============================================
  Files         111         1      -110     
  Lines        4820         1     -4819     
  Branches      957         0      -957     
============================================
- Hits         4249         1     -4248     
+ Misses        417         0      -417     
+ Partials      154         0      -154     
Flag Coverage Δ
docs 100.00% <ø> (ø)
junifer ?

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

see 110 files with indirect coverage changes

@synchon synchon added the enhancement New feature or request label Feb 2, 2024
@synchon synchon requested a review from fraimondo February 2, 2024 10:32
@synchon synchon mentioned this pull request Feb 6, 2024
3 tasks
@synchon
Copy link
Member Author

synchon commented Feb 6, 2024

Needs to go after #298

@synchon synchon added this to the 0.0.4 (alpha 3) milestone Feb 8, 2024
fraimondo
fraimondo previously approved these changes Feb 14, 2024
@synchon synchon force-pushed the feat/multi-mni-support branch from 51c0d78 to c11a7b3 Compare February 14, 2024 15:12
Copy link

github-actions bot commented Feb 14, 2024

PR Preview Action v1.4.7
Preview removed because the pull request was closed.
2024-04-03 16:19 UTC

@synchon synchon requested a review from fraimondo February 16, 2024 14:08
@synchon synchon force-pushed the feat/multi-mni-support branch from 1745d0f to 2465877 Compare February 21, 2024 14:49
@synchon synchon added the coordinate Issues or pull requests related to coordinates (ROIs) label Feb 22, 2024
@synchon synchon self-assigned this Feb 22, 2024
@synchon synchon added parcellation Issues or pull requests related to parcellations (atlases) mask Issues or pull requests related to masks template-space Issues or pull requests related to template spaces and removed coordinate Issues or pull requests related to coordinates (ROIs) labels Feb 22, 2024
@synchon synchon force-pushed the feat/multi-mni-support branch from aa60ee7 to 3089a7a Compare March 4, 2024 10:18
@synchon synchon force-pushed the feat/multi-mni-support branch 3 times, most recently from 1720b57 to 5211abf Compare March 15, 2024 12:25
@synchon synchon dismissed fraimondo’s stale review March 19, 2024 09:48

Stale due to new changes.

@synchon synchon force-pushed the feat/multi-mni-support branch 3 times, most recently from fce0120 to 2b4e463 Compare March 26, 2024 11:19
@synchon
Copy link
Member Author

synchon commented Mar 27, 2024

@fraimondo Can we merge this?

@fraimondo
Copy link
Contributor

@synchon Just that one question.

@synchon synchon force-pushed the feat/multi-mni-support branch from 293137b to e8f8ede Compare March 28, 2024 09:38
@synchon synchon requested a review from fraimondo March 28, 2024 09:38
synchon added 27 commits April 2, 2024 10:04
@synchon synchon force-pushed the feat/multi-mni-support branch from a6723d1 to 78d63d6 Compare April 2, 2024 08:04
@synchon synchon merged commit 407d189 into main Apr 3, 2024
26 of 28 checks passed
@synchon synchon deleted the feat/multi-mni-support branch April 3, 2024 16:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request mask Issues or pull requests related to masks parcellation Issues or pull requests related to parcellations (atlases) template-space Issues or pull requests related to template spaces
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants