Skip to content

Conversation

@VisLab
Copy link
Member

@VisLab VisLab commented Jan 21, 2026

No description provided.

Copy link
Contributor

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 adds a new validation script for HED annotations in tabular files (TSV). The script validates tabular data against HED schemas, optionally using sidecar files for additional context, and includes CLI integration and comprehensive tests.

Changes:

  • Added validate_hed_tabular script with support for validating TSV files with HED annotations
  • Integrated the new command into the CLI as hedpy validate tabular
  • Added comprehensive test coverage for both the script and CLI parameter parity

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
hed/scripts/validate_hed_tabular.py New validation script for tabular files with argument parser, main validation logic, and error handling following existing patterns
hed/cli/cli.py CLI command definition for validate tabular with proper option groups and parameter mapping
tests/scripts/test_validate_hed_tabular.py Comprehensive test suite covering valid/invalid files, sidecar integration, error limiting, JSON output, and error handling
tests/test_cli_parameter_parity.py Parity test ensuring CLI parameters match the underlying script parser
pyproject.toml Script entry point registration for validate_hed_tabular command

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

@VisLab VisLab merged commit 2e3183e into hed-standard:main Jan 21, 2026
27 checks passed
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.

1 participant