Chris Costes

Results 9 comments of Chris Costes

Sure, this is on the latest commit on `main`, python 3.11.6, macOS 13.5 on an M1 MacBook Air. Let me know if there's any other info that might be useful....

Digging deeper, I am also getting the same exception from a few of the the C++ tests, including [autograd_tests](https://github.com/ml-explore/mlx/blob/main/tests/autograd_tests.cpp#L769), [creations_tests](https://github.com/ml-explore/mlx/blob/main/tests/creations_tests.cpp#L100), and most interestingly this simple test from [metal_tests](https://github.com/ml-explore/mlx/blob/main/tests/metal_tests.cpp#L105): ``` a...

Debugging just the `test metal reduce` test case it is reliably hitting a divide-by-zero (not floating-point as I saw in the others) exception in [reduce.cpp](https://github.com/ml-explore/mlx/blob/main/mlx/backend/metal/reduce.cpp#L56) due to `thread_group_size` (as well...

I added an early-exit to `reduce` which allowed the metal tests to pass, but now I'm seeing more failures from `ops_tests` (in addition to `autograd` and `creations` mentioned above). That,...

I think I found the problem: Rosetta! When building I didn't notice the following: ``` -- Building MLX for x86_64 processor on Darwin CMake Warning at CMakeLists.txt:33 (message): Building for...

That's the x86 brew cmake installation. Might just need to `brew install cmake` again. On Thu, Jan 11, 2024 at 12:43 PM Jhenner Tigreros ***@***.***> wrote: > Thanks in advance...

Got it, thanks! Will look into trying to implement it; seems like it might be straightforward. I'm enjoying mlx so far!

Don't have time to dig into it right now, but the fact that I was able to use this library to tune an FM station makes me think it's probably...

Turns out it was an error in the test - as the comment in `read_reg` states: `Read registers as little endian, but write as big; not sure why`, but the...