Reduce WAZP original code to metadata functionality#1
Merged
Conversation
This commit modernizes the beekeeper application by removing legacy WAZP functionality and focusing specifically on video metadata management. Key changes: - Fix obsolete Dash API (app.run_server -> app.run) - Remove unused DLC/pose estimation code (~200 lines) - Update deprecated pandas method (applymap -> map) - Remove legacy dependencies (tables, blosc2, h5py references) - Add proper error handling for missing configurations - Update test fixtures to use tmp_path instead of external data - Modernize documentation with current functionality - Add comprehensive user and developer workflow guides The application now provides a clean, focused interface for managing video metadata through YAML files, spreadsheet imports, and interactive table editing. 🤖 Generated with Claude Code (claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
…d appears first. - Add integration tests to verify that the key field is consistently the first column in the metadata table. - Update unit tests to confirm that the metadata table creation respects the specified column order based on the key field.
- Add module docstrings to all core beekeeper modules - Fix mypy type errors with proper type annotations and ignores - Update pre-commit hooks to use latest ruff (v0.12.11) and mypy (v1.17.1) - Fix ruff configuration format (tool.ruff.lint section) - Resolve SIM210 warnings (unnecessary True/False expressions) - Add missing function docstrings and parameter descriptions - Fix numerous docstring formatting issues (D205, D417, D400, D415) - Apply comprehensive code formatting with ruff and black - Fix codespell errors (doesn't, triggered) - Add noqa: C901 comment for complex function - Fix wheel dependency issue for check-manifest - Fix RST documentation formatting issues All pre-commit checks now pass: mypy, ruff, black, check-manifest, codespell. Reduced linting errors from 54 to 0 in core codebase. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
sfmig
force-pushed
the
smg/metadata-only
branch
from
c222dbe to
18e3cdf
Compare
sfmig
force-pushed
the
smg/metadata-only
branch
from
18e3cdf to
ce8c419
Compare
- Consolidated overlapping "Quick Start" and "Getting Started" sections into single "Getting Started" section - Removed redundant instructions and streamlined installation steps - Updated badges to match movement repository style: kept License, CI, Ruff, and pre-commit badges - Maintained clear "Core Operations" section as requested 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
…ation to point to the new repository at sfmig/beekeeper.
|
Claude encountered an error —— View job I'll analyze this and get back to you. |
|
Claude encountered an error —— View job I'll analyze this and get back to you. |
1 similar comment
|
Claude encountered an error —— View job I'll analyze this and get back to you. |
- Removed types-attrs as attrs is not used in the codebase - Removed types-requests as requests module is not used - Removed attrs from additional_dependencies as it's not needed Co-authored-by: sfmig <sfmig@users.noreply.github.com>
Welcome to Codecov 🎉Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests. Thanks for integrating Codecov - We've got you covered ☂️ |
Since Events and ROIs columns are no longer part of the metadata functionality after the WAZP to beekeeper transformation, all columns can now be editable. Co-authored-by: sfmig <sfmig@users.noreply.github.com>
- Update requires-python to >=3.11.0 - Update Python version classifiers to 3.11, 3.12, 3.13 - Remove Python 3.9 and 3.10 support per SPEC0 guidelines Co-authored-by: sfmig <sfmig@users.noreply.github.com>
No usage of PIL/Pillow found in the codebase, and black is not needed as the project uses ruff for formatting. Co-authored-by: sfmig <sfmig@users.noreply.github.com>
…less restrictive
4 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This PR modernizes the beekeeper application by transforming it from the legacy WAZP codebase into a focused, streamlined tool for video metadata management in animal behaviour experiments.
Key Changes:
• Major codebase restructuring: Renamed package from
wazptobeekeeperand removed ~3,000 lines of legacy code• Simplified application focus: Eliminated pose estimation, ROI management, and dashboard features to focus solely on metadata management
• Modernized technical stack: Updated deprecated Dash API calls, pandas methods, and removed obsolete dependencies
• Enhanced testing infrastructure: Updated test fixtures, improved integration tests, and modernized CI configuration
• Documentation overhaul: Comprehensive README with user workflows, updated project structure, and developer guidance
Test Plan
start-beekeeper🤖 Generated with Claude Code