-
Notifications
You must be signed in to change notification settings - Fork 51
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
Add filterwarnings in some onnx tests #197
Conversation
Can we narrow down the filter a bit more? (maybe limit to torch.floor_divide) |
@@ -142,6 +142,7 @@ def _fn2(x): | |||
|
|||
|
|||
@pytest.mark.filterwarnings("ignore::torch.jit.TracerWarning") | |||
@pytest.mark.filterwarnings("ignore::UserWarning") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Deprecation warning of torch.floor_divide
raised from here: https://github.com/pytorch/pytorch/blob/v1.9.0/torch/nn/functional.py#L2358
@@ -62,6 +62,7 @@ def _helper(model, args, d, **kwargs): | |||
return output_dir | |||
|
|||
|
|||
@pytest.mark.filterwarnings("ignore::UserWarning") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
An experimental warning for named tensors is raised, but this test looks not to use named tensor 🤔
UserWarning: Named tensors and all their associated APIs are an experimental feature and subject to change. Please do not use them for anything important until they are released as stable. (Triggered internally at /pytorch/c10/core/TensorImpl.h:1156.)
Sync github master to PPE
@kmaehashi Could you take another look? |
Could you resolve conflicts? LGTM otherwise! |
/test |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
Part #193