-
Notifications
You must be signed in to change notification settings - Fork 171
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
[BUG] Should subsection "Task events" be under "Modality specific files" as opposed to "Modality agnostic files"? #1868
Comments
It is already in Modality specific files: May be you meant "the other way"? But it seems that the section names (agnostic vs specific) is merely to separate which files go above But overall I feel that in principle we could
So it could be events not associated with any task! It could be annotation of some events happening during anatomical MRI acquisition (when subject blinked, or wiped nose, or ...).
Should we change the title? ;-) |
Yes, that's what I meant. Thanks for making it clearer :)
I don't know the specs with sufficient depth to be sure that we can just remove "Task" from "Task Events". If that is the case I'm +1 on this idea -- it would simplify things so much (e.g., #1751 and #1692 [seems like I'm partially repeating myself, I had forgotten of these when I posted])
Sure -- and also close if you think #1692 and/or #1751 are already covering this. Thanks for looking into this |
Ha, @VisLab thought alike me on removing "Task" in #1751 (comment) . Indeed I feel like this issue is largely a duplicate of the so let's try to remember it exists and close this one for now. |
Duplicate of #1751. |
Describe your problem in detail.
Tasks files do not seem modality-attached, and it is my experience that finding them under modality-specific is confusing for newcomers, especially those with more training in multimodal behavioral experiments.
Describe what you expected.
I believe tasks description and metadata should better be under modality agnostic.
Stimulus are likely to fall under the same problem.
BIDS specification section
https://bids-specification.readthedocs.io/en/stable/modality-specific-files/task-events.html
The text was updated successfully, but these errors were encountered: