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: Finalize missing workbench commands #54

Merged
merged 4 commits into from
May 3, 2021

Conversation

mgxd
Copy link
Collaborator

@mgxd mgxd commented May 3, 2021

Closes #49

These should all eventually just be upstreamed to Nipype directly.

@pep8speaks
Copy link

pep8speaks commented May 3, 2021

Hello @mgxd! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻

Comment last updated at 2021-05-03 19:31:57 UTC

@codecov-commenter
Copy link

codecov-commenter commented May 3, 2021

Codecov Report

Merging #54 (0a5f9cf) into master (90e6e41) will increase coverage by 2.08%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #54      +/-   ##
==========================================
+ Coverage   29.64%   31.73%   +2.08%     
==========================================
  Files          42       42              
  Lines        3140     3236      +96     
==========================================
+ Hits          931     1027      +96     
  Misses       2209     2209              
Impacted Files Coverage Δ
nibabies/interfaces/conftest.py 100.00% <ø> (ø)
nibabies/interfaces/workbench.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 90e6e41...0a5f9cf. Read the comment docs.

@mgxd mgxd force-pushed the enh/wb-commands-49 branch from a4d0c71 to 0a5f9cf Compare May 3, 2021 19:31
@mgxd mgxd merged commit 5d22fb0 into nipreps:master May 3, 2021
@mgxd mgxd deleted the enh/wb-commands-49 branch May 3, 2021 19:36
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.

Piecewise subcortical alignment for CIFTI mapping
3 participants