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.

Sign up for GitHub

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

Jump to bottom

Fixed Average-Precision Integration Error #684

Merged
merged 2 commits into from
Jul 29, 2024
Merged

Fixed Average-Precision Integration Error #684

merged 2 commits into from
Jul 29, 2024

Conversation

czaloom
Copy link
Collaborator

@czaloom czaloom commented Jul 29, 2024

Changes

  • Fixed 101-point interpolation. A floating-point error was re-binning precisions to higher recalls.
  • Fixed functional test, now directly comparable to other sources.

@czaloom czaloom self-assigned this Jul 29, 2024
@czaloom czaloom added the bug Something isn't working label Jul 29, 2024
@czaloom czaloom marked this pull request as ready for review July 29, 2024 16:42
@czaloom czaloom requested review from ntlind and ekorman as code owners July 29, 2024 16:42
@czaloom czaloom changed the title Fixed Averaged-Precision Integration Error Fixed Average-Precision Integration Error Jul 29, 2024
@czaloom czaloom merged commit 5fcbd52 into main Jul 29, 2024
12 checks passed
@czaloom czaloom deleted the czaloom-obj-det-integration-bugfix branch July 29, 2024 17:13
ntlind added a commit that referenced this pull request Aug 7, 2024
@ntlind
add test changes from #684
5682cf4
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ekorman ekorman ekorman approved these changes

@ntlind ntlind Awaiting requested review from ntlind

Assignees

@czaloom czaloom

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@czaloom @ekorman