Skip to content

Remove visualization module, migrate to hedvis package#1199

Closed
Copilot wants to merge 1 commit intomainfrom
copilot/review-hed-python-pr-1198
Closed

Remove visualization module, migrate to hedvis package#1199
Copilot wants to merge 1 commit intomainfrom
copilot/review-hed-python-pr-1198

Conversation

Copy link
Contributor

Copilot AI commented Feb 3, 2026

The visualization code (word cloud generation) has been extracted into a separate hedvis package to reduce core dependencies and improve modularity.

Changes

  • Removed modules: hed/tools/visualization/ (284 lines)
  • Removed tests: tests/tools/visualization/ (202 lines) and test data
  • Dependencies removed: matplotlib>=3.9.0, wordcloud==1.9.5, implicit pillow
  • Dependencies updated: Added pandas version constraint <3.0.0
  • Documentation: Added migration note pointing to hedvis package

Migration

# Before (removed):
from hed.tools.visualization import create_wordcloud

# After:
# pip install hedvis
from hedvis import create_wordcloud

Notes

  • No internal dependencies on visualization module found
  • hedvis package published and available on PyPI
  • Breaking change: users importing hed.tools.visualization will need to install hedvis separately
Original prompt

GitHub is having an issue and I cannot request a Copilot review on #1198. Please review it and create a comment with your review on the PR.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI changed the title [WIP] Review changes in hed-python PR 1198 Remove visualization module, migrate to hedvis package Feb 3, 2026
Copilot AI requested a review from VisLab February 3, 2026 16:57
@VisLab VisLab closed this Feb 3, 2026
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.

2 participants