Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Implement NMT in Rust + make it easy to use from other languages #84

Closed
liamsi opened this issue Dec 7, 2022 · 4 comments
Closed

Implement NMT in Rust + make it easy to use from other languages #84

liamsi opened this issue Dec 7, 2022 · 4 comments

Comments

@liamsi
Copy link
Member

liamsi commented Dec 7, 2022

We got this feedback:

Verifying NMT proofs will probably a common operation. It would be nice eventually to have an implementation in a language like C/C++/Rust that people could access via FFI/bindings.

I think we should prioritize this post-launch.
@staheri14 staheri14 mentioned this issue Apr 4, 2023
Closed
@rootulp
Copy link
Collaborator

rootulp commented May 25, 2023

Implement NMT in Rust

https://github.com/Sovereign-Labs/nmt-rs

@adlerjohn
Copy link
Member

Should this issue be closed in favor of the above repo (perhaps documenting in the readme that non-endorsed implementations in other languages exist?), or do we want to have an in-house implementation? Thoughts @liamsi @evan-forbes?

@evan-forbes
Copy link
Member

at the minimum linking other nmt repos and their current status (ie maintained or feature complete etc) seems like a really good idea

rootulp added a commit that referenced this issue Aug 28, 2023
@rootulp
docs: add a table for alternative implementations
47cc6ff 
Motivation: #84 (comment)

Note this PR doesn't describe the status of "feature parity"  b/c I'm not sure what status that would be for nmt-rs. Seems like we can add that later.
@rootulp rootulp mentioned this issue Aug 28, 2023
Merged
rootulp added a commit that referenced this issue Aug 29, 2023
@rootulp
docs: add a table for alternative implementations (#229)
e63158e 
Motivation:
#84 (comment)

Note this PR doesn't describe the status of "feature parity" b/c I'm not
sure what status that would be for nmt-rs. Seems like we can add that
later.
@rootulp
Copy link
Collaborator

rootulp commented Apr 17, 2024

Closing this as done via #229

@rootulp rootulp closed this as completed Apr 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@adlerjohn @rootulp @liamsi @evan-forbes