bids-standard / bids-bep016

BEP016: diffusion derivatives
Creative Commons Attribution 4.0 International
6 stars 7 forks source link

Move the `desc-` entity to the end of the file name. #107

Open arokem opened 3 months ago

arokem commented 3 months ago

See discussion in https://github.com/bids-standard/bids-specification/issues/1895

arokem commented 3 months ago

Also, @tsalo proposes that we change desc-mean to desc-stat, which would gel more nicely with the functional derivatives BEP and @tsalo's own GLM derivatives BEP. If folks think that's agreeable, I can implement that here as well.

tsalo commented 3 months ago

Sorry I miscommunicated my idea. I mean stat-mean instead of desc-mean, as in BEP 012.

arokem commented 3 months ago

Of course - thanks for setting me straight. I am actually not sure that it entirely makes sense for the use-case shown here, because we have desc-mean in contrast to desc-merged and while stat-mean makes sense, I am not sure that stat-merged does.

PeerHerholz commented 3 months ago

Was the idea to only have stat- or desc- or could both be used, ie stat- for mean and desc- for merged given that the underlying data is obtained through different operations, one being a statistic while the other isn't?

Sorry if I missed something obvious.