catalyst-team / catalyst

Accelerated deep learning R&D
https://catalyst-team.com
Apache License 2.0
3.3k stars 388 forks source link

Update _average_precision.py #1431

Closed gaarutyunov closed 1 year ago

gaarutyunov commented 2 years ago

Pretty sure it is not necessary to calculate the precision twice for each k. If it is not some kind of trick :)

Pull Request FAQ

Description

line is repeated twice

Related Issue

-

Type of Change

PR review

Anyone in the community is free to review the PR once the tests have passed. If we didn't discuss your PR in Github issues there's a high chance it will not be merged.

Checklist

CLAassistant commented 2 years ago

CLA assistant check
All committers have signed the CLA.

stale[bot] commented 1 year ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.