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

Benchmark transfer contract performance #361

Closed
moCello opened this issue Oct 11, 2021 · 0 comments · Fixed by #362 or #421
Closed

Benchmark transfer contract performance #361

moCello opened this issue Oct 11, 2021 · 0 comments · Fixed by #362 or #421
Assignees

Comments

@moCello
Copy link
Member

moCello commented Oct 11, 2021

Describe what you want implemented
Add a benchmark for the performance of the following transfer contracts:

  • send_to_contract_transparent()
  • send_to_contract_obfuscated()
  • withdraw_from_transparent()
  • withdraw_from_obfuscated()

Describe "Why" this is needed
With of the future integration of rusk-vm v2 it is important to know how the performance of the transfer contracts changes.

Describe alternatives you've considered
A benchmarking is the only sensible solution for this issue

Additional context
A benchmark for the gas consumption of the same transfer contracts will be added as well.

Disclaimer
For now we are actually not benchmarking the transfer contracts themselves but some wrapper functions. A probable performance change between Rusk v1 and Rusk v2 will be observable but it is important to note that the outcome of the benchmark is not the performance of the contract itself.

@moCello moCello self-assigned this Oct 11, 2021
@moCello moCello linked a pull request Oct 14, 2021 that will close this issue
@ZER0 ZER0 assigned ureeves and unassigned moCello Oct 19, 2021
@ureeves ureeves reopened this Jan 12, 2022
@HDauven HDauven self-assigned this Sep 12, 2023
ureeves added a commit that referenced this issue Nov 23, 2023
ureeves added a commit that referenced this issue Nov 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants