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

Will this work be merged to upstream plonky2 repo? #2

Open
vivekvpandya opened this issue Jan 9, 2024 · 10 comments
Open

Will this work be merged to upstream plonky2 repo? #2

vivekvpandya opened this issue Jan 9, 2024 · 10 comments

Comments

@vivekvpandya
Copy link

Thanks for open sourcing this work! Are there any future plans to merge upstream repo?

@liangping
Copy link
Contributor

We are happy to do that, but I don't think the upstream repo will accept.

@vivekvpandya
Copy link
Author

Why not ? I think code needs little bit of cleanup and perhaps squash commits which don't have important commit message. Also instead of my_prove() it would be better to use prove_with_cuda() or some other better naming convention.

@matthiasgoergens
Copy link

matthiasgoergens commented Jan 10, 2024

We are happy to do that, but I don't think the upstream repo will accept.

I work with @vivekvpandya. We can try to help you work with upstream to get your code in, if you are interested.

Btw, would you be willing to change the license to the one upstream uses?

Why do you think upstream wouldn't want your code?

@liangping
Copy link
Contributor

Which upstream repo do you mean?
We was trying to add a link on the Readme, 0xPolygonZero/plonky2#1441 (comment), but it's been rejected.

@vivekvpandya
Copy link
Author

Which upstream repo do you mean? We was trying to add a link on the Readme, 0xPolygonZero/plonky2#1441 (comment), but it's been rejected.

The same where your PR was created.
But I am curious why you want to add a link to README? why not just merge commits from this repo (after polishing) to plonky2 upstream repo?

@matthiasgoergens
Copy link

matthiasgoergens commented Jan 15, 2024

Yes, we had in mind helping you to contribute your code directly to the main plonky2 repository. Not to add a link in a readme somewhere.

We've contributed to plonk2 before, and found them fairly welcoming.

@liangping
Copy link
Contributor

But I am curious why you want to add a link to README? why not just merge commits from this repo (after polishing) to plonky2 upstream repo?

We can polish code for the merge. This project is actually supported by Side Labs. Do you mind if we add "Side Labs" somewhere for its early supporting.

@matthiasgoergens
Copy link

I don't mind, but we'd need to check with plonky2 folks what they think.

@mertwole
Copy link

mertwole commented May 6, 2024

@liangping are you guys still planning merging it upstream?

@matthiasgoergens
Copy link

Yes, but haven't worked on it much recently. We had some people join and leave.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants