Skip to content

[evaluation] Add performance/resource usage evaluation script#779

Draft
MaxWipfli wants to merge 8 commits intomainfrom
mwipfli/eval-script
Draft

[evaluation] Add performance/resource usage evaluation script#779
MaxWipfli wants to merge 8 commits intomainfrom
mwipfli/eval-script

Conversation

@MaxWipfli
Copy link
Collaborator

This adds a pair of scripts to perform a full evaluation of Dynamatic on a fixed number of kernels from the integration tests. Compared to the GTest-based integration test flow, this also runs synthesis, and it is easier to control and change the *.dyn script used. Also, there is a script to extract key figures (clock cycles, resource usage, critical path) into a CSV file for further processing.

Builds Dynamatic via ninja, then runs the embedded .dyn script for each
kernel in parallel.
Archives each kernel's output directory after evaluation finishes.
Check for FATAL in stdout, and "C and VHDL outputs match" in
sim/report.txt.
There are some issues with the default (4 ns) clock period. For example,
histogram takes almost twice as many cycles for some reason when
reducing the period from 5 ns to 4 ns.
@MaxWipfli MaxWipfli force-pushed the mwipfli/eval-script branch from 1ef2c85 to 4611497 Compare March 10, 2026 10:46
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