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

Integration of the existing FSL parser + update of the action for calculating the nidm examples on the 2 parsers #85

Closed thomasbtnfr closed 1 year ago

thomasbtnfr commented 1 year ago

This pull request adds the existing FSL parser and updates the Github action which now computes nidm examples in bids-prov format from the 2 parsers (SPM + FSL).

Question:

For the next steps, I think I will be able to make the same modifications as those made to SPM (uuid, mapping for example) if the current outputs are already correct and I can work on boutiques.

cmaumet commented 1 year ago

@thomasbtnfr: To answer your question above: you are right the FSL examples should use the same context as SPM (but if I am correct you have already implemented this).

thomasbtnfr commented 1 year ago

The changes have been made. For the context, in this PR, it's ok for the results of the github action but not for the script in general. This is fixed in the following PR #87