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

Add ComparisonAnnotator, xyxy_to_polygons #1716

Merged
merged 4 commits into from
Dec 11, 2024
Merged

Conversation

LinasKo
Copy link
Contributor

@LinasKo LinasKo commented Dec 6, 2024

Description

ComparisonAnnotator enables comparing of two Detections objects, highlighting the difference. It's useful for both model-model and prediction-target comparison.

Features:

  • ComparisonAnnotator itself
  • ComparisonAnnotator automatically selects whether obb, mask or xyxy should be used.
  • If labels are specified, text boxes with color markers will be drawn
  • Only one parameter is required to modify label size - no need to think about text thickness, font, etc
  • xyxy_to_polygons utils function

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • This change requires a documentation update

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

Examples in comments.

Any specific deployment considerations

None

Docs

  • Docs updated? What were the changes:

@LinasKo
Copy link
Contributor Author

LinasKo commented Dec 6, 2024

comparison_det_dog

comparison_det_ppl

comparison_seg_dog

comparison_seg_ppl

comparison_obb

LinasKo and others added 2 commits December 11, 2024 16:58
…review

* Up to 20 tabs can be made at once. This is likely a bug relating to the underlying mkdocs plugin, but a quick check for hardcoded values yielded nothing.
@LinasKo
Copy link
Contributor Author

LinasKo commented Dec 11, 2024

Docs updated, but commented out two tabs in the preview: BoxCornersandCrop` annotators.

This is due to an issue I found - a maximum of 20 tabs is possible at the moment. #1729

@LinasKo LinasKo merged commit 1633522 into develop Dec 11, 2024
12 checks passed
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.

1 participant