Commit f168dbf
[ez][ET-VK] Small fix for choose_qparams_affine_impl (#16186)
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>1 parent 2284446 commit f168dbf
File tree
2 files changed
+10
-6
lines changed- backends/vulkan/runtime/graph/ops/impl
2 files changed
+10
-6
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
158 | 158 | | |
159 | 159 | | |
160 | 160 | | |
161 | | - | |
| 161 | + | |
| 162 | + | |
162 | 163 | | |
163 | 164 | | |
164 | 165 | | |
| |||
170 | 171 | | |
171 | 172 | | |
172 | 173 | | |
173 | | - | |
| 174 | + | |
| 175 | + | |
174 | 176 | | |
175 | 177 | | |
176 | 178 | | |
| |||
Lines changed: 6 additions & 4 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
369 | 369 | | |
370 | 370 | | |
371 | 371 | | |
372 | | - | |
| 372 | + | |
| 373 | + | |
373 | 374 | | |
374 | 375 | | |
375 | 376 | | |
| |||
380 | 381 | | |
381 | 382 | | |
382 | 383 | | |
383 | | - | |
| 384 | + | |
384 | 385 | | |
385 | 386 | | |
386 | 387 | | |
| |||
392 | 393 | | |
393 | 394 | | |
394 | 395 | | |
395 | | - | |
| 396 | + | |
| 397 | + | |
396 | 398 | | |
397 | 399 | | |
398 | 400 | | |
| |||
405 | 407 | | |
406 | 408 | | |
407 | 409 | | |
408 | | - | |
| 410 | + | |
409 | 411 | | |
410 | 412 | | |
411 | 413 | | |
| |||
0 commit comments