bids-standard / bids-examples

A set of BIDS compatible datasets with empty raw data files that can be used for writing lightweight software tests.
http://bids-standard.github.io/bids-examples/
169 stars 134 forks source link

[ENH] BEP022 - Magnetic Resonance Spectroscopy #425

Closed markmikkelsen closed 1 month ago

markmikkelsen commented 8 months ago

This pull request adds MRS BIDS-compliant datasets to the bids-examples repository. It includes three datasets: mrs_2dmrsi/, mrs_biggaba/, and mrs_fmrs/.

Remi-Gau commented 7 months ago

@markmikkelsen do you think you could add a README.md to each dataset?

one of them has an excellent README but the 2 others have none

There is a README template in the bids starter kit if that can help: https://github.com/bids-standard/bids-starter-kit/blob/main/templates/README.MD?plain=1

Remi-Gau commented 7 months ago

@effigies do you think we should / could use this PR to use citation.cff files?

I think we do not have any example that make use of them.

effigies commented 7 months ago

Not unless Mark already has CITATION.cff files he wants to include. We don't need to pile on tasks to this BEP. Let's make a separate PR that adds a citation file to an existing example.

Remi-Gau commented 7 months ago

oh yeah of course!! that's fair!

will open an issue to see if we can actually do it in another dataset that already exist.

Remi-Gau commented 7 months ago

🤦🏾 👴🏾

markmikkelsen commented 7 months ago

@markmikkelsen do you think you could add a README.md to each dataset?

one of them has an excellent README but the 2 others have none

There is a README template in the bids starter kit if that can help: https://github.com/bids-standard/bids-starter-kit/blob/main/templates/README.MD?plain=1

Yep! I've just added these files.

markmikkelsen commented 7 months ago

This pull request is nearly ready for merging. The check failures relate to the MRS filename suffices not being included in the bids-validator.

effigies commented 2 months ago

There are no examples of nuc- or voi- entities. These would be helpful for validating entity ordering.

markmikkelsen commented 2 months ago

There are no examples of nuc- or voi- entities. These would be helpful for validating entity ordering.

I'm ignorant of how the validator works, but does it rely on examples? My impression was that it was contingent on the current stable specification. So, how do example datasets factor into validation?

Remi-Gau commented 2 months ago

There are no examples of nuc- or voi- entities. These would be helpful for validating entity ordering.

I'm ignorant of how the validator works, but does it rely on examples? My impression was that it was contingent on the current stable specification. So, how do example datasets factor into validation?

in short and in a very hand wavy way:

the validator tries to implement the specification so we need examples to ensure that the validator work as expected (they act as some sort of ground truth)

so the richer the examples, the more we can be sure that the validator has fewer blind spots

markmikkelsen commented 2 months ago

Gotcha. Since the datasets are not real. I can just create a bunch of example of fake datasets to satisfy this.

effigies commented 1 month ago

@markmikkelsen @wtclarke Note that BIDS doesn't permit null values, but instead leaves fields blank. Fixed in bc92a4d. Not sure if you need to update converters.