studyforrest/data-phase2
2
0
Fork 0
Code Issues 2 Pull requests Packages Activity

BIDS compliance of eyetracking data #19

Merged
adswa merged 3 commits from eyetracking into master 2021-04-30 11:09:29 +00:00
Conversation 6 Commits 3 Files changed 2 +8 -1
adswa commented 2021-04-30 09:25:59 +00:00 (Migrated from github.com)
Copy link

Fixes https://github.com/psychoinformatics-de/studyforrest-data/issues/12

Fixes https://github.com/psychoinformatics-de/studyforrest-data/issues/12
loj (Migrated from github.com) reviewed 2021-04-30 09:50:18 +00:00
loj (Migrated from github.com) left a comment
Copy link

Just a small change to the .bidsignore file. Otherwise things look good. Thanks!

Just a small change to the `.bidsignore` file. Otherwise things look good. Thanks!
.bidsignore
loj (Migrated from github.com) commented 2021-04-30 09:49:40 +00:00
Copy link

Almost... it needs to be

**/*.asc.gz

Almost... it needs to be `**/*.asc.gz`
loj (Migrated from github.com) reviewed 2021-04-30 09:57:54 +00:00
.bidsignore
loj (Migrated from github.com) commented 2021-04-30 09:57:53 +00:00
Copy link

Ah, and one other thing that could be added to the .bidsignore file:

task-*_physio.json

See https://github.com/bids-standard/bids-validator/issues/1275#issuecomment-827644778

Ah, and one other thing that could be added to the `.bidsignore` file: `task-*_physio.json` See https://github.com/bids-standard/bids-validator/issues/1275#issuecomment-827644778
adswa (Migrated from github.com) reviewed 2021-04-30 10:59:19 +00:00
.bidsignore
adswa (Migrated from github.com) commented 2021-04-30 10:59:19 +00:00
Copy link

ahhhh crap, thanks for catching that

ahhhh crap, thanks for catching that
adswa (Migrated from github.com) reviewed 2021-04-30 10:59:31 +00:00
.bidsignore
adswa (Migrated from github.com) commented 2021-04-30 10:59:31 +00:00
Copy link
**/*.asc.gz
```suggestion **/*.asc.gz ```
adswa commented 2021-04-30 11:07:14 +00:00 (Migrated from github.com)
Copy link

Should be good now, I applied both of your suggestions.

Should be good now, I applied both of your suggestions.
loj commented 2021-04-30 11:09:12 +00:00 (Migrated from github.com)
Copy link

Looks good. Thanks!

Looks good. Thanks!
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
bug

   
duplicate

   
enhancement

   
help wanted

   
invalid

   
question

   
wontfix
No labels
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
studyforrest/data-phase2!19
No description provided.