Skip to content

Preload OD models before import to check quantization#2056

Merged
samfreund merged 16 commits intoPhotonVision:mainfrom
boomermath:check-model-quant
Aug 16, 2025
Merged

Preload OD models before import to check quantization#2056
samfreund merged 16 commits intoPhotonVision:mainfrom
boomermath:check-model-quant

Conversation

@boomermath
Copy link
Contributor

@boomermath boomermath commented Aug 11, 2025

Description

Preloads a model to check for successful loading and quantization before importing it into PV.

Meta

Merge checklist:

  • Pull Request title is short, imperative summary of proposed changes
  • The description documents the what and why
  • If this PR changes behavior or adds a feature, user documentation is updated
  • If this PR touches photon-serde, all messages have been regenerated and hashes have not changed unexpectedly
  • If this PR touches configuration, this is backwards compatible with settings back to v2024.3.1
  • If this PR touches pipeline settings or anything related to data exchange, the frontend typing is updated
  • If this PR addresses a bug, a regression test for it is added

@github-actions github-actions bot added the backend Things relating to photon-core and photon-server label Aug 11, 2025
@samfreund

This comment has been minimized.

@samfreund
Copy link
Member

Can we adjust the description so it reflects what we're currently doing?

@samfreund samfreund marked this pull request as ready for review August 14, 2025 17:50
@samfreund samfreund requested a review from a team as a code owner August 14, 2025 17:50
samfreund
samfreund previously approved these changes Aug 14, 2025
@boomermath boomermath changed the title Add quantization check for ObjectDetector Preload OD models before import to check quantization Aug 14, 2025
samfreund
samfreund previously approved these changes Aug 15, 2025
samfreund
samfreund previously approved these changes Aug 16, 2025
@samfreund samfreund dismissed their stale review August 16, 2025 00:31

I should actually test it ngl

@samfreund samfreund merged commit 2eb224a into PhotonVision:main Aug 16, 2025
42 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend Things relating to photon-core and photon-server

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants