Alejandro de la Vega

Results 129 comments of Alejandro de la Vega

That's great. Question: is ancp bids indexing meta-data by default? I think it would be useful to try pybids in those unit tests with `index_metadata=False` to evaluate the impact of...

Another idea is to try to run the main unit tests in pybids w/ ancp bids. Obviously many will fail simply because the APIs are not identical, but it would...

@erdalkaraca I'm trying to reconcile your performance numbers w a previous investigation into performance: see: - https://github.com/bids-standard/pybids/issues/609 - https://github.com/bids-standard/pybids/issues/521 Basically, we concluded that most time was actually spent in `os.walk`...

@erdalkaraca what's the best way to contact you? we're having a bids event soon that i'd like to invite you to participate in to discuss this further. if you want,...

Progress update, it looks like this has less to do with `ignore`, and more with the way my entities are being added to the variables I include in the other...

If this behavior (which did change from 0.8), doesn't seem like I bug, I'm happy to close. I can easily fix this by including a meta-data file (with RT) in...

Actually, it looks like I'm already including that meta-data in the bundle, so there may still be a bug here.... Let me look.

Conclusion is that `task-movieDM_bold.json` would not match `sub-NDARAC904DMU_task-movieDM_events.tsv`. Instead, we need: `task-movieDM_events.tsv` for proper inheritance. And I need to change the code in the celery worker to make bundles that...

Spoke too soon. Looks like even with renaming that `.json` file I still have problems. Looks like its because even though the `_events` file now gets the appropriate entities, the...

Now that I'm looking at the code, it looks like `RepetitionTime` could never be an entity that is associaed with a variable, as it limits it to `ALL_VARIABLES` which is...