Skip to content

[ENH]: Add CAT12 SANLM denoising filter - #3374

Merged
effigies merged 25 commits into
nipy:masterfrom
0rC0:cat12_sanlm
Oct 18, 2021
Merged

[ENH]: Add CAT12 SANLM denoising filter#3374
effigies merged 25 commits into
nipy:masterfrom
0rC0:cat12_sanlm

Conversation

@0rC0

@0rC0 0rC0 commented Sep 12, 2021

Copy link
Copy Markdown
Contributor

Summary

Fixes #2783.

List of changes proposed in this PR (pull-request)

Acknowledgment

  • [X ] (Mandatory) I acknowledge that this contribution will be available under the Apache 2 license.

@effigies effigies left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this! Here's some quick comments. Please also apply black styling and run make specs.

Comment thread nipype/interfaces/cat12/preprocess.py
Comment thread nipype/interfaces/cat12/preprocess.py
Comment thread nipype/interfaces/cat12/preprocess.py Outdated
Comment thread nipype/interfaces/cat12/preprocess.py Outdated
Comment thread nipype/interfaces/cat12/preprocess.py Outdated
Comment thread nipype/interfaces/cat12/preprocess.py Outdated
Comment thread nipype/interfaces/cat12/preprocess.py Outdated
@0rC0

0rC0 commented Sep 30, 2021

Copy link
Copy Markdown
Contributor Author

Thank you for the review @effigies and sorry for the delay, I had a lot to work in the past days.

I'm going to work again on the PR next week!

BW

@0rC0 0rC0 changed the title **[WIP]** [ENH]: Add CAT12 SANLM denoising filter [ENH]: Add CAT12 SANLM denoising filter Oct 10, 2021

@effigies effigies left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is what I was thinking for spm_type:

Comment thread nipype/interfaces/cat12/preprocess.py
Comment thread nipype/interfaces/cat12/preprocess.py Outdated
@effigies

Copy link
Copy Markdown
Member

Can you merge/rebase master to fix the tests? Also, you still need to run make specs.

@effigies effigies mentioned this pull request Oct 15, 2021
15 tasks
0rC0 and others added 4 commits October 17, 2021 20:47
Co-authored-by: Chris Markiewicz <effigies@gmail.com>
Co-authored-by: Chris Markiewicz <effigies@gmail.com>
Comment thread nipype/interfaces/cat12/preprocess.py Outdated
Co-authored-by: Chris Markiewicz <effigies@gmail.com>
@effigies
effigies merged commit 071e575 into nipy:master Oct 18, 2021
@0rC0

0rC0 commented Oct 18, 2021

Copy link
Copy Markdown
Contributor Author

Super! :-) and thanks for the help @effigies

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants