Skip to content

Fix cocotools change for missing info key #3416

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
Merged
vfdev-5 merged 1 commit into from
Jun 11, 2025
Merged

Fix cocotools change for missing info key #3416

vfdev-5 merged 1 commit into from
Jun 11, 2025

Conversation

@goanpeca
Copy link
Collaborator

@goanpeca goanpeca commented Jun 11, 2025 •
edited
Loading

It seems Coco now expects the annotations file, since it is copying info key, so I added that to the dataset in the meantime.

Requires #3414 to get fixes for nightly

2025-06-11T17:39:05.3129969Z =========================== short test summary info ============================
2025-06-11T17:39:05.3130730Z FAILED tests/ignite/metrics/vision/test_object_detection_map.py::test_empty_data[cpu] - KeyError: 'info'
2025-06-11T17:39:05.3131593Z ERROR tests/ignite/metrics/vision/test_object_detection_map.py::test_iou[sample0-cpu] - KeyError: 'info'
2025-06-11T17:39:05.3132669Z ERROR tests/ignite/metrics/vision/test_object_detection_map.py::test_iou[sample1-cpu] - KeyError: 'info'
2025-06-11T17:39:05.3133503Z ERROR tests/ignite/metrics/vision/test_object_detection_map.py::test_iou[sample2-cpu] - KeyError: 'info'
2025-06-11T17:39:05.3134307Z ERROR tests/ignite/metrics/vision/test_object_detection_map.py::test_iou[sample3-cpu] - KeyError: 'info'
2025-06-11T17:39:05.3135138Z ERROR tests/ignite/metrics/vision/test_object_detection_map.py::test_iou[sample4-cpu] - KeyError: 'info'
2025-06-11T17:39:05.3136343Z ERROR tests/ignite/metrics/vision/test_object_detection_map.py::test_iou[sample5-cpu] - KeyError: 'info'
2025-06-11T17:39:05.3137185Z ERROR tests/ignite/metrics/vision/test_object_detection_map.py::test_iou[sample6-cpu] - KeyError: 'info'
2025-06-11T17:39:05.3138004Z ERROR tests/ignite/metrics/vision/test_object_detection_map.py::test_iou[sample7-cpu] - KeyError: 'info'
2025-06-11T17:39:05.3138810Z ERROR tests/ignite/metrics/vision/test_object_detection_map.py::test_compute[sample0] - KeyError: 'info'
2025-06-11T17:39:05.3139601Z ERROR tests/ignite/metrics/vision/test_object_detection_map.py::test_compute[sample1] - KeyError: 'info'
2025-06-11T17:39:05.3140246Z ERROR tests/ignite/metrics/vision/test_object_detection_map.py::test_compute[sample2] - KeyError: 'info'
2025-06-11T17:39:05.3140888Z ERROR tests/ignite/metrics/vision/test_object_detection_map.py::test_compute[sample3] - KeyError: 'info'
2025-06-11T17:39:05.3141533Z ERROR tests/ignite/metrics/vision/test_object_detection_map.py::test_compute[sample4] - KeyError: 'info'
2025-06-11T17:39:05.3142159Z ERROR tests/ignite/metrics/vision/test_object_detection_map.py::test_compute[sample5] - KeyError: 'info'
2025-06-11T17:39:05.3142794Z ERROR tests/ignite/metrics/vision/test_object_detection_map.py::test_compute[sample6] - KeyError: 'info'
2025-06-11T17:39:05.3143448Z ERROR tests/ignite/metrics/vision/test_object_detection_map.py::test_compute[sample7] - KeyError: 'info'
2025-06-11T17:39:05.3144100Z ERROR tests/ignite/metrics/vision/test_object_detection_map.py::test_integration[sample0] - KeyError: 'info'
2025-06-11T17:39:05.3144765Z ERROR tests/ignite/metrics/vision/test_object_detection_map.py::test_integration[sample1] - KeyError: 'info'
2025-06-11T17:39:05.3145426Z ERROR tests/ignite/metrics/vision/test_object_detection_map.py::test_integration[sample2] - KeyError: 'info'
2025-06-11T17:39:05.3146199Z ERROR tests/ignite/metrics/vision/test_object_detection_map.py::test_integration[sample3] - KeyError: 'info'
2025-06-11T17:39:05.3146975Z ERROR tests/ignite/metrics/vision/test_object_detection_map.py::test_integration[sample4] - KeyError: 'info'
2025-06-11T17:39:05.3147671Z ERROR tests/ignite/metrics/vision/test_object_detection_map.py::test_integration[sample5] - KeyError: 'info'
2025-06-11T17:39:05.3148347Z ERROR tests/ignite/metrics/vision/test_object_detection_map.py::test_integration[sample6] - KeyError: 'info'
2025-06-11T17:39:05.3149014Z ERROR tests/ignite/metrics/vision/test_object_detection_map.py::test_integration[sample7] - KeyError: 'info'
2025-06-11T17:39:05.3149766Z ERROR tests/ignite/metrics/vision/test_object_detection_map.py::test_distrib_update_compute[gloo_cpu-sample0] - KeyError: 'info'
2025-06-11T17:39:05.3150550Z ERROR tests/ignite/metrics/vision/test_object_detection_map.py::test_distrib_update_compute[gloo_cpu-sample1] - KeyError: 'info'
2025-06-11T17:39:05.3151417Z ERROR tests/ignite/metrics/vision/test_object_detection_map.py::test_distrib_update_compute[gloo_cpu-sample2] - KeyError: 'info'
2025-06-11T17:39:05.3152273Z ERROR tests/ignite/metrics/vision/test_object_detection_map.py::test_distrib_update_compute[gloo_cpu-sample3] - KeyError: 'info'
2025-06-11T17:39:05.3153043Z ERROR tests/ignite/metrics/vision/test_object_detection_map.py::test_distrib_update_compute[gloo_cpu-sample4] - KeyError: 'info'
2025-06-11T17:39:05.3153816Z ERROR tests/ignite/metrics/vision/test_object_detection_map.py::test_distrib_update_compute[gloo_cpu-sample5] - KeyError: 'info'
2025-06-11T17:39:05.3154651Z ERROR tests/ignite/metrics/vision/test_object_detection_map.py::test_distrib_update_compute[gloo_cpu-sample6] - KeyError: 'info'
2025-06-11T17:39:05.3155429Z ERROR tests/ignite/metrics/vision/test_object_detection_map.py::test_distrib_update_compute[gloo_cpu-sample7] - KeyError: 'info'

@goanpeca goanpeca marked this pull request as ready for review June 11, 2025 18:11
vfdev-5
vfdev-5 approved these changes Jun 11, 2025
View reviewed changes
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 Gonzalo!

@vfdev-5 vfdev-5 merged commit 87e4c8d into pytorch:master Jun 11, 2025
10 of 17 checks passed
@goanpeca goanpeca deleted the fix/cocotools branch June 11, 2025 20:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vfdev-5 vfdev-5 vfdev-5 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@goanpeca @vfdev-5