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

[R9K3] Pipeline reproduction (SPM, deriv) #98

Closed
3 of 9 tasks
Remi-Gau opened this issue Sep 21, 2023 · 2 comments
Closed
3 of 9 tasks

[R9K3] Pipeline reproduction (SPM, deriv) #98

Remi-Gau opened this issue Sep 21, 2023 · 2 comments

Comments

@Remi-Gau
Copy link
Contributor

Remi-Gau commented Sep 21, 2023

Softwares

SPM

Input data

derivatives (fMRIprep)

Additional context

This pipeline already exist but is incomplete and its structure does not conform to the update code architecture.

Opening an issue to track the refactoring and extension of its code.

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).
@Remi-Gau Remi-Gau added the 🚦 status: awaiting triage Has not been triaged & therefore, not ready for work label Sep 21, 2023
@Remi-Gau Remi-Gau mentioned this issue Aug 29, 2023
8 tasks
@bclenet bclenet added 🏁 status: ready for dev Ready for work and removed 🚦 status: awaiting triage Has not been triaged & therefore, not ready for work labels Sep 21, 2023
@bclenet bclenet moved this from In progress to Backlog in NARPS Open Pipelines | Reproductions Jan 9, 2024
@cmaumet cmaumet changed the title [R9K3] Pipeline reproduction [R9K3] Pipeline reproduction (SPM, derivatives) Feb 12, 2024
@cmaumet cmaumet changed the title [R9K3] Pipeline reproduction (SPM, derivatives) [R9K3] Pipeline reproduction (SPM, deriv) Feb 12, 2024
@bclenet bclenet moved this from Backlog to In progress in NARPS Open Pipelines | Reproductions Feb 26, 2024
@bclenet bclenet self-assigned this Feb 26, 2024
@bclenet
Copy link
Collaborator

bclenet commented Feb 26, 2024

I am currently working on this pipeline.

@bclenet bclenet mentioned this issue Feb 26, 2024
8 tasks
@bclenet
Copy link
Collaborator

bclenet commented Mar 15, 2024

Looks promising !
Current correlation values :

Team Number of subjects Test status Correlation values
R9K3 20 success [0.60, 0.38, 0.60, 0.38, 0.50, 0.42, 0.50, 0.42, 0.37]
R9K3 40 failure [0.76, 0.66, 0.76, 0.66, 0.76, 0.57, 0.76, 0.57, 0.65]
R9K3 60 failure [0.83, 0.77, 0.83, 0.77, 0.85, 0.68, 0.85, 0.68, 0.77]
R9K3 80 failure [0.87, 0.81, 0.87, 0.81, 0.88, 0.71, 0.88, 0.71, 0.79]
R9K3 108 failure [0.91, 0.88, 0.91, 0.88, 0.92, 0.82, 0.92, 0.82, 0.89]

Tested with 102 subject (6 subject excluded by the team)

Team Number of subjects Correlation values
R9K3 102 [0.92, 0.93, 0.92, 0.93, 0.92, 0.89, 0.92, 0.89, 0.92]

@bclenet bclenet closed this as completed Mar 18, 2024
@bclenet bclenet removed their assignment Mar 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

No branches or pull requests

3 participants