This repository has been archived by the owner on Nov 16, 2023. It is now read-only.

[Bug] Compute a node many times #53

Closed

NTT123 opened this issue Dec 25, 2018 · 1 comment

Labels

bug

Contributor

NTT123 commented Dec 25, 2018 •

edited

Loading

https://github.com/Microsoft/onnxjs/blob/c2f3de6d934e807ec7880ea3e9438ce519d1b8f6/lib/execution-plan.ts#L136

When a node has two identical inputs, e.g. y = concat(x, x, dim=0), the above line will add y twice to the list sequence. This causes an exception at https://github.com/Microsoft/onnxjs/blob/c2f3de6d934e807ec7880ea3e9438ce519d1b8f6/lib/execution-plan.ts#L118 when y is computed the second time.

The text was updated successfully, but these errors were encountered:

fs-eire added the bug label

fs-eire mentioned this issue

bugfix: compute a node many times #88

Merged

Member

hariharans29 commented Feb 21, 2019

Fixed by #88

hariharans29 closed this as completed

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.