-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
[UX] Improve TensorIR jupyter misuse error message improvement suggestions #9551
Comments
UX issue should be treated as a bug |
The second issue is easier to fix: rewrite the error message. The first issue is more profound: it comes from the type checking in packed function interface. One workaround is to check the input types on python side for each API. However I don't know better any general approaches to do so. On the other hand, if |
Is it possible to turn off the stacktrace? It doesn't look useful in jupyter |
Let us check on the python side for the first one |
Updated note on the c++ stacktrace side. The main annoying part are the expanded macro dispatchers in the FFI boundary that does not provide additional information. So one solution is to update the trace normalizer to simplify and trim some of the less informative traces(e.g. unpackcall_dispatcher) but keep the rest of the useful traces |
@tqchen The dispatcher can be completely avoided by using other metaprogramming tricks |
Some lessons when trying out TensorIR on a jupyter session due to mis-use
https://gist.github.com/tqchen/01915c90334dbc2699971afe40dd2996
The text was updated successfully, but these errors were encountered: