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

Add plotting tool #609

Merged
merged 1 commit into from
Jan 21, 2025
Merged

Conversation

joanaxcruz
Copy link
Contributor

Checklist

  • I have read the contributing guidelines.
  • I have considered portability of my change across platforms and architectures.
  • I have self-reviewed my code.
  • I have commented my code where necessary.
  • I have updated the documentation accordingly.
  • I have added tests that prove my fix is effective or that my feature works.

What is the purpose of this pull request?

  • Add internal tool to plot graphs with benchmark results.

What changes did you make?

Add a tool capable of processing and plotting the results produced by the new benchmark tool. Also support comparison with reference (assumes libm).
Produces 1 graph per (precision, accuracy, arch, compiler) showing performance for various variants (scalar, vector and scalable vector).

Relates to Issue #9.

@joanaxcruz joanaxcruz force-pushed the joanaxcruz/add_plotting_tool branch from 0d97ab0 to 49d55e9 Compare January 16, 2025 15:27
@blapie
Copy link
Collaborator

blapie commented Jan 16, 2025

Workflow is updated now, if you rebase your patch, the precommit tests should pass.

@joanaxcruz joanaxcruz force-pushed the joanaxcruz/add_plotting_tool branch from 49d55e9 to c19789f Compare January 17, 2025 12:00
Add a tool capable of processing and plotting the results produced
by the new benchmark tool. Also support comparison with reference
(assumes libm).
Produces 1 graph per (precision, accuracy, arch, compiler) showing
performance for various variants (scalar, vector and scalable vector).
@joanaxcruz joanaxcruz force-pushed the joanaxcruz/add_plotting_tool branch from c19789f to 394ca26 Compare January 20, 2025 15:42
@blapie blapie merged commit 34a8188 into shibatch:master Jan 21, 2025
36 checks passed
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.

2 participants