-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Empty FDR p-values for clusters #48
Comments
@cmaumet I forget what our stance was on missing values... was "[]"
preferred or just omitted the value from the serialisation. I have a
memory that we discussed it but I don't remember the outcome.
|
@nicholst: when we have missing values, we usually omit the attribute in the serialisation because it makes it easier to identify the missingness using queries. This also avoids having to decide on a the value which represents missing data ([], NaN, etc.). But in the case of |
@cmaumet: So should we move away from using
https://neurovault.org/collections/2210/ for testing or fix it?
|
@nicholst: I'd say let's 1/ check if this is a bug in the exporter & fix it 2/ release a new version of the SPM NIDM exporter 3/ recompute the examples in the a new NeuroVault collection. How does that sound? |
In the NIDM-Results pack
ex_spm_thr_voxelfdrp05
generated by the exporter (given in this neurovault collection), for eachnidm_SupraThresholdCluster
object, annidm_qValueFDR
field has been included but is left as empty (See below for examples taken from the jsonld and ttl serializations respectively). This is the only dataset in this collection for which this is happening.It does not appear as though the
nidm_HeightThreshold
ornidm_ExtentThreshold
objects in this pack have any similar issues.The text was updated successfully, but these errors were encountered: