Skip to content

Conversation

@VisLab
Copy link
Member

@VisLab VisLab commented Jan 27, 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 pull request removes the remodeling tools from the HED Python package in response to issue #1180. The remodeling functionality has been moved to a separate package called table-remodeler.

Changes:

  • Removed all remodeling tool implementation files from hed/tools/remodeling/
  • Deleted all remodeling-related test files from tests/tools/remodeling/
  • Moved test data from tests/data/remodel_tests/ to tests/data/other_tests/ for files still needed by other tests
  • Updated test file paths to reference the new other_tests directory
  • Updated documentation to note that remodeling tools have been moved to a separate package
  • Cleaned up imports in hed/tools/__init__.py to remove remodeling references
  • Updated pyproject.toml to add [jupyter] extra to black dependency

Reviewed changes

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

Show a summary per file
File Description
tests/tools/util/test_data_util.py Updated test data path from remodel_tests to other_tests
tests/tools/bids/test_bids_tabular_file.py Updated test data paths to other_tests
tests/tools/analysis/*.py Updated test data paths (5 files)
tests/models/test_tabular_input.py Updated test data path
tests/tools/remodeling/**/*.py Deleted all remodeling test files (~30 files)
tests/data/remodel_tests/** Deleted remodeling test data and config files
tests/data/other_tests/** Added test data files moved from remodel_tests
hed/tools/remodeling/**/*.py Deleted all remodeling implementation files (~20 files)
hed/tools/init.py Removed all remodeling-related imports
pyproject.toml Changed black dependency to include jupyter extra
docs/.md, docs/.rst Added meta tags for SEO
README.md Added note about remodeling tools being moved to separate package

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

@VisLab VisLab merged commit 8005d83 into hed-standard:main Jan 27, 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