Bug fix: gather() steps shouldn't invoke methods

webmachinelearning / webnn

🧠 Web Neural Network API

https://www.w3.org/TR/webnn/

Other

398 stars 48 forks source link

Bug fix: gather() steps shouldn't invoke methods #578

Closed inexorabletash closed 9 months ago

inexorabletash commented 9 months ago

The method steps check an operand's data type, which should not be done by invoking a method (because side effects) but by looking at the internal state.

Preview | Diff

inexorabletash commented 9 months ago

Totally trivial, I just happened to notice this minor glitch. A quick search didn't turn up any other examples.