Skip to content

Conversation

@yarikoptic
Copy link
Member

No description provided.

@yarikoptic yarikoptic requested review from Copilot and kabilar and removed request for kabilar November 12, 2025 14:58
Copilot finished reviewing on behalf of yarikoptic November 12, 2025 15:00
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR enhances the BIDS validation documentation by adding important guidance about excluding DANDI-specific files from BIDS validation and expanding validation tool recommendations.

Key changes:

  • Adds instruction to use .bidsignore file for dandiset.yaml to prevent BIDS validation errors
  • Expands validation recommendations to include both BIDS validator and dandi validate command
  • Notes that dandi upload also performs validation

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +51 to +53
- perform validation of the files ideally using both
- the stock [BIDS validator](https://bids.neuroimaging.io/tools/validator.html)
- and [dandi validate](https://dandi.readthedocs.io/en/latest/cmdline/validate.html) since it would also validate using other helper tools such as [NWB inspector](https://nwb.org/tools/core/nwbinspector/) if you have NWBs, to ensure that your data is compliant to all used standards.
Copy link
Member

@kabilar kabilar Nov 12, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- perform validation of the files ideally using both
- the stock [BIDS validator](https://bids.neuroimaging.io/tools/validator.html)
- and [dandi validate](https://dandi.readthedocs.io/en/latest/cmdline/validate.html) since it would also validate using other helper tools such as [NWB inspector](https://nwb.org/tools/core/nwbinspector/) if you have NWBs, to ensure that your data is compliant to all used standards.
- perform validation of the files using [dandi validate](https://dandi.readthedocs.io/en/latest/cmdline/validate.html) to ensure your data is compliant with the BIDS and/or NWB standards. Alternatively, you could independently validate your files using the [BIDS Validator](https://bids.neuroimaging.io/tools/validator.html) and/or [NWB Inspector](https://nwb.org/tools/core/nwbinspector/).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants