-
Notifications
You must be signed in to change notification settings - Fork 7.1k
MKL warning on Python 3.10 unittest linux CI #7372
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
Comments
I finally have a reproducer:
But removing numpy makes everything work as expected. This happens because But it behaves the same for 1.13, so this is not a new problem. Even simpler reproducer that affects all our conda builds (why
|
Following fixes the test issues: diff --git a/test/test_transforms.py b/test/test_transforms.py
index 03b385e..8ff61ce 100644
--- a/test/test_transforms.py
+++ b/test/test_transforms.py
@@ -2283,6 +2283,7 @@ def test_functional_deprecation_warning(import_statement, from_private):
import_statement = import_statement.replace("functional", "_functional")
source = f"""
import warnings
+ import numpy
with warnings.catch_warnings():
warnings.simplefilter("error")
@@ -2291,6 +2292,7 @@ def test_functional_deprecation_warning(import_statement, from_private):
else:
source = f"""
import pytest
+ import numpy
with pytest.warns(UserWarning, match="removed in 0.17"):
{import_statement}
""" |
But this is still a significant issue, right? If every single user on 3.10 gets a warning depending on whether or not they import numpy, this seems like a pretty bad UX? |
Indeed, it should be fixed, though it would prefer to understand what is causing this as well. As in the same environment, both |
Unit-tests on Linux CPU / tests (3.10) / linux-job is failing because
import torchvision
is causing the following warning:This is an issue likely due to the way the CI is setup as the issue is not reproducible in different settings e.g. #7299 (comment) or #7299 (comment)
The text was updated successfully, but these errors were encountered: