-
Notifications
You must be signed in to change notification settings - Fork 521
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
zeropool-2-snjax #316
Conversation
Thanks for the delivery. We will look into it as soon as possible. |
Hey @snjax, milestone deliveries should use the delivery template (you used our evaluation template) and go into the |
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. |
Use delivery template
@snjax I had a quick look at your delivery. I'm still waiting for the complete tutorial.
The compatibility is not yet demonstrated in this milestone. |
@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. |
@snjax I see your point that since your library is |
@mmagician, assessing both milestones in parallel would be much more convenient for us. |
@snjax just checking in, when do you think you'll be able to deliver M3? |
Could you provide an update, @snjax? Does it make sense to keep this PR open if you want to finish and submit M3 alongside? |
Hey @snjax, we are still waiting for an update from your side. How is it looking, any news since your last comment? |
Hi, @semuelle . We have deployed the testnet at https://substrate.testnet.console.v2.zeropool.network/ . Currently, we are finalizing the documentation of the project. |
@semuelle We have updated the milestone delivery. So, please, have a look and we are waiting for your feedback. |
@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. |
ping @snjax |
@semuelle We received your comments and working on documentation now. You will get the update in August. |
* xtoken * update description * update * update payment denomination, and milestone 1 deliverables * file name change, milestone cost updated Co-authored-by: Bryan Chen <[email protected]>
@snjax friendly reminder - when do you think you'll be able to get back to us? |
@snjax I just wanted to check in regarding the current status. Would you still like to finish the delivery? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(see above)
Thanks for the ping. alxs has left and @Noc2 is currently ooo, but someone from our team is going to review it soon. |
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/ |
There was a problem hiding this 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
``
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. |
There was a problem hiding this 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.
ping @snjax |
@semuelle We have published the overview&tutorial how to use ZeroPool privacy engine on substrate tech stack. |
1 similar comment
@semuelle We have published the overview&tutorial how to use ZeroPool privacy engine on substrate tech stack. |
@semuelle Thanks for accepting all milestones. |
@snjax, someone submitted an invoice through the invoice form the same day this PR was submitted.
|
@semuelle the invoice for Milestones 1 and 2 is sent. |
Hi @snjax, Could you please add your complete address under your name? Many thanks, |
@fededubbi, I have updated the invoice with the information you requested. |
hi @snjax we transferred the payment today |
@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. |
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. |
@Noc2 thank you for your reply. I will send email. |
Milestone Delivery Checklist
Link to the application pull request: w3f/Grants-Program#208