search

elixir-nx / nx

Multi-dimensional arrays (tensors) and numerical definitions for Elixir
2.66k stars 193 forks source link

Rename sigils to ~VEC and ~MAT #1114

Closed josevalim closed 6 months ago

josevalim commented 1 year ago

I sent a proposal for multi-letter uppercase sigils in Elixir. If accepted, we should rename our sigils to the ones above.

dannote commented 1 year ago

Can you explain why you don't want to keep the shorter versions? Are they confusing?

josevalim commented 1 year ago

Yes, pretty much. And given we are before 1.0 and still early in the adoption process, we can afford to rename without taking a single letter sigil. Furthermore, single letter sigils are by definition limited and it increases the chances of conflicts with other community projects. Elixir (and libraries) rarely use shortcuts and sigils were an exception so far.

josevalim commented 8 months ago

We can now tackle this. Next Nx version can depend on Elixir v1.15.

josevalim commented 6 months ago

Done in 510e68903c46362eedb8f169cb079cf68da0f6c5.