Awni Hannun
Awni Hannun
Cool, I don't mind adding that though I find it a bit odd how numpy has different syntax for the same thing. In `mx` we went with `None` for adding...
Yes sure!
Sent a comment, but o/w looks great, thanks for adding thos!
That doesn't look good. Which commit are you building on? Also can you give some more details on your environment? E.g. OS, python version, ...
I'm not able to repro on my M1 but my situation is a bit different :\. I'm on 14.2 and M1 Max. Does running the test standalone cause the issue...
Wow, that's interesting.. Can you try running some of those tests standalon? Like just the test that fails? (Comment out the rest or something). I'm really curious about: 1. Is...
Cool, thanks for finding that! That looks like an easy fix now that you narrowed it down. Interested in sending a PR for it?
> added an early-exit to reduce which allowed the metal tests to pass Rather than an early exit, I was thinking to check if there is a zero in the...
@ccostes There is definitely a divide by zero. The fact we don't get an exception makes me think you are using a different compiler as it is generally undefined behavior...
@ccostes could you check my PR and see if it fixes your issue? I didn't see any other instances where we divide by integer 0. #388