Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
[ENH] Define channel column for events and Delimiter field for column descriptions #1483
[ENH] Define channel column for events and Delimiter field for column descriptions #1483
Changes from 3 commits
a3e3d25
ab0e6a9
df07a15
8a88eaf
e64314b
d1056c1
2a12171
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This might be better as:
"If a column entry may be interpreted as..."
Also, does the Delimiter only apply to "channel" column or to any column as the above implies.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Delimiter will from now on apply to any column.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't completely understand why this is in the task.yaml file. Is task.yaml where it determines if tabular data applies to a particular modality?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
task.yaml
is an arbitrary name. It could have been calledevents.yaml
, but the ruleschema.rules.tabular_data.task.TaskEvents
defines what are valid columns in an_events.tsv
file. The filename construction (and hence what modalities apply) are defined in the filename rules:bids-specification/src/schema/rules/files/raw/task.yaml
Lines 2 to 19 in 426de56
bids-specification/src/schema/rules/files/raw/task.yaml
Lines 2 to 19 in 426de56