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

Unclear which file in "You should define 'HEDVersion' for this file." #2122

Open
yarikoptic opened this issue Sep 4, 2024 · 2 comments
Open
Labels

Comments

@yarikoptic
Copy link
Contributor

> datalad containers-run -n bids-validator $PWD

        8: [WARN] You should define 'HEDVersion' for this file. If you don't provide this information, the HED validation will use the latest version available. (code: 109 - HED_VERSION_NOT_DEFINED)

        Please visit https://neurostars.org/search?q=HED_VERSION_NOT_DEFINED for existing conversations about this issue.

whenever we have

(deno) yoh@typhon:/mnt/DATA/data/yoh/1076_spacetop$ grep HEDVersion dataset_description.json
  "HEDVersion": "8.2.0",

(deno) yoh@typhon:/mnt/DATA/data/yoh/1076_spacetop$ grep bids-vali .datalad/config 
[datalad "containers.bids-validator"]
        image = code/containers/images/bids/bids-validator--1.14.11.sing

so we have HEDVersion in dataset_description.json but there is still that Warning and not clear which file it is actually talking about

@VisLab
Copy link
Member

VisLab commented Sep 4, 2024

@rwblair @nellh This is the deno version for the validator. I believe we are relying on BIDS to pass us this information now since we are doing by-file validation rather than taking in the entire dataset. Can you please check and advise?

P.S. HED no longer uses the latest version available, but requires an actual version. The message:

the HED validation will use the latest version available. (code: 109 - HED_VERSION_NOT_DEFINED)

does not appear in the hed-validator.

@yarikoptic
Copy link
Contributor Author

@rwblair @nellh This is the deno version for the validator.

FWIW, I think this is still an original node version, which we run out of distributed container. Yet to get to try deno version

@effigies effigies added the legacy label Sep 6, 2024
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

3 participants