-
Notifications
You must be signed in to change notification settings - Fork 754
[ez][ET-VK] Update Vulkan runtime application name #16187
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
Conversation
It seems that `choose_qparams_affine` has recently appended some arguments to the schema. This causes newly exported models to break because at runtime, the output arg can no longer be found. Fix by locating the output argument as the last entry in the args vector, rather than continuously incrementing the args index. Update quantize/dequantize ops as well since it seems quantized_decomposed namespace ops are subject to change in the future. Note that it would be good to do this for all operators in the Vulkan backend as a later refactor. Differential Revision: [D88887463](https://our.internmc.facebook.com/intern/diff/D88887463/) [ghstack-poisoned]
TSIA! Differential Revision: [D88802100](https://our.internmc.facebook.com/intern/diff/D88802100/) [ghstack-poisoned]
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/16187
Note: Links to docs will display an error until the docs builds have been completed. ❌ 3 New Failures, 3 Unrelated FailuresAs of commit 9754870 with merge base a0a6278 ( NEW FAILURES - The following jobs have failed:
FLAKY - The following jobs failed but were likely due to flakiness present on trunk:
UNSTABLE - The following job is marked as unstable, possibly due to flakiness on trunk:
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
TSIA! Differential Revision: [D88802100](https://our.internmc.facebook.com/intern/diff/D88802100/) ghstack-source-id: 328515025 Pull Request resolved: #16187
This PR needs a
|
…ion name" TSIA! Differential Revision: [D88802100](https://our.internmc.facebook.com/intern/diff/D88802100/) cc manuelcandales digantdesai cbilgin [ghstack-poisoned]
TSIA! Differential Revision: [D88802100](https://our.internmc.facebook.com/intern/diff/D88802100/) cc manuelcandales digantdesai cbilgin [ghstack-poisoned]
Pull Request resolved: #16187 TSIA! Differential Revision: [D88802100](https://our.internmc.facebook.com/intern/diff/D88802100/) ghstack-source-id: 328525109
Stack from [ghstack](https://github.com/ezyang/ghstack) (oldest at bottom): * #16187 * __->__ #16186 It seems that `choose_qparams_affine` has recently appended some arguments to the schema. This causes newly exported models to break because at runtime, the output arg can no longer be found. Fix by locating the output argument as the last entry in the args vector, rather than continuously incrementing the args index. Update quantize/dequantize ops as well since it seems quantized_decomposed namespace ops are subject to change in the future. Note that it would be good to do this for all operators in the Vulkan backend as a later refactor. Differential Revision: [D88887463](https://our.internmc.facebook.com/intern/diff/D88887463/) --------- Co-authored-by: ssjia <ssjia@devvm1479.ncg0.facebook.com>
Stack from ghstack (oldest at bottom):
TSIA!
Differential Revision: D88802100
cc @manuelcandales @digantdesai @cbilgin