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.

Sign up for GitHub

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

Jump to bottom

List MRtrix modules of interest #45

Closed
AlexVCaron opened this issue Nov 21, 2024 · 2 comments · Fixed by #67, #71 or #78
Closed

List MRtrix modules of interest #45

AlexVCaron opened this issue Nov 21, 2024 · 2 comments · Fixed by #67, #71 or #78
Assignees
@arnaudbore
Labels
hackathon

Comments

@AlexVCaron
Copy link
Collaborator

Create a list of interesting modules from MRtrix to implement into nf-neuro.
@arnaudbore
Copy link
Contributor

Here are some directions to improve our current modules:

betcrop/antsbet - everything mrtrix or scilpy
betcrop/cropvolume (does not bet should go in image) - everything mrtrix or scilpy
betcrop/fslbetcrop remove scilpy
denoising/mppca remove scilpy
image/applymask - everything mrtrix
image/mrconvert - everything mrtrix
preproc/gibbs - everything mrtrix
preproc/normalize - everything mrtrix

Suggestions:
image/denoising: add nlsam ? https://github.com/samuelstjean/nlsam/releases
image/resample can become image/scil_resample and we can have a image/mrtrix_resample

@Manonedde
Copy link
Contributor

Manonedde commented Dec 6, 2024 •
edited by arnaudbore
Loading

Tasks assignment:

If we have time:

This was linked to pull requests Dec 12, 2024
#45 - MRtrix module - image/applymask #67
Merged
#45 - MRtrix module - betcrop/betantsbet #71
Merged
#45 - MRtrix module - betcrop/fslbetcrop [WIP] #72
Closed
#45 - Change category cropvolume module (move to image) [WIP] #78
Merged
github-merge-queue bot pushed a commit that referenced this issue Dec 18, 2024
@arnaudbore
Merge pull request #67 from Manonedde/fix_mrtrix_applymask
6951c58 
#45 - MRtrix module - image/applymask
github-merge-queue bot pushed a commit that referenced this issue Dec 18, 2024
@arnaudbore
Merge pull request #76 from ThoumyreStanislas/Mrtrix_Gibbs
a7dde7f 
#45 MRtrix module - preproc/gibbs
github-merge-queue bot pushed a commit that referenced this issue Dec 18, 2024
@arnaudbore
Merge pull request #75 from ThoumyreStanislas/mrtrix_nlmeans
9bd2a96 
#45 MRtrix module - denoising/mppca
AlexVCaron added a commit that referenced this issue Jan 15, 2025
@AlexVCaron
Merge pull request #71 from Manonedde/fix_mrtrix_betantsbet
5c2bc5c 
#45 - MRtrix module - betcrop/betantsbet
github-merge-queue bot pushed a commit that referenced this issue Jan 15, 2025
@AlexVCaron
Merge pull request #78 from Manonedde/fix_betcropvolume
38949cc 
#45 - Change category cropvolume module (move to image) [WIP]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@arnaudbore arnaudbore

Labels
hackathon
Projects
None yet
Milestone
No milestone
3 participants
@arnaudbore @AlexVCaron @Manonedde