-
Notifications
You must be signed in to change notification settings - Fork 169
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
[SCHEMA] Allow physio files for anat datatype #1961
[SCHEMA] Allow physio files for anat datatype #1961
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #1961 +/- ##
=======================================
Coverage 86.87% 86.87%
=======================================
Files 16 16
Lines 1410 1410
=======================================
Hits 1225 1225
Misses 185 185 ☔ View full report in Codecov by Sentry. |
Co-authored-by: Chris Markiewicz <[email protected]>
Co-authored-by: Chris Markiewicz <[email protected]>
Co-authored-by: Chris Markiewicz <[email protected]>
Implements a change agreed upon in #513. |
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.
Good with me.
…ttype * commit 'v1.10.0-35-g5f7004819': (218 commits) Include entity-less "scans.json" into an example of inheritance principle (bids-standard#1945) fix(checks): Enforce timing mutual exclusions on BOLD/ASL data only (bids-standard#1969) refactor contributing (bids-standard#1965) [pre-commit.ci] pre-commit autoupdate (bids-standard#1967) [SCHEMA] Allow physio files for anat datatype (bids-standard#1961) [pre-commit.ci] pre-commit autoupdate Add an empty line in hope to get table rendered properly in "Ordering rules" section (bids-standard#1953) schema: add check for duplicate READMEs (bids-standard#1952) [MAINT] switch bidsschematools to pyproject.toml (bids-standard#1948) fix(schema): Disable TaskName check for channels and markers files Permit and warn on task/acquisition/run for electrodes and coordsystems [FIX] Allow (but discourage) task entity for coordsystem.json fix(schema): Limit MRI metadata checks to NIfTIs fix: Only check for sorted times in arrays py3.13 (bids-standard#1947) [pre-commit.ci] pre-commit autoupdate (bids-standard#1946) [FIX] Update changelog links to avoid redirects (bids-standard#1944) [ENH] Update DWI suffixes to include most common scanner derivatives (bids-standard#1864) [pre-commit.ci] pre-commit autoupdate [MAINT] Update Release_Protocol.md ...
* add anat to datatypes field for timeseries no task in schema/rules/files/raw/task * Update src/schema/rules/files/raw/task.yaml Co-authored-by: Chris Markiewicz <[email protected]> * Update src/schema/rules/files/raw/task.yaml Co-authored-by: Chris Markiewicz <[email protected]> * Update src/schema/rules/files/raw/task.yaml Co-authored-by: Chris Markiewicz <[email protected]> * Update src/schema/rules/files/raw/task.yaml --------- Co-authored-by: Chris Markiewicz <[email protected]>
* origin/master: (26 commits) Update tools/schemacode/src/bidsschematools/types/namespace.py fail build on macro fails graceful error when failing to load invalid yml fix Update mkdocs.yml fix more seg Update src/modality-specific-files/electroencephalography.md fix channel, elec, sim recording fix key value and in-out fix tabular and seg reset fix internal links enh: Link all DICOM tags following standard pattern chore(links): Fix http -> https 301 redirects Update Release_Protocol.md (#1997) schema-1.0.1-dev schema-1.0.0 [SCHEMA] Allow physio files for anat datatype (#1961) FIX: Add 1.10.0 to known versions list (#1994) ...
Issue outlined in bids-standard/bids-validator#2164
This would fail to cover echo, part, and chunk entities which are legal for anatomical images and so too would be legal in the physio filenames.