Skip to content

Update test_average_precision.py to resolve issue #3335 #3337

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

Merged
merged 4 commits into from
Feb 28, 2025

Conversation

puhuk
Copy link
Contributor

@puhuk puhuk commented Feb 27, 2025

Related to #3335

Description: Resolved the issue with test_average_precision.py when running in distributed mode, as reported in #3335.

Check list:

  • New tests are added (if a new feature is added)
  • New doc strings: description and/or example code are in RST format
  • Documentation is updated (if required)

Copy link
Collaborator

@vfdev-5 vfdev-5 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR @puhuk
I left few comments how to make it better

@vfdev-5
Copy link
Collaborator

vfdev-5 commented Feb 27, 2025

Let's keep track of https://github.com/pytorch/ignite/actions/runs/13568080551/job/37925851877?pr=3337 job and see whether there are related failing tests. If some related tests are failing we need to provide a fix in this PR.

@puhuk
Copy link
Contributor Author

puhuk commented Feb 27, 2025

Let's keep track of https://github.com/pytorch/ignite/actions/runs/13568080551/job/37925851877?pr=3337 job and see whether there are related failing tests. If some related tests are failing we need to provide a fix in this PR.

Got it! Keeping an eye on it! If anything breaks, I’ll be ready to fix it. 🚀

@puhuk
Copy link
Contributor Author

puhuk commented Feb 28, 2025

@vfdev-5
I have a question regarding the mps-tests. It reports an error in test_precision.
Why does this error occur even though I only modified test_average_precision?
Are you suggesting that test_precision should also be fixed in this PR?

@vfdev-5
Copy link
Collaborator

vfdev-5 commented Feb 28, 2025

@vfdev-5 I have a question regarding the mps-tests. It reports an error in test_precision. Why does this error occur even though I only modified test_average_precision? Are you suggesting that test_precision should also be fixed in this PR?

This should be done in #3334
which is almost finished (remains one failure in one test). Once that PR is merged you can rebase your PR on the master and the unrelated failures should disappear.

Copy link
Collaborator

@vfdev-5 vfdev-5 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @puhuk ! LGTM

@vfdev-5 vfdev-5 enabled auto-merge (squash) February 28, 2025 14:16
@vfdev-5 vfdev-5 merged commit 6401a59 into pytorch:master Feb 28, 2025
13 of 14 checks passed
@puhuk puhuk deleted the test_average_precision_device branch February 28, 2025 16:18
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