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

motion_systemvalidation: update to latest version #404

Merged
merged 3 commits into from
Aug 25, 2023

Conversation

JuliusWelzel
Copy link
Contributor

This updated dataset passes the validator

Comment on lines +2 to +5
motion/sub-pp002_task-backwards_tracksys-imu_motion.tsv 2023-05-05T17:39:47.307Z
motion/sub-pp002_task-backwards_tracksys-omc_motion.tsv 2023-05-05T17:39:47.307Z
motion/sub-pp002_task-obstacleHigh_tracksys-imu_motion.tsv 2023-05-05T17:39:51.873Z
motion/sub-pp002_task-obstacleHigh_tracksys-omc_motion.tsv 2023-05-05T17:39:51.873Z
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this potentially PII? BIDS generally recommends date anonymization. See Units:

Dates can be shifted by a random number of days for privacy protection reasons. To distinguish real dates from shifted dates, is is RECOMMENDED to set shifted dates to the year 1925 or earlier. Note that some data formats do not support arbitrary recording dates. For example, the EDF data format can only contain recording dates after 1985. For longitudinal studies dates MUST be shifted by the same number of days within each subject to maintain the interval information. For example: 1867-06-15T13:45:30

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I used the date when I converted the dataset to BIDS. That date is not related to anything.

@effigies effigies merged commit e857597 into bids-standard:master Aug 25, 2023
6 checks passed
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.

2 participants