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

epic: diff-profiling between cairo zero and cairo for Kakarot Core EVM #812

Open
3 tasks
Eikix opened this issue Aug 7, 2024 · 1 comment
Open
3 tasks
Assignees
Labels
enhancement New feature or request epic Macro-task - Usually a collection of sub-issues

Comments

@Eikix
Copy link
Member

Eikix commented Aug 7, 2024

In an effort to highlight parts that could be optimized by the compiler, produce several benchmarks of core components and functions of the Kakarot implementation, and compare them to their CairoZero counterpart.

Profiling can be done using the cairo-profiler

Data Structures

  • Profile the Stack
  • Profile the Memory

Utils

  • Profile conversion from and to bytes

Needs more scoping.

@Eikix Eikix added the enhancement New feature or request label Aug 7, 2024
@Eikix
Copy link
Member Author

Eikix commented Aug 8, 2024

Problem reported by @enitrat:

  • Packages moved to sn-foundry can be profiled easily
  • Cannot move tests to sn-foundry without rewriting effort

Solution:

  • Profile when it is easy with sn-foundry to assert if the profiling is good -> assert whether we want sn-foundry for profiling or custom dev effort
  • Later, move tests

@ClementWalter ClementWalter added the epic Macro-task - Usually a collection of sub-issues label Oct 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request epic Macro-task - Usually a collection of sub-issues
Projects
Status: 🆕 Backlog
Development

No branches or pull requests

3 participants