-
Notifications
You must be signed in to change notification settings - Fork 182
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
Support for Tip5 #187
Comments
@ameya-deshmukh, I like your idea and have taken the liberty to start hacking on it. The PR (#188) is not complete yet, as described in greater detail there. I'm not sure how much additional time I'll spend coding on the PR – feel free to take it as a draft for your own hacking. 😊 |
Wow, one of the authors of the paper himself! Glad you liked the idea. I'll be hacking on it independently and would love to compare code across implementations. Are there any specific optimizations etc that you'd like to see added which you may not have the time to implement? Would love to do it |
Most of the code I wrote for that PR is adapter code, i.e., getting the reference implementation of Tip5 to fit the requirements of the winterfell codebase. As far as we can tell, the reference implementation of Tip5 is in a pretty good spot right now.1 This does not mean you shouldn't go hunting for speedups if you want to – we welcome contributions to the code base! Additionally, it's quite possible that the adapter stuff I hacked together for the PR is not optimal. For example, I chose to go with chaining iterators instead of updating mutable slices, without knowing which of the two approaches is faster. Gut feel says that this won't be the bottleneck, but gut feeling is not a very good profiler. 😁 Footnotes
|
Got it! I actually want to discuss the project I'm working on with you, what's the best channel of communication for that? |
I messaged you on an instant messenger. 👍 |
@irakliyk hi, I am doing research on recursive STARKs and was wondering if support for arithmetization-oriented hash functions which are suitable for recursive verification is possible within Winterfell. I am especially looking at Tip5 and was curious to know the feasibility of adding support in Winterfell for the same. Would be more than happy to take this up as an independent task under your guidance. Thanks!
The text was updated successfully, but these errors were encountered: