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

[UK24] Pipeline reproduction (SPM - raw) #60

Open
2 of 9 tasks
cmaumet opened this issue Jul 20, 2023 · 3 comments
Open
2 of 9 tasks

[UK24] Pipeline reproduction (SPM - raw) #60

cmaumet opened this issue Jul 20, 2023 · 3 comments

Comments

@cmaumet
Copy link
Contributor

cmaumet commented Jul 20, 2023

Softwares

SPM

Input data

raw data

Additional context

None

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 file as a template if needed.
  • 🧠 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).
  • 📥 create a pull request from your code.
@cmaumet cmaumet added the 🚦 status: awaiting triage Has not been triaged & therefore, not ready for work label Jul 20, 2023
@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 Jul 21, 2023
@bclenet bclenet moved this from In progress to Backlog in NARPS Open Pipelines | Reproductions Jan 9, 2024
@cmaumet
Copy link
Contributor Author

cmaumet commented Feb 12, 2024

I am continuing this pipeline during Empenn hackathon

@bclenet bclenet moved this from Backlog to In progress in NARPS Open Pipelines | Reproductions Feb 12, 2024
@cmaumet cmaumet mentioned this issue Feb 12, 2024
8 tasks
@cmaumet cmaumet assigned bclenet and unassigned cmaumet Feb 12, 2024
@cmaumet
Copy link
Contributor Author

cmaumet commented Feb 12, 2024

This is now in a PR that is ready for you @bclenet - changing the assignee accordingly :)

@cmaumet cmaumet changed the title [UK24] Pipeline reproduction [UK24] Pipeline reproduction (SPM - raw) Feb 13, 2024
@bclenet bclenet mentioned this issue Feb 23, 2024
8 tasks
@cmaumet
Copy link
Contributor Author

cmaumet commented Mar 7, 2024

Discussed with @bclenet on March 7.

The code defined in Matlab in #161 was successfully ported into nipype by @bclenet in #179. But we end up with a set of contrasts that cannot be estimated due to colinearity (grey regressors in the SPM design matrix --> "not uniquely specified").

After looking back at the description from the team and at the original data paper, it is unclear to us how the possible_gain regressor and possible_loss regressor (mentionned in "a block-task regressor was derived respectively for all trials where the gamble could result in a possible gain, in a possible loss, and in a no-loss-nor-gain situation (i.e. 3 block-task regressors)" differ. Indeed our understanding is that all trials in which the participant choose accept (weak or strong) may result both in a possible loss and possible gain.

@cmaumet cmaumet moved this from In progress to Left aside in NARPS Open Pipelines | Reproductions Mar 7, 2024
@bclenet bclenet removed their assignment Mar 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Left aside
Development

No branches or pull requests

2 participants