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

Merge roofline modules #69

Closed
coleramos425 opened this issue Jan 17, 2023 · 0 comments
Closed

Merge roofline modules #69

coleramos425 opened this issue Jan 17, 2023 · 0 comments
Assignees
Labels
enhancement New feature or request Roofline Related to Omniperf's Roofline Analysis Standalone GUI Related to Omniperf's standalone GUI

Comments

@coleramos425
Copy link
Collaborator

At the moment, there are two areas where Omniperf computes Empirical Roofline data

  1. src/utils/plot_roofline.py
  2. src/omniperf_analyze/utils/roofline_calc.py

A lot of this code is duplicated so I propose we reorganize this into one module.

(1) is used in the standalone roofline capability (i.e. --roof-only) and generates a .pdf file with roofline leveraging matplotlib tools.
(2) returns critical data points for roofline to our Dash interface where a plot is sent to the html webpage

@coleramos425 coleramos425 added enhancement New feature or request Standalone GUI Related to Omniperf's standalone GUI Roofline Related to Omniperf's Roofline Analysis labels Jan 17, 2023
@coleramos425 coleramos425 self-assigned this Jan 17, 2023
@coleramos425 coleramos425 changed the title Merge GUI and Standalone roofline Merge GUI and Standalone roofline modules Jan 17, 2023
@coleramos425 coleramos425 changed the title Merge GUI and Standalone roofline modules Merge roofline modules Jan 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Roofline Related to Omniperf's Roofline Analysis Standalone GUI Related to Omniperf's standalone GUI
Projects
None yet
Development

When branches are created from issues, their pull requests are automatically linked.

1 participant