-
Notifications
You must be signed in to change notification settings - Fork 21.6k
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
Debug TorchScript error from fastNLP_Bert #93742
Labels
Comments
18 tasks
You can assign this to me |
eellison
referenced
this issue
Mar 28, 2022
… in JIT" Fix for https://github.com/facebookresearch/torchdynamo/issues/93 Because the constructor follow a non-standard input schema (variadic integers), they are handled specially in ir_emitter. [ghstack-poisoned]
eellison
referenced
this issue
Mar 28, 2022
Fix for https://github.com/facebookresearch/torchdynamo/issues/93 Because the constructor follow a non-standard input schema (variadic integers), they are handled specially in ir_emitter. [ghstack-poisoned]
eellison
referenced
this issue
Mar 28, 2022
… in JIT" Fix for https://github.com/facebookresearch/torchdynamo/issues/93 Because the constructor follow a non-standard input schema (variadic integers), they are handled specially in ir_emitter. [ghstack-poisoned]
eellison
referenced
this issue
Mar 28, 2022
Fix for https://github.com/facebookresearch/torchdynamo/issues/93 Because the constructor follow a non-standard input schema (variadic integers), they are handled specially in ir_emitter. [ghstack-poisoned]
eellison
referenced
this issue
Mar 31, 2022
… in JIT" Fix for https://github.com/facebookresearch/torchdynamo/issues/93 Because the constructor follow a non-standard input schema (variadic integers), they are handled specially in ir_emitter. [ghstack-poisoned]
eellison
referenced
this issue
Mar 31, 2022
Fix for https://github.com/facebookresearch/torchdynamo/issues/93 Because the constructor follow a non-standard input schema (variadic integers), they are handled specially in ir_emitter. [ghstack-poisoned]
eellison
referenced
this issue
Apr 5, 2022
… in JIT" Fix for https://github.com/facebookresearch/torchdynamo/issues/93 Because the constructor follow a non-standard input schema (variadic integers), they are handled specially in ir_emitter. Differential Revision: [D35362762](https://our.internmc.facebook.com/intern/diff/D35362762) [ghstack-poisoned]
eellison
referenced
this issue
Apr 5, 2022
Fix for https://github.com/facebookresearch/torchdynamo/issues/93 Because the constructor follow a non-standard input schema (variadic integers), they are handled specially in ir_emitter. Differential Revision: [D35362762](https://our.internmc.facebook.com/intern/diff/D35362762) [ghstack-poisoned]
facebook-github-bot
referenced
this issue
Apr 6, 2022
Summary: Pull Request resolved: #74785 Fix for https://github.com/facebookresearch/torchdynamo/issues/93 Because the constructor follow a non-standard input schema (variadic integers), they are handled specially in ir_emitter. Test Plan: Imported from OSS Reviewed By: ejguan Differential Revision: D35362762 Pulled By: eellison fbshipit-source-id: 960badf08ba2ab0818af5fd331aff3542051250f
pytorchmergebot
referenced
this issue
Apr 6, 2022
Summary: Pull Request resolved: #74785 Fix for https://github.com/facebookresearch/torchdynamo/issues/93 Because the constructor follow a non-standard input schema (variadic integers), they are handled specially in ir_emitter. Test Plan: Imported from OSS Reviewed By: ejguan Differential Revision: D35362762 Pulled By: eellison fbshipit-source-id: 960badf08ba2ab0818af5fd331aff3542051250f (cherry picked from commit bd579de)
@eellison I still see this error after getting your latest PR. The error signature looks like this. Reopening for now.
|
@anijain2305 can you post the full error? |
@eellison This is the whole error and stack trace
|
@anijain2305 filed #75465 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Later on there is the error (when jit.trace fails, it tries jit.script)
These sounds like just missing ops? So perhaps easy to fix? They prevent all TorchScript-based backends from working on these models.
@eellison do these look familar to you?
cc @ezyang @soumith @msaroufim @wconstab @ngimel @bdhirsh
The text was updated successfully, but these errors were encountered: