Chris Markiewicz
Chris Markiewicz
Ah, that's my bad. It should be just `data.load()`. Incidentally, what are you using this function for? It might make more sense just to stop using the function.
Would this be sufficient? ```python schema = bst.schema.load_schema() return schema.bids_version ```
So what happened here is https://github.com/bids-standard/bids-specification/pull/2115, where we started compiling the `schema.json` file during the package build, so we now include that instead of the `schema/` directory. Loading JSON is...
> Probably the simplest way to get the versions is now: > > ``` > import bidsschematools as bst > schema_version = bst.__version__ > bids_version = bst.__bids_version__ > ``` Actually,...
I don't think I understand that last comment. What's ruamel.yaml trying to do?
> here, @effigies states that electrodes.tsv is actually optional for iEEG: > > * [electrodes.tsv for simultaneous MEG/EEG data bids-specification#1550 (comment)](https://github.com/bids-standard/bids-specification/issues/1550#issuecomment-1666051107) > > > though I don't know where that...
> The problem is that, in contrast with EEG data where channel positions have standardized names, iEEG data often do not make any sense if electrode positions are not shared...
I can't access that. Sent a request, if you have admin. In any event, if it should have been required, we should open a PR to clearly state that it...
FWIW, here are the datasets with iEEG directories with no `electrodes.tsv`: * [ds002844](https://openneuro.org/datasets/ds002844) * [ds003029](https://openneuro.org/datasets/ds003029) * [ds003078](https://openneuro.org/datasets/ds003078) * [ds003498](https://openneuro.org/datasets/ds003498) * [ds003754](https://openneuro.org/datasets/ds003754) * [ds004127](https://openneuro.org/datasets/ds004127) I agree that people using non-BIDS files...
I think making the file required is fine. The values in tabular files have always been allowed to be set to `n/a` to indicate missing data. We do not need...