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

[narps_open_pipelines#220][94GU] Preprocessing #230

Open

youennmerel wants to merge 4 commits into Inria-Empenn:main from youennmerel:94GU_reproduction

youennmerel commented Dec 18, 2024

This Pull Request is related to issue #220.
Changes proposed in this Pull Request:

Init implementation of pipeline 94GU
Add preprocessing
- successfully tested locally with 1 subject

Checklist:

Descriptive title
Targets the main branch
Changes are functional
My code is explicit and comments were added to it
Code conforms with PEP8
Tests were added for the changes and they complete successfully
Existing tests were updated (if needed) and they complete successfully
Documentation was updated

youennmerel added 2 commits

December 18, 2024 14:43


          [narps_open_pipelines#220][94GU] Preprocessing

2c10287


          [narps_open_pipelines#220][94GU] remove unused imports

e30dc86

youennmerel mentioned this pull request

[94GU] Pipeline reproduction (SPM - raw) #220

Open

9 tasks


          [narps_open_pipelines#220][94GU] correct code spells

7dfb2e6

Collaborator

bclenet commented Dec 20, 2024

Thanks @youennmerel !
Are you ok to implement unit test for the methods you created ?


          [narps_open_pipelines#220][94GU] Add unit tests

054fc36

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

Labels

None yet