change argmin/argmax to take scalar axis

webmachinelearning / webnn

🧠 Web Neural Network API

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

Other

397 stars 48 forks source link

change argmin/argmax to take scalar axis #724

Closed philloooo closed 4 months ago

philloooo commented 4 months ago

fixes #629

Changes argmin/argmax to take scalar axis instead of axes list.
Remove default value for axis, so now it becomes positional argument

@fdwr @huningxin please take a look :)

Preview | Diff

fdwr commented 4 months ago

Note forced pushes make it harder to compare iterations in the Files tab 🔍😐, as the "Show changes since your last review" button doesn't work correctly.