Skip to content

Conversation

LinasKo
Copy link
Contributor

@LinasKo LinasKo commented Sep 3, 2024

Description

COCO 101 point Average Precision algorithm previously interpolated the precision values, causing skew in the precision line.

Now, we're integrating over the max precision value seen up to a given recall level.

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?

Multiple notebooks, primarily this.

Any specific deployment considerations

Docs

@LinasKo LinasKo self-assigned this Sep 3, 2024
Copy link
Contributor

github-actions bot commented Sep 3, 2024

Preview

Preview and run these notebook edits with Google Colab: Rendered notebook diffs available on ReviewNB.com. If commits are added to the pull request, synchronize your local branch: git pull origin fix/101-coco-algorithm

@SkalskiP
Copy link
Collaborator

SkalskiP commented Sep 4, 2024

I'm merging this PR to develop as we are actively working on integrating mAP into maestro. If we will find any previously unseen bugs, we will just create a new PR in supervision.

@SkalskiP SkalskiP merged commit 621f8e4 into develop Sep 4, 2024
9 checks passed
@onuralpszr onuralpszr deleted the fix/101-coco-algorithm branch September 23, 2024 15:10
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