Commit c8f3078
jorgep31415
[ET-VK] Fix SDK
Pull Request resolved: #7738
## Issue
In the ET-SDK, we assign an `event_name` to each operation. In ET-VK, we compose a unique `event_name` using the `node_id`. The `node_id` exists for every `OperatorCall` but not for input/output with `nchw_to_image`/`image_to_nchw`. Those cases collapse into `node_id == 0` which means all `nchw_to_image` had the same `event_name` and hence only one was stored. The same reasoning results in storage of only one `image_to_nchw`.
## Solution
Ignore the serialized `node_id` and use the operation's `prepack_node`/`execute_node` vector index.
TODO: Determine if we can remove the serialized `node_id`, or whether this should be fixed differently and still reference it.
ghstack-source-id: 262309622
@exported-using-ghexport
Differential Revision: [D68344534](https://our.internmc.facebook.com/intern/diff/D68344534/)event_name for inputs and outputs1 parent 8611aaa commit c8f3078
1 file changed
+9
-19
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
334 | 334 | | |
335 | 335 | | |
336 | 336 | | |
337 | | - | |
338 | | - | |
339 | | - | |
340 | 337 | | |
341 | 338 | | |
342 | 339 | | |
| |||
351 | 348 | | |
352 | 349 | | |
353 | 350 | | |
354 | | - | |
355 | | - | |
356 | | - | |
357 | | - | |
358 | | - | |
359 | | - | |
360 | | - | |
361 | | - | |
362 | | - | |
363 | | - | |
364 | | - | |
365 | | - | |
366 | | - | |
367 | | - | |
368 | | - | |
369 | | - | |
370 | 351 | | |
371 | 352 | | |
372 | 353 | | |
| |||
379 | 360 | | |
380 | 361 | | |
381 | 362 | | |
| 363 | + | |
| 364 | + | |
| 365 | + | |
| 366 | + | |
| 367 | + | |
| 368 | + | |
| 369 | + | |
| 370 | + | |
| 371 | + | |
382 | 372 | | |
383 | 373 | | |
384 | 374 | | |
| |||
0 commit comments