Stefan Appelhoff

Results 90 issues of Stefan Appelhoff

Most software packages of similar size and purpose as bids-validator have a comprehensive documentation. Meanwhile, we have a long README and a CONTRIBUTING file (which is not very visible). What...

help wanted

as described in https://github.com/bids-standard/bids-examples/pull/291#issuecomment-929407747 > The development build does require Node 14 now, but the user requirement is Node 12. The Node 12 end of life is pretty soon, so...

effort: low
impact: low

we are currently in the final review round of a PR in bids-specification that will RECOMMEND CMIXF formatting of SI units: https://github.com/bids-standard/bids-specification/pull/411 Once that PR is merged, we should "switch...

copying to this public issue from a private chat: > wouldn't it be fairly easy to supply the web-validator as both `stable` (as done currently), AND `dev`? We can just...

enhancement
effort: low
impact: high

The validator is now raising an error when empty files are encountered. For the CTF system (MEG), it is customary to have a bunch of files in a directory ending...

help wanted
MEG

crossref: - https://github.com/bids-standard/bids-validator/pull/1190#issuecomment-783520897 - https://github.com/bids-standard/bids-examples/issues/244 According to the spec, electrodes and coordsystem files should accept: - sub - ses - acq - space see https://bids-specification.readthedocs.io/en/stable/04-modality-specific-files/03-electroencephalography.html#electrodes-description-_electrodestsv But they accept many more....

Came up in https://github.com/bids-standard/bids-examples/issues/229#issuecomment-774096886 related: https://github.com/bids-standard/bids-validator/issues/1129 Trying to run `bids-validator eeg_matchingpennies/ -c bidsconfig.json` from the root of a cloned `bids-examples` repository results in an error. Same issue if I specify...

effort: medium
impact: high

maybe related #1165 According to these lines: https://github.com/bids-standard/bids-validator/blob/970921cbf2e3e85df195ca16f63ee460c588e596/bids-validator/utils/options.js#L65-L72 the following bids-validator call should raise issue 44 (again, from bids-examples repo root): `bids-validator eeg_matchingpennies --config $PWD/iDontExist.json` But no error is raised.

bug
effort: medium
impact: high

The CLI is currently not exhaustively tested, or at least I couldn't find it. (If it **is** tested, can somebody please point me to the tests?) As far as I...

effort: low
impact: medium

following quote by @arnodelorme: > This BIDS dataset contains both .edf and .bdf file (which are very small): https://openneuro.org/datasets/ds002034/versions/1.0.1 > sub-01/ses-01/eeg/sub-01_ses-01_task-offline_run-01_eeg.edf > sub-01/ses-01/eeg/sub-01_ses-01_task-offline_run-01_eeg.bdf > I believe it should not have...

enhancement
effort: high
impact: high