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

zeropool-2-snjax #316

Merged
merged 6 commits into from
Jun 22, 2023
Merged

zeropool-2-snjax #316

merged 6 commits into from
Jun 22, 2023

Conversation

snjax
Copy link
Contributor

@snjax snjax commented Dec 3, 2021

Milestone Delivery Checklist

Link to the application pull request: w3f/Grants-Program#208

@snjax snjax mentioned this pull request Dec 3, 2021
6 tasks
@Noc2
Copy link
Collaborator

Noc2 commented Dec 3, 2021

Thanks for the delivery. We will look into it as soon as possible.

@semuelle
Copy link
Member

semuelle commented Dec 3, 2021

Hey @snjax, milestone deliveries should use the delivery template (you used our evaluation template) and go into the deliveries folder. Could you fix that, please?

@mmagician mmagician self-assigned this Dec 6, 2021
@mmagician
Copy link
Contributor

Let me know once you've got the tutorial ready, so I know where to start! I guess a lot has changed since Milestone 1.

@mmagician
Copy link
Contributor

@snjax I had a quick look at your delivery. I'm still waiting for the complete tutorial.
In addition, there is no deliverable showing how your libraries integrate with substrate. According to your initial grant application:

We will create a library for proving and verifying private transactions, compatible with the substrate pallet

The compatibility is not yet demonstrated in this milestone.

@snjax
Copy link
Contributor Author

snjax commented Jan 5, 2022

@mmagician, our current implementation is compatible with substrate, we use no_std rust for parts of fawkes-crypto, used to implement the substrate module (like final field math).

I could show you the substrate module as part of milestone 3 this month.

Also, currently, we have a live EVM version on kovan with custom, not EVM-specific ABI.

We are working on a substrate module, supporting this ABI.

@mmagician
Copy link
Contributor

@snjax I see your point that since your library is no_std compatible, the integration with substrate would be straightforward. However, since that integration is not explicitly demonstrated in M2, it might be easier to wait for M3 as you suggest and assess both milestones in parallel. What do you think?

@snjax
Copy link
Contributor Author

snjax commented Jan 11, 2022

@mmagician, assessing both milestones in parallel would be much more convenient for us.

@alxs
Copy link
Contributor

alxs commented Feb 1, 2022

@snjax just checking in, when do you think you'll be able to deliver M3?

@snjax
Copy link
Contributor Author

snjax commented Feb 6, 2022

@alxs the work is in progress. We were focused on more common non-substrate-specific points.

Here is current WIP repository for substrate. It seems this month we are able to show you the final result (like our evm kovan testnet).

@semuelle
Copy link
Member

semuelle commented Mar 8, 2022

It seems this month we are able to show you the final result (like our evm kovan testnet).

Could you provide an update, @snjax? Does it make sense to keep this PR open if you want to finish and submit M3 alongside?

@snjax
Copy link
Contributor Author

snjax commented Mar 13, 2022

@semuelle We have built it. Here is the screencast of deposit, transfer, and withdrawal implemented on substrate tech stack and ZeroPool circuit.

Now we are working on documentation and deployment.

I will push an update into this PR soon.

@semuelle
Copy link
Member

semuelle commented May 4, 2022

Hey @snjax, we are still waiting for an update from your side. How is it looking, any news since your last comment?

@snjax
Copy link
Contributor Author

snjax commented May 8, 2022

Hi, @semuelle . We have deployed the testnet at https://substrate.testnet.console.v2.zeropool.network/ . Currently, we are finalizing the documentation of the project.

@snjax
Copy link
Contributor Author

snjax commented May 23, 2022

@semuelle We have updated the milestone delivery. So, please, have a look and we are waiting for your feedback.

@semuelle semuelle requested a review from mmagician May 25, 2022 07:26
@mmagician
Copy link
Contributor

@snjax Apologies for taking so long. I've created a first version of the evaluation in this PR, where you'll find the comments I have so far.

@semuelle
Copy link
Member

ping @snjax

@snjax
Copy link
Contributor Author

snjax commented Jul 25, 2022

@semuelle We received your comments and working on documentation now. You will get the update in August.

failfmi pushed a commit to LimeChain/Grant-Milestone-Delivery that referenced this pull request Sep 26, 2022
* xtoken

* update description

* update

* update payment denomination, and milestone 1 deliverables

* file name change, milestone cost updated

Co-authored-by: Bryan Chen <[email protected]>
@alxs
Copy link
Contributor

alxs commented Oct 18, 2022

@snjax friendly reminder - when do you think you'll be able to get back to us?

@alxs alxs requested review from mmagician and removed request for mmagician November 21, 2022 15:39
@Noc2 Noc2 self-assigned this Dec 30, 2022
@github-actions github-actions bot added the stale label Jan 14, 2023
@github-actions github-actions bot added to close and removed stale labels Jan 29, 2023
@Noc2
Copy link
Collaborator

Noc2 commented Feb 9, 2023

@snjax I just wanted to check in regarding the current status. Would you still like to finish the delivery?

Copy link
Collaborator

@Noc2 Noc2 left a comment

Choose a reason for hiding this comment

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

(see above)

@snjax
Copy link
Contributor Author

snjax commented Feb 17, 2023

@Noc2 More substrate-specific documentation is added here.

We are currently working on switching to Plonk. But Groth16 implementation is stable.

Sure, I would like to finish the delivery.

@snjax
Copy link
Contributor Author

snjax commented Apr 10, 2023

@Noc2 @alxs Have you viewed the documentation? Have you got any comments?

We added Plonk support to our prover.

@semuelle
Copy link
Member

Have you viewed the documentation? Have you got any comments?

Thanks for the ping. alxs has left and @Noc2 is currently ooo, but someone from our team is going to review it soon.

@snjax
Copy link
Contributor Author

snjax commented Apr 11, 2023

@semuelle

the current video of using the frontend of our pallet is available here https://www.youtube.com/watch?v=YRJadaIrBJs

The web interface is deployed at https://substrate.testnet.console.v2.zeropool.network/

Copy link
Member

@semuelle semuelle left a comment

Choose a reason for hiding this comment

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

Hi @snjax. Sorry for the long pause. I have now started reviewing your delivery.
When I run the ZeroPool node through docker (docker-compose -f docker-compose.full.yml up), the relayer stops with the following error message. Do you know what might be the reason?

relayer_polkadot    | thread '<unnamed>' panicked at 'called `Result::unwrap()` on an `Err` value: Os { code: 2, kind: NotFound, message: "No such file or directory" }', src/params.rs:30:36
relayer_polkadot    | note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
relayer_polkadot    | /app/zp-relayer/pool.js:32
relayer_polkadot    |         this.treeParams = libzeropool_rs_node_1.Params.fromFile('./params/tree_params.bin');
relayer_polkadot    |                                                        ^
relayer_polkadot    | 
relayer_polkadot    | Error: internal error in Neon module: called `Result::unwrap()` on an `Err` value: Os { code: 2, kind: NotFound, message: "No such file or directory" }
relayer_polkadot    |     at new Pool (/app/zp-relayer/pool.js:32:56)
relayer_polkadot    |     at Object.<anonymous> (/app/zp-relayer/pool.js:196:16)
relayer_polkadot    |     at Module._compile (node:internal/modules/cjs/loader:1103:14)
relayer_polkadot    |     at Object.Module._extensions..js (node:internal/modules/cjs/loader:1155:10)
relayer_polkadot    |     at Module.load (node:internal/modules/cjs/loader:981:32)
relayer_polkadot    |     at Function.Module._load (node:internal/modules/cjs/loader:822:12)
relayer_polkadot    |     at Module.require (node:internal/modules/cjs/loader:1005:19)
relayer_polkadot    |     at require (node:internal/modules/cjs/helpers:102:18)
relayer_polkadot    |     at Object.<anonymous> (/app/zp-relayer/utils/redisFields.js:15:16)
relayer_polkadot    |     at Module._compile (node:internal/modules/cjs/loader:1103:14)
relayer_polkadot    | error Command failed with exit code 1.
relayer_polkadot    | info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
relayer_polkadot exited with code 1
``

@snjax
Copy link
Contributor Author

snjax commented Jun 11, 2023

@semuelle

This is because you run it with no parameters like proving and verifying keys.

We added the description of parameters into the README of zeropoolnetwork/zeropool-substrate.

Also, you can get it from here.

Copy link
Member

@semuelle semuelle left a comment

Choose a reason for hiding this comment

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

We added the description of parameters into the README of zeropoolnetwork/zeropool-substrate.

The path to the params folder in the docker-compose file wasn't quite right, but I got it to work.

Unless I'm overlooking something, the article is still missing for both milestones. Could you still do that and update the delivery doc? Since you are submitting both milestones at once, one would be enough.

@semuelle
Copy link
Member

ping @snjax

@snjax
Copy link
Contributor Author

snjax commented Jun 20, 2023

@semuelle We have published the overview&tutorial how to use ZeroPool privacy engine on substrate tech stack.

https://medium.com/zeropool/zeropool-implementing-privacy-preserving-transactions-with-zksnarks-and-substrate-1bd060505166

1 similar comment
@snjax
Copy link
Contributor Author

snjax commented Jun 20, 2023

@semuelle We have published the overview&tutorial how to use ZeroPool privacy engine on substrate tech stack.

https://medium.com/zeropool/zeropool-implementing-privacy-preserving-transactions-with-zksnarks-and-substrate-1bd060505166

@semuelle
Copy link
Member

Thanks, @snjax. I am happy to report that both or your milestones are hereby finally accepted. See evaluation notes here and here.

You will probably have to resend your invoices, as they require a somewhat recent date on them. :)

@semuelle semuelle merged commit a5f918d into w3f:master Jun 22, 2023
@snjax
Copy link
Contributor Author

snjax commented Jun 22, 2023

@semuelle Thanks for accepting all milestones.
It's not very clear to me, what you mean.
If you write about the accessibility of 0xb6F9F891497C0B72a8A817f3D3E3C721fca6f9CC in the submission there are no changes from my side and I have not sent any other invoices.

@semuelle
Copy link
Member

@snjax, someone submitted an invoice through the invoice form the same day this PR was submitted.

  • The invoice is only for M2, we don't have one for M1.
  • The invoice is from Dec 2021. Could you send one with a more recent date?
  • Also, the payment address doesn't match the application. Both addresses must match, so you either have to change it in the invoice (easiest) or submit an amendment to the contract.

@snjax
Copy link
Contributor Author

snjax commented Jun 23, 2023

@semuelle the invoice for Milestones 1 and 2 is sent.

@fededubbi
Copy link

Hi @snjax,

Could you please add your complete address under your name?

Many thanks,
Federica

@snjax
Copy link
Contributor Author

snjax commented Jun 23, 2023

@fededubbi, I have updated the invoice with the information you requested.

@RouvenP
Copy link

RouvenP commented Jun 29, 2023

hi @snjax we transferred the payment today

@snjax
Copy link
Contributor Author

snjax commented Jul 17, 2023

@semuelle Thank you for the experience of cooperation with Web3 Foundation team. It was amazing. Btw, are you at EthCC? It would be nice to meet up.

@Noc2
Copy link
Collaborator

Noc2 commented Jul 18, 2023

Hi @snjax, @semuelle is currently OOO and not at EthCC, but @bhargavbh from the Web3 Foundation might be there. Other than that, feel free to reach out via email: [email protected] and we can potentially schedule a call.

@snjax
Copy link
Contributor Author

snjax commented Jul 18, 2023

@Noc2 thank you for your reply. I will send email.

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

Successfully merging this pull request may close these issues.

7 participants