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.

Sign up for GitHub

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

Jump to bottom

Add script for results plotting #31

Merged
merged 3 commits into from
Oct 19, 2023
Merged

Add script for results plotting #31

merged 3 commits into from
Oct 19, 2023

Conversation

compor
Copy link
Contributor

@compor compor commented Oct 17, 2023

This PR:

  • Adds a Python script to plot collected results from Verilator traces
  • Python requirements.txt file for above
  • Rudimentary documentation

@compor compor added the enhancement New feature or request label Oct 17, 2023
@compor compor self-assigned this Oct 17, 2023
This was referenced Oct 17, 2023
Closed
Merged
JosseVanDelm
JosseVanDelm approved these changes Oct 18, 2023
View reviewed changes
Copy link
Contributor

@JosseVanDelm JosseVanDelm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is probably the prettiest plot script i've ever used 👍

scripts/plotting/plot_barchart.py Outdated Show resolved Hide resolved
scripts/plotting/config/cycles/barchart.json Show resolved Hide resolved
@compor compor force-pushed the christos/results-plotting branch from 8b6001f to 75256a5 Compare October 19, 2023 08:11
@compor compor merged commit 5e13af9 into main Oct 19, 2023
1 check passed
@compor compor deleted the christos/results-plotting branch October 19, 2023 08:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JosseVanDelm JosseVanDelm JosseVanDelm approved these changes

@superlopuh superlopuh superlopuh approved these changes

@nazavode nazavode Awaiting requested review from nazavode

Assignees

@compor compor

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@compor @superlopuh @JosseVanDelm