Skip to content
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

Different validator outputs between webapp version and command line version #2186

Open
tientong98 opened this issue Nov 1, 2024 · 4 comments
Labels

Comments

@tientong98
Copy link

tientong98 commented Nov 1, 2024

For the command line version, I'm using
node v22.6.0
npm 10.8.2
bids-validator 1.15.0

And bids-validator doesn't return any error or warning, while the webapp does -- on the same data.

I'm using data from https://github.com/PennLINC/CuBIDS/raw/main/cubids/tests/data/BIDS_Dataset.zip, but just included the dataset_description.json, README, and sub-02 folder.

In addition, when running the webapp version, I got this error:

The number of volumes in this scan does not match the number of volumes in the associated '.bvec' and '.bval' files.
/sub-02/ses-phdiff/dwi/sub-02_ses-phdiff_acq-HASC55AP_dwi.nii.gz

However, NVols = 61, bval has 61 values, and bvec is an array with the shape of 3x61, so this shouldn't be an error.

@effigies
Copy link
Collaborator

effigies commented Nov 1, 2024

I can look at this in a bit, but figure I'll ping @tsalo due to CuBIDS, as he might know something I don't.

@effigies
Copy link
Collaborator

effigies commented Nov 1, 2024

@tientong98
Copy link
Author

@effigies effigies added the schema label Nov 1, 2024
@effigies
Copy link
Collaborator

effigies commented Nov 1, 2024

Okay, so the difference is that you're using the schema validator webapp and the legacy validator CLI.

We'll look at the data to determine if there's a bug in the schema validator or a problem with the check in the legacy validator.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants