Taylor Salo

Results 525 comments of Taylor Salo

We should also perform an audit of our pull requests/commit history in order to identify likely sources of differences. I am going to create a label that we can use...

Once the list is finalized, we should absolutely incorporate it into our documentation somewhere.

Kinda forgot that this was part of #720... 🤦 Well at least having an issue about it will help with tracking things.

@notZaki would it be possible for you to add the nifti files' sidecar files to the OSF repository? They would need to be manually produced, I think, using the original...

Yes, I think so. You can add [`Sources` and `RawSources`](https://bids-specification.readthedocs.io/en/stable/05-derivatives/02-common-data-types.html#common-file-level-metadata-fields) too, if you want. I believe the `RawSources` could be a URI pointing to the OpenNeuro dataset, which would be...

Also, it looks like the downsampled files' headers were overwritten, as the headers now indicate a TR of 1.

Beyond the niftis, confounds files, and jsons, are there any other files users would realistically need for this dataset? Should we include T1ws or transforms? EDIT: Or maybe... would we...

@notZaki two questions: 1. Have you had a chance to fix the downsampled images' headers and add the sidecar files? 2. Would you be willing to write up how you...

>I guess one thing I'm not sure is whether to have the downsampled images + sidecars be on their own with a flattened directory structure (as they are currently) or...

[This script](https://github.com/NBCLab/power-replication/blob/506e9937dcbc787f1f462cccc4677fb975e5b2a6/processing/collect_fmriprep.py) by @JulioAPeraza works for 20.2.1 (at least with the settings we've been using).