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

[C88N] Pipeline reproduction (SPM - deriv) #115

Open
8 of 9 tasks
bclenet opened this issue Oct 3, 2023 · 1 comment · Fixed by #116
Open
8 of 9 tasks

[C88N] Pipeline reproduction (SPM - deriv) #115

bclenet opened this issue Oct 3, 2023 · 1 comment · Fixed by #116
Labels
deriv ✨ goal: improvement Improvement to an existing feature 🤝 help wanted Extra attention is needed SPM

Comments

@bclenet
Copy link
Collaborator

bclenet commented Oct 3, 2023

Softwares

SPM

Input data

derivatives (fMRIprep)

Additional context

The current code allows to get the following correlation results (hypothesis 1 to 9).

0.911169161
0.919511409
0.911169161
0.919511409
0.920257494
0.869181931
0.920257494
0.869181931
0.985404891

These results could be improved : some debugging may be done.

List of tasks

Please tick the boxes below once the corresponding task is finished. 👍

  • 👌 A maintainer of the project approved the issue, by assigning a 🏁status: ready for dev label to it.
  • 🌳 Create a branch on your fork to start the reproduction.
  • 🌅 Create a file team_{team_id}.py inside the narps_open/pipelines/ directory. You can use a file inside narps_open/pipelines/templates as a template if needed.
  • 📥 Create a pull request as soon as you completed the previous task.
  • 🧠 Write the code for the pipeline, using Nipype and the file architecture described in docs/pipelines.md.
  • 📘 Make sure your code is documented enough.
  • 🐍 Make sure your code is explicit and conforms with PEP8.
  • 🔬 Create tests for your pipeline. You can use files in tests/pipelines/test_team_* as examples.
  • 🔬 Make sure your code passes all the tests you created (see docs/testing.md).
@bclenet bclenet added the 🏁 status: ready for dev Ready for work label Oct 3, 2023
@bclenet bclenet mentioned this issue Oct 3, 2023
8 tasks
@bclenet bclenet linked a pull request Oct 3, 2023 that will close this issue
8 tasks
@bclenet bclenet added the 🤝 help wanted Extra attention is needed label Jan 4, 2024
@bclenet
Copy link
Collaborator Author

bclenet commented Jan 4, 2024

PR #116 Merged to main : C88N refactored.

The code allows to get the following correlation results (hypothesis 1 to 9).
0.911169161
0.919511409
0.911169161
0.919511409
0.920257494
0.869181931
0.920257494
0.869181931
0.985404891

@bclenet bclenet reopened this Jan 4, 2024
@bclenet bclenet added the ✨ goal: improvement Improvement to an existing feature label Jan 5, 2024
@bclenet bclenet moved this from Done to In review in NARPS Open Pipelines | Reproductions Jan 9, 2024
@cmaumet cmaumet changed the title [C88N] Pipeline reproduction [C88N] Pipeline reproduction (SPM - deriv) Feb 12, 2024
@bclenet bclenet moved this from Needs improvement to Done in NARPS Open Pipelines | Reproductions Apr 3, 2024
@bclenet bclenet removed the 🏁 status: ready for dev Ready for work label Dec 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deriv ✨ goal: improvement Improvement to an existing feature 🤝 help wanted Extra attention is needed SPM
Projects
Development

Successfully merging a pull request may close this issue.

2 participants