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.

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

Update serialization to use serde #15

Closed
piotr-roslaniec opened this issue Nov 15, 2022 · 3 comments
Closed

Update serialization to use serde #15

piotr-roslaniec opened this issue Nov 15, 2022 · 3 comments
Assignees

Comments

@piotr-roslaniec
Copy link

piotr-roslaniec commented Nov 15, 2022

@piotr-roslaniec piotr-roslaniec added enhancement New feature or request blocked 🛑 labels Nov 15, 2022
@piotr-roslaniec piotr-roslaniec mentioned this issue Nov 15, 2022
2 tasks
@mimoo
Copy link

mimoo commented Feb 2, 2023

btw is this is helpful you can see how we use serde_with to make it work in kimchi: https://github.com/o1-labs/proof-systems/blob/master/kimchi/src/proof.rs

@piotr-roslaniec
Copy link
Author

@mimoo Thanks for chipping in. I played around with it a while back and I think this is going to be our preferred serialization method moving forward.

@arjunhassard arjunhassard moved this to To-do (Important) in v7.0.0 (TACo) Feb 13, 2023
@piotr-roslaniec piotr-roslaniec self-assigned this Feb 14, 2023
@arjunhassard arjunhassard moved this to To-do (Important) in v7.0.0 (TACo) Feb 15, 2023
@KPrasch KPrasch moved this from To-do (Important) to To-do (Urgent) in v7.0.0 (TACo) Feb 16, 2023
@piotr-roslaniec
Copy link
Author

Closed by #67

@github-project-automation github-project-automation bot moved this from To-do (Urgent) to Completed (Proof-of-Concept) in v7.0.0 (TACo) Apr 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Completed
Development

No branches or pull requests

3 participants