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

zkwasm rollups transfer milestone 1 #962

Conversation

ashWhiteHat
Copy link
Contributor

@ashWhiteHat ashWhiteHat commented Aug 7, 2023

Milestone Delivery Checklist

  • The milestone-delivery-template.md has been copied and updated.
  • The invoice form 📝 has been filled out for this milestone.
  • This pull request is being made by the same account as the accepted application.
  • I have disclosed any and all sources of reused code in the submitted repositories and have done my due diligence to meet its license requirements.
  • In case of acceptance, the payment will be transferred to the BTC/ETH/fiat account provided in the application.
  • The delivery is according to the Guidelines for Milestone Deliverables.

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

@dsm-w3f dsm-w3f self-assigned this Aug 8, 2023
@dsm-w3f
Copy link
Contributor

dsm-w3f commented Aug 8, 2023

@ashWhiteHat thank you for the milestone delivery. Please see the evaluation document and provide proper answers and fixes. Let me know when I can continue this evaluation.

@ashWhiteHat
Copy link
Contributor Author

Could you explain how you got the values for deliverable 2 and if I can reproduce it?

Sorry for lack of explanation.

Jubjub Curve

https://github.com/KogarashiNetwork/Kogarashi/tree/master/primitive/jubjub

Unit Test

cd primitive/jubjub
cargo test

https://github.com/KogarashiNetwork/Kogarashi/tree/master/primitive/jubjub#test

Benchmark

https://github.com/KogarashiNetwork/Kogarashi/tree/master/benches

Jubjub curve points arithmetic benchmark.
https://github.com/KogarashiNetwork/Kogarashi/blob/master/Cargo.toml#L99
Jubjub field arithmetic benchmark.
https://github.com/KogarashiNetwork/Kogarashi/blob/master/Cargo.toml#L91

The repository has some broken links

Thank you for letting me know.
It seems relative path doesn't work.
I fixed all links.

or is it the all the content of the site?

I would say all the content of the site.

However, I think developers still don't know why we should use RedDSA with above contents.
I am going to publish library using RedDSA and article describing it next week.

I had some problems with the fund command.

Umm...
Let me check.
It might be problem of command option when we run node.

The pallets could have better coverage.

Exactly.
I am going to publish library so I will improve coverage then.

@ashWhiteHat
Copy link
Contributor Author

I had some problems with the fund command.

I fixed this error.
I would appreciate it if you could check.
Thank you!

@dsm-w3f
Copy link
Contributor

dsm-w3f commented Aug 9, 2023

@ashWhiteHat thank you for the improvements and fixes. I tried again and I was able to run the tutorial. See details in the evaluation document. I asked a cryptography specialist from our research team to check the cryptography part. He should give us a feedback in some days.

@ashWhiteHat
Copy link
Contributor Author

Hi @dsm-w3f
Thank you for the review.

Could you specify what machine configuration was used for running the benchmark?

We do benchmark on Github Actions.
https://docs.github.com/en/actions/using-github-hosted-runners/about-github-hosted-runners#supported-runners-and-hardware-resources

Spec
2 core
7GB RAM
14GB SSD

I asked a cryptography specialist from our research team to check the cryptography part

Thank you!

@dsm-w3f
Copy link
Contributor

dsm-w3f commented Aug 14, 2023

@ashWhiteHat just to let you know that the research team will need more time to complete this review. Probably they will two weeks to review. I'll keep you posted about the progress.

@ashWhiteHat
Copy link
Contributor Author

Hi @dsm-w3f
Copy that.
Please take your time 👍

@github-actions github-actions bot added the stale label Aug 29, 2023
@ashWhiteHat
Copy link
Contributor Author

Hi @dsm-w3f
Is everything okay?

@dsm-w3f
Copy link
Contributor

dsm-w3f commented Sep 13, 2023

@ashWhiteHat my apologies for the long time waiting for a review of this milestone. Our research team is in high demand. I'll wait for them to take a look until the end of this week, otherwise, I'll approve this milestone and the zk part will be checked later.

@dsm-w3f
Copy link
Contributor

dsm-w3f commented Sep 15, 2023

@ashWhiteHat my apologies again for making you wait so long to finish this evaluation. We will evaluate the zk part in the next milestone. This milestone is accepted. I'll forward your invoice internally and the payment should take place within two weeks. Great job!

@dsm-w3f dsm-w3f merged commit 4a7b5fe into w3f:master Sep 15, 2023
6 checks passed
@github-actions
Copy link

Congratulations on completing the first milestone of this grant! As part of the Grants Program, we want to help grant recipients acknowledge their grants publicly. To that end, we’ve created a badge for projects that successfully deliver their first milestone. Please use the badge only in reference to the work that has been completed as part of this grant, so please do not display it on your team or project's homepage unless accompanied by a short description of the grant. Furthermore, you're now welcome to announce the grant publicly. Please remember to observe the foundation’s guidelines in doing so. If you haven't already, reach out to [email protected] for feedback on your announcement and cross-promotion.

Thank you for your contribution, and good luck! If you have any remaining milestone, let us know if you encounter any delays by leaving a comment on the application PR or submitting an amendment.

@ashWhiteHat
Copy link
Contributor Author

@dsm-w3f
Thank you for the review!

@dsm-w3f
Copy link
Contributor

dsm-w3f commented Sep 20, 2023

@ashWhiteHat did you provide an invoice for this milestone? I was not able to find it. Could you please resend it using our invoice form?

@ashWhiteHat
Copy link
Contributor Author

@dsm-w3f
I sent you the invoice.
I would appreciate it if you could confirm.
Thank you!

@semuelle
Copy link
Member

@ashWhiteHat confirmed

@ashWhiteHat
Copy link
Contributor Author

Hello there
Did you transfer the fee?
I seems we still haven't received it.

I would appreciate it if you could confirm.
Thank you.

@fededubbi
Copy link

Hi @ashWhiteHat,

We noticed that you sent in the past an invoice with the same number.
Could you please update and resend?

Many thanks,
Federica

@ashWhiteHat
Copy link
Contributor Author

Hi @fededubbi
Thank you for letting me know.
I submitted the invoice.

I would appreciate it if you could confirm.
Thank you.

@fededubbi
Copy link

Hi @ashWhiteHat,

That's correct.

Many thanks,
Federica

@ashWhiteHat
Copy link
Contributor Author

@fededubbi
Thank you for the confirmation.
When can we receive it?

@fededubbi
Copy link

Hi @ashWhiteHat,

The invoice will be paid on Friday this week.

Many thanks,
Federica

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

Successfully merging this pull request may close these issues.

4 participants