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

Include AR metrics in COCOEvaluator results #5175

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

carandraug
Copy link

This PR adds AR metrics to the results of COCOEvaluator. This was requested on the discussion of issue #1296 to which the reply was to submit a PR:

Originally posted by @ppwwyyxx in #1296 (comment):

does not contain the AR,

AR is not commonly used, and is therefore not included in results. You can send a PR to add it.

or any of the stats from above the line "Evaluation results for bbox".

everything except for AR are included.

Here's that PR.

I split this into two commits. The first commit adds the "global" AR metrics only (effectively the value displayed by COCOEval.summarize()). The second commit adds the per-class AR.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Dec 6, 2023
@carandraug carandraug changed the title Coco evaluator ar Include AR metrics in COCOEvaluator results Dec 6, 2023
@carandraug
Copy link
Author

The failed checks appear to be due to some issues with the CI configuration (indeed the last pushed commit that passed all the checks was 9 months ago)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants