Tal Yarkoni

Results 133 comments of Tal Yarkoni

The issue of column names conflicting is a separate one, IMO... In principle users could also put columns named `subject`, `run`, or `session` in their tsv files, and I don't...

> I think that ideally PyBIDS should be as close to BIDS specification as possible in terms of the metadata fields it extracts/associates with the files. It should not add...

> hold on -- I am not trying to get rid of it entirely! Just from the files where there is no `suffix` anyhow associated with, e.g. `participants.tsv` Dude I...

II didn't say it was in 0.8, just that I'm okay making that change. I tried it out locally and it broke some things I'll need to fix, and I...

I agree that better documentation would be a good thing here, though we might also want to consider removing `config_filename` (see below). To elaborate on the difference: * `config` defines...

@cboulay, sorry, I'm only now coming back to this. To answer your question, inheritance is definitely not the intended model here; we don't want to encourage people to modify the...

Any status update on this, post resampling changes? (Fine if not, this is a low priority from my perspective.)

I don't know about handling lists of dicts; seems a bit clunky. But maybe we can allow incremental setup, so that you could call `setup()` multiple times. Will look into...

I took a quick look at this, and while it's in principle pretty straightforward to implement incremental setup, it would become quite inefficient given the current implementation. The problem is...

The docs are out of date; see [this tutorial](https://github.com/bids-standard/pybids/blob/master/examples/pybids_tutorial.ipynb) for the current usage pattern. You need to drop the `.filename` on the end, and then you can do `f.get_metadata()`. Leaving...