Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rename exclude_session field in *_task-things_desc-perTrial_annotation.tsv #42

Open
emdupre opened this issue Nov 8, 2024 · 0 comments

Comments

@emdupre
Copy link

emdupre commented Nov 8, 2024

Thanks for all of your work in putting this dataset together !

One small stumbling block that has come up in my initial iterations with @MarieStLaurent : currently, there's a field in the perTrial_annotation.tsv files that is labelled exclude_session. This is appropriately described in the accompanying JSON file as only referring to memory analyses (that is, users should filter on that flag for memory analyses) rather than for analyses more generally (e.g., in perception, where the trials should be kept).

As I'm working with the trial-averaged beta maps (available in model-fitHrfGLMdenoiseRR_stats-imageBetas_desc-zscore_statseries.h5), filtering on exclude_session incorrectly suggests that there are discrepancies between the tsv and h5 files.

Would it make sense to rename the field to refer to memory directly ? I'd suggest something like incorrect_order.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant