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

improve Triton profile #191

Merged
merged 3 commits into from
Apr 20, 2023
Merged

improve Triton profile #191

merged 3 commits into from
Apr 20, 2023

Conversation

jan-ferdinand
Copy link
Member

  • introduce more weights for relative runtime costs
  • print relative time for all tasks and sub-tasks
  • add profiling of categories

The usual prof_start!(…) macro can be used as before. The same macro now accepts an additional argument, a category: calling prof_start!(profiler, "task name", "category") will accumulate the times of all tasks with the same category and print additional reports for the categories. For example:

### Categories
hash    84.60ms  8.88%
AIR     80.87ms  8.48%
cc     485.42ms 50.93%
LDE    165.70ms 17.38%

@jan-ferdinand jan-ferdinand merged commit 7edd1a2 into master Apr 20, 2023
@jan-ferdinand jan-ferdinand deleted the categories_in_profiler branch April 20, 2023 09:52
jan-ferdinand added a commit that referenced this pull request Apr 24, 2023
- do DEEP-ALI instead of plain ALI in the zk-STARK (#190) (9606441)
- add convenience functions for using Triton VM (0dab32a) (dda05e4)
- improve Triton profiler (#191) (7edd1a2)
  - add profiling of categories
  - introduce more weights for relative runtime costs
  - print relative time for all tasks and sub-tasks
- make method `debug` more powerful (ab49df7)
  - add optional argument: initial state
  - add optional argument: number of execution steps
- add construction of AET (witness generation) to profiling (c6e7b1e)
- use `cfg(debug_assertions)`, not environment variable (b0052f1)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant