Chris Markiewicz

Sorry, this got lost in the summer conference season. Are you still seeing this with the latest fmriprep?

The legacy validator errored on the absence of subject directories, so this was written for consistency with that. I don't see the use-case for subject-less datasets. Calling a directory that...

@arokem @barbarastrasser I see you gave thumbs ups on the initial post. Would either of you care to make a formal review?

For the record, I agree with the argument that this error is not reflected in the text, so we should either update the text or relax the warning. OpenNeuro will...

Not to speak for dcm2niix developers, but I am skeptical that a request to replace two JSON keys encoding `(name, version)` with a provenance sidecar will be well received. The...

@satra Could you elaborate how, for example, https://github.com/OpenNeuroDatasets/ds000224/blob/master/task-rest_bold.json would be changed to use the provenance keys?

Yes, I'd like to see what the inline prov option is. I couldn't guess it from the bep text.

So Yarik's example would become: ```yaml "GeneratedBy": { "Id": "urn:conversion-", "Type": "Activity", "Label": "Conversion", "Used": "bids:sourcedata:somefile.dcm", "AssociatedWith": { "Id": "urn:dcm2niix-", "Type": "Software", "Label": "dcm2niix", "Version": "v1.0.20170411 GCC4.4.7" } } ```...

Documentation here: https://bidsschematools--2133.org.readthedocs.build/en/2133/api/bidsschematools.types.context.html 

@ubdbra001 Just a heads up on this. This is going to replace much of https://github.com/bids-standard/python-validator/pull/24, which will instead do something like what I have above.