Skip to content

🚚 Additional improvement to toolbox setup workflow - #1112

Open
abdol wants to merge 8 commits into
developfrom
1111-additional-improvement-to-setup-workflow
Open

🚚 Additional improvement to toolbox setup workflow#1112
abdol wants to merge 8 commits into
developfrom
1111-additional-improvement-to-setup-workflow

Conversation

@abdol

@abdol abdol commented Aug 13, 2026

Copy link
Copy Markdown
Collaborator

Resolves #1111

This pull request makes a few more enhancements to the repo's setup process, in particular to the newly-added DeepSpot functionality:

TODO:

  • Move DeepSpot setup.py to pyproject.toml
  • Add DeepSpot install instructions to the developer install steps
  • Adding uv.lock to .gitignore (already done in 🧑‍💻 Add uv setup workflow #1058)
  • Use MonkeyPatch in the uv unit tests The unit tests does not run models as it verifies versions and imports some classes like semantic_segmentor.

Related to #1103 and #1058. Please feel free to test on different platform to verify that it works on Windows, macOS, and Linux. Let me know if you have any comments, questions, or feedback.

@abdol
abdol requested a review from shaneahmed August 13, 2026 10:03
@abdol abdol self-assigned this Aug 13, 2026
@abdol abdol added documentation Improvements or additions to documentation dependencies Pull requests that update a dependency file labels Aug 13, 2026
@abdol abdol linked an issue Aug 13, 2026 that may be closed by this pull request
@abdol
abdol force-pushed the 1111-additional-improvement-to-setup-workflow branch from dd8760e to ced3d18 Compare August 13, 2026 10:04
@abdol abdol changed the title 🚚 Additional improvement to uv setup workflow 🚚 Additional improvement to toolbox setup workflow Aug 13, 2026
@codecov

codecov Bot commented Aug 13, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.90%. Comparing base (8d13267) to head (b5a2691).

Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #1112   +/-   ##
========================================
  Coverage    99.90%   99.90%           
========================================
  Files           93       93           
  Lines        11911    11911           
  Branches      1562     1562           
========================================
  Hits         11900    11900           
  Misses           5        5           
  Partials         6        6           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Comment thread docs/installation.rst Outdated
Comment thread CONTRIBUTING.rst Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Additional improvement to setup workflow

2 participants