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

adding Le Petit Prince to public repo #3

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from
Draft

adding Le Petit Prince to public repo #3

wants to merge 5 commits into from

Conversation

man-shu
Copy link
Collaborator

@man-shu man-shu commented Jul 30, 2022

Mods (relative to the version on private cognitive_protocols repo):

  • some reorganisation
  • added English translations for instructions and attention questions
  • compiled scattered readmes into one README.md
  • removed participant_answers sub-dir

TODO:

  • add OpenNeuro link to audio files when available
  • add info on how to change the downloaded stim file names so that the script can find them
  • localizer does not run in the first attempt from the bash script - requires running the python script separately first

Copy link
Member

@bthirion bthirion left a comment

Choose a reason for hiding this comment

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

Thx for opening. o particular commet
Please proceed and merge whenever you can.

@man-shu
Copy link
Collaborator Author

man-shu commented Aug 1, 2022

I just wanted to confirm if we are okay with uploading the audio files to this repo.

As of now, they are only available on the 3T stimulation PC.

@bthirion
Copy link
Member

bthirion commented Aug 1, 2022

You should ask Christophe whether this is legal.
Best,

@man-shu
Copy link
Collaborator Author

man-shu commented Aug 1, 2022

OK, will do. Thanks!

@ferponcem
Copy link
Collaborator

ferponcem commented Apr 21, 2023

Hey! Bringing this discussion back.
The event labels issue is fixed!
The discussion is whether upload the audio files to this repo or just point to Christophe's OpenNeuro site where all the files already are. @bthirion @man-shu


## How to run

* Download the audio files (`.wav` files with suffix `task-lppFR`) from OpenNeuro: https://openneuro.org/datasets/ds003643/versions/2.0.1
Copy link
Collaborator Author

@man-shu man-shu Apr 21, 2023

Choose a reason for hiding this comment

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

I had added the openneuro link in the README. We should probably add more info about where to save these stim files so that the script can find it.

Copy link
Collaborator

Choose a reason for hiding this comment

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

I had added the openneuro link in the README. We should probably add more info about where to save these stim files so that the script can find it.

Maybe in that case having the files directly here is better, so that we don't give much room for confusion?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

We definitely can put the files here, Christophe also said so. We were discussing simply pointing to openneuro, so as to avoid any legal issues and having to take them down from multiple places in the future.

Copy link
Member

Choose a reason for hiding this comment

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

I may have said the converse recently, But I think that it's OK to just have the link.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants