Chris Markiewicz

Results 2126 comments of 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...