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

feat: Incorporate downloading Sphinx parameters using AWS cli #63

Merged
merged 3 commits into from
Jun 20, 2024

Conversation

storojs72
Copy link
Member

This PR introduces specific version of Sphinx parameters downloaders using aws utility. This is necessary because our AWS bucket is not shared publicly for security reasons and can be accessed only by us currently.

Please refer to AWS documentation for installing the aws and contact @samuelburnham for getting credentials for our AWS bucket (s3://sphinx-plonk-params)

@storojs72 storojs72 force-pushed the generate-contracts branch from a9c5b41 to 092e842 Compare June 20, 2024 13:21
Copy link
Member

@samuelburnham samuelburnham left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved in spirit, though the CI error seems to indicate a need to check whether the artifacts are installed already

@storojs72
Copy link
Member Author

Approved in spirit, though the CI error seems to indicate a need to check whether the artifacts are installed already

Yes. I'm currently re-building new parameters after merging #49, this is a time-consuming process. Once it finishes I can upload new parameters and test_verifier_export test should pass

Copy link
Contributor

@wwared wwared left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm ok with this change in principle too, as long as later on we incorporate the artifacts with the bundled releases somehow and only download them if they're not present

@storojs72 storojs72 force-pushed the generate-contracts branch 5 times, most recently from bd4fceb to 9e2c6ad Compare June 20, 2024 17:06
@storojs72 storojs72 force-pushed the generate-contracts branch from 9e2c6ad to 89ccc1c Compare June 20, 2024 17:08
@storojs72 storojs72 force-pushed the generate-contracts branch from 7cd967f to 95f036b Compare June 20, 2024 19:09
@storojs72 storojs72 merged commit 2019a00 into plonk Jun 20, 2024
5 of 6 checks passed
@storojs72 storojs72 deleted the generate-contracts branch June 20, 2024 19:40
huitseeker pushed a commit that referenced this pull request Jun 24, 2024
* feat: Incorporate downloading using AWS cli

* chore: SP1 -> Sphinx renaming in Solidity assets

* ci: Add S3 credentials (#64)

---------

Co-authored-by: Samuel Burnham <45365069+samuelburnham@users.noreply.github.com>
wwared pushed a commit that referenced this pull request Jun 24, 2024
* feat: Incorporate downloading using AWS cli

* chore: SP1 -> Sphinx renaming in Solidity assets

* ci: Add S3 credentials (#64)

---------

Co-authored-by: Samuel Burnham <45365069+samuelburnham@users.noreply.github.com>
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

Successfully merging this pull request may close these issues.

3 participants