You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This is not new after the v2 port though. It goes all the way back to the combination of #4025 and #4280. The former introduced the import of common_utils in conftest while the latter introduced a torchvision import in common_utils. Meaning, we haven't been running our torchhub tests for roughly 1.5 years now:
For example https://app.circleci.com/pipelines/github/pytorch/vision/23876/workflows/d96da5f3-9ca0-4615-9c08-0373c00233a0/jobs/1849866?invite=true#step-105-10
The problem is that we skip in case
vision/test/test_hub.py
Line 20 in 5850f37
but we do that on multiple occasions during collection. The newest examples are
vision/test/conftest.py
Lines 6 to 9 in 5850f37
and implicitly in
vision/test/conftest.py
Line 11 in 5850f37
for example
vision/test/common_utils.py
Lines 25 to 28 in 5850f37
This is not new after the v2 port though. It goes all the way back to the combination of #4025 and #4280. The former introduced the import of
common_utils
inconftest
while the latter introduced atorchvision
import incommon_utils
. Meaning, we haven't been running our torchhub tests for roughly 1.5 years now:https://app.circleci.com/pipelines/github/pytorch/vision/9847/workflows/813b8bbf-7aa4-4a84-b3ab-2b453035c206/jobs/735233?invite=true#step-102-64
We should really not just skip in such cases.
The text was updated successfully, but these errors were encountered: