Skip to content

Conversation

mgxd
Copy link
Collaborator

@mgxd mgxd commented Feb 2, 2022

Addresses problem raised in #149

Initial implementation using a custom Function, but this may be worth a proper interface.

Adds a new interface, MapLabels, to handle label remapping.

Interfaces within nibabies.interfaces.nibabel (BinaryDilation, MergeROIs, and MapLabels were ported into niworkflows 1.5.0.

Adds label remapping step to CIFTI generation.

Addresses problem raised in nipreps#149

Initial implementation using a custom `Function`, but this may be worth a proper interface.
@codecov-commenter
Copy link

codecov-commenter commented Feb 2, 2022

Codecov Report

Merging #186 (a1aefbb) into master (be1e869) will increase coverage by 1.26%.
The diff coverage is 95.65%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #186      +/-   ##
==========================================
+ Coverage   32.10%   33.36%   +1.26%     
==========================================
  Files          45       45              
  Lines        3728     3779      +51     
==========================================
+ Hits         1197     1261      +64     
+ Misses       2531     2518      -13     
Impacted Files Coverage Δ
nibabies/workflows/bold/alignment.py 27.69% <0.00%> (+3.94%) ⬆️
nibabies/interfaces/nibabel.py 83.16% <97.50%> (+9.39%) ⬆️
nibabies/interfaces/tests/test_nibabel.py 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update be1e869...a1aefbb. Read the comment docs.

@mgxd mgxd force-pushed the enh/cifti-robustness branch from ffa6b4b to 20d0517 Compare February 2, 2022 22:27
@mgxd mgxd marked this pull request as ready for review February 2, 2022 22:32
@mgxd mgxd requested review from effigies and oesteban February 3, 2022 14:44
@mgxd mgxd added this to the 22.0.0 milestone Feb 7, 2022
mgxd added 2 commits February 7, 2022 16:02
Since nibabel interfaces have been ported over, they can be removed from this codebase.
@mgxd mgxd merged commit 734c4a4 into nipreps:master Feb 8, 2022
@mgxd mgxd deleted the enh/cifti-robustness branch February 8, 2022 21:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants