Add elixir bindings #535
Add elixir bindings #535
Conversation
| return make_error(env, ckzg_atoms.invalid_precompute_arg); | ||
|
|
||
| unsigned int len; | ||
| if (!enif_get_string_length(env, argv[0], &len, ERL_NIF_LATIN1)) |
There was a problem hiding this comment.
note that enif_get_string_length was only added since OTP 26.0 which was released in May 2023, not sure if we want to support older versions too?
|
Hey @BlazeWasHere, this is cool! I would be happy to include this when it's ready. Out of curiosity, what will use these bindings? And another TODO item would be to add CI tests, see: https://github.com/ethereum/c-kzg-4844/tree/main/.github/workflows |
|
@jtraglia no worries, one main user would be elixir_ethers via ExWeb3/elixir_ethers#178. also saw that lambdaclass used c-kzg-4844 in their elixir consensus client but used bindings from c -> rust -> elixir in |
|
Okay nice. So this does have real value. Feel free to ping me if you have any questions. |
Adds elixir bindings through an Erlang NIF
Was using the python bindings as a reference FFI implementation, not sure if i am missing some things
also not sure if this even cross compiles (i hope :cc_precompiler works)
Still a WIP:
compute_cellsfunction #534