Closed inexorabletash closed 8 months ago
@BruceDai can you take an initial look?
I apologize for my late response.
LGTM, thanks for these clarification texts.
Thanks @BruceDai - and no worries. I'm generating a lot of review requests. @fdwr and @huningxin - please review/merge at your leisure!
The shape calculation for argMin/argMax and reduction ops is now fully specified, but the implied behavior of {axes:[]} for these ops and layerNormalization was not particularly clear.
Add a little text to make it clearer.
Fixes #493
Preview | Diff