Skip to content

Conversation

@VisLab
Copy link
Member

@VisLab VisLab commented Jan 6, 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 command-line script for extracting tabular summaries from TSV files without requiring BIDS dataset organization. It enhances the existing TabularSummary class with overflow column tracking functionality to handle cases where categorical columns exceed specified limits.

Key changes include:

  • New extract_tabular_summary.py script with comprehensive argument parsing, logging, and output formatting capabilities
  • Enhanced TabularSummary class to track columns that exceed the categorical limit via the new overflow_columns attribute
  • Updated categorical value tracking logic to continue counting existing values even after reaching the limit
  • Documentation dependency version updates to newer versions of Sphinx, Furo, myst-parser, and linkify-it-py

Reviewed changes

Copilot reviewed 7 out of 8 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
hed/scripts/extract_tabular_summary.py New command-line script for extracting tabular summaries with support for filtering, categorical limits, and multiple output formats
tests/scripts/test_extract_tabular_summary.py Comprehensive test suite covering all script functionality including edge cases and error handling
hed/tools/analysis/tabular_summary.py Adds overflow_columns tracking and refactors categorical update logic to preserve existing value counts after limit is reached
tests/tools/analysis/test_tabular_summary.py Extensive new tests for overflow column functionality and updated assertions for categorical limit behavior
pyproject.toml Updates documentation dependency versions (Sphinx, Furo, linkify-it-py)
requirements-dev.txt Updates documentation dependency versions to match pyproject.toml
docs/requirements.txt Updates documentation dependency versions to match pyproject.toml
hed/tools/util/io_util.py Minor formatting improvement to parameter documentation alignment

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

@VisLab VisLab merged commit b89d44c into hed-standard:main Jan 6, 2026
21 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