Skip to content

Expose and document NMS slider#2028

Merged
mcm001 merged 1 commit intoPhotonVision:mainfrom
samfreund:expose-nms
Aug 8, 2025
Merged

Expose and document NMS slider#2028
mcm001 merged 1 commit intoPhotonVision:mainfrom
samfreund:expose-nms

Conversation

@samfreund
Copy link
Copy Markdown
Member

@samfreund samfreund commented Aug 4, 2025

Description

The Non-Maximum Suppresion (NMS) Threshold slider is used to filter out overlapping detections. Lower values mean more detections are allowed through, but may result in false positives. It's generally recommended that teams leave this set at the default, unless they find they're unable to get usable results with solely the Confidence slider.

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

@samfreund samfreund requested a review from a team as a code owner August 4, 2025 01:52
@samfreund samfreund force-pushed the expose-nms branch 3 times, most recently from d05a556 to 7c06b27 Compare August 4, 2025 03:59
@samfreund samfreund requested a review from Gold856 August 4, 2025 04:07
@samfreund samfreund added the frontend Having to do with PhotonClient and its related items label Aug 4, 2025
@github-actions github-actions Bot added documentation Anything relating to https://docs.photonvision.org backend Things relating to photon-core and photon-server labels Aug 5, 2025
@Gold856 Gold856 removed the backend Things relating to photon-core and photon-server label Aug 6, 2025
@mcm001

This comment has been minimized.

Comment thread docs/source/docs/objectDetection/about-object-detection.md
@mcm001 mcm001 merged commit 3ac509b into PhotonVision:main Aug 8, 2025
43 checks passed
@samfreund samfreund deleted the expose-nms branch August 8, 2025 20:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Anything relating to https://docs.photonvision.org frontend Having to do with PhotonClient and its related items

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants