elixir-nx / emlx

MLX Backend for Nx
Apache License 2.0
64 stars 1 forks source link

Add CI workflow #29

Closed polvalente closed 1 week ago

polvalente commented 1 week ago

We need basic CI for running tests + ensuring format and warnings. Adapting the one from Nx/Torchx should be fine

samrat commented 1 week ago

I can take this one up.

samrat commented 1 week ago

Looks like the setup-beam action we use in Nx only supports Windows & Linux:

Error: @erlef/setup-beam only supports Ubuntu and Windows at this time

Trying to see if someone has done this before for macOS. If not, I'll look into adding macOS support in https://github.com/erlef/setup-beam/blob/ff4f054c9196e7939f3d1316771d9236c46ef126/src/setup-beam.js#L751

samrat commented 1 week ago

Ah, never mind. Looks like it's already in progress in #38

cocoa-xu commented 1 week ago

Hi @samrat sorry I was testing while you were replying to this issue.

Looks like the setup-beam action we use in Nx only supports Windows & Linux:

We can use https://elixir-lang.org/install.sh to install both Erlang/OTP and Elixir for macOS. I've created a PR for it