-
Notifications
You must be signed in to change notification settings - Fork 45
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
[torchlib] Improve xfail
decorator to be more robust and accurate
#794
Labels
topic: torch_lib
Related to the torch/aten function lib in development
Comments
BowenBao
added
the
topic: torch_lib
Related to the torch/aten function lib in development
label
Jun 19, 2023
Code is here: onnxscript/onnxscript/tests/function_libs/torch_lib/ops_test_common.py Lines 562 to 591 in 2134fa3
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
xfail
now is powered bypytest.xfail(reason=reason)
. While it is great and better thanskip
, it does not offer check on the actual failure reason. Hence through time the descriptivereason
may become outdated and misleading.An option is to rewrite decorator with
self.assertRaiseRegex
.cc @justinchuby, @titaiwangms
The text was updated successfully, but these errors were encountered: