You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
TotalAcquiredPairs is compared against the number of rows in aslcontext, but the TotalAcquiredPairs value should be the number of control-label pairs, while the aslcontext file should have one row for each volume.
Describe what you expected.
I think the check should be that TotalAcquiredPairs is <= the number of rows in the aslcontext file, since at most the aslcontext file could have 2x the number of pairs, if all of the volumes in the file as control or label volumes, but at minimum there could just be a single pair with a bunch of m0scan, deltam, and/or cbf volumes filling out the rest of the asl file.
The number of values for 'TotalAcquiredPairs' for this file does not match number of
volumes in the associated 'aslcontext.tsv'.
'TotalAcquiredPairs' is the original number of 3D volumes acquired for each volume defined in the
associated 'aslcontext.tsv'.
level: warning
selectors:
- suffix == "asl"
- type(associations.aslcontext) != "null"
- '"TotalAcquiredPairs" in sidecar'
checks:
- aslcontext.n_rows == sidecar.TotalAcquiredPairs
The text was updated successfully, but these errors were encountered:
tsalo
added
bug
Something isn't working
schema
Issues related to the YAML schema representation of the specification. Patch version release.
labels
Nov 3, 2024
Describe your problem in detail.
TotalAcquiredPairs is compared against the number of rows in aslcontext, but the TotalAcquiredPairs value should be the number of control-label pairs, while the aslcontext file should have one row for each volume.
Describe what you expected.
I think the check should be that TotalAcquiredPairs is <= the number of rows in the aslcontext file, since at most the aslcontext file could have 2x the number of pairs, if all of the volumes in the file as control or label volumes, but at minimum there could just be a single pair with a bunch of m0scan, deltam, and/or cbf volumes filling out the rest of the asl file.
BIDS specification section
bids-specification/src/schema/rules/checks/asl.yaml
Lines 199 to 213 in c2685cc
The text was updated successfully, but these errors were encountered: