Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix Example in Detections.merge #1803

Closed
wants to merge 1 commit into from
Closed

Conversation

Gk-rohan
Copy link
Contributor

Description

fix the example given for merging detections here https://supervision.roboflow.com/0.20.0/detection/core/#supervision.detection.core.Detections.merge

Type of change

  • This change requires a documentation update

How has this change been tested, please provide a testcase or example of how you tested the change?

Followed the procedure using mkdocs to serve the documentation locally.

Docs

  • Docs updated? What were the changes:

@onuralpszr
Copy link
Collaborator

onuralpszr commented Mar 18, 2025

@Gk-rohan I didn't understand what wrong because it works fine in current example code but your change break the example

image

@onuralpszr
Copy link
Collaborator

onuralpszr commented Mar 18, 2025

Description

fix the example given for merging detections here https://supervision.roboflow.com/0.20.0/detection/core/#supervision.detection.core.Detections.merge

Type of change

  • This change requires a documentation update

How has this change been tested, please provide a testcase or example of how you tested the change?

Followed the procedure using mkdocs to serve the documentation locally.

Docs

  • Docs updated? What were the changes:

You are also are using "old" version docs which we already fixed in latest stable as well.

@Gk-rohan
Copy link
Contributor Author

ok my bad, didn't check the docs link. Thanks @onuralpszr, closing this.

@Gk-rohan Gk-rohan closed this Mar 19, 2025
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