bids-standard / BEP028_BIDSprov

Organizing and coordinating BIDS extension proposal 28 : BIDS Provenance
https://bids.neuroimaging.io/bep028
Creative Commons Attribution 4.0 International
4 stars 12 forks source link

jsonld at any level #144

Open cmaumet opened 5 days ago

cmaumet commented 5 days ago

Update proposal for BIDS Prov (BEP028)

Proposal by @yarikoptic

cmaumet commented 2 days ago

@yarikoptic -- the rationale behind having all the _prov.jsonld files in a single folder was to make it easy to delete all of them at once if needed.

But I see your point and I can't see why we should not make this possible. @satra: just let us know if you think otherwise.

@bclenet can you help us and update the gdoc accordingly and let us know of any issues you identify? Thanks!

kabilar commented 16 hours ago

Thanks all. I agree with the above proposal.

Clarifying question. Since the inheritance principle would not be in effect here, ideally would every data file need a sidecar *.jsonld file?

yarikoptic commented 11 hours ago

That is what BEP028 indeed suggests that as one of the ways. I have concern though with potentially nearly doubling number of files in the dataset that way, which I think should be avoided unless there is good rationale and indeed the PROV record could not be just "inherited". That is why IMHO there should be the another way at .json file level: