Skip to content

Conversation

harp-intel
Copy link
Contributor

This pull request enhances the cmd/metrics/resources/base.html file by improving the visualization and clarity of the TMA (Top-down Microarchitecture Analysis) sunburst chart and related line charts. The main changes include introducing a consistent color scheme for TMA categories and their children, refactoring the sunburst chart configuration for maintainability, adding explicit series names to line charts, and making minor improvements to metric descriptions.

Visualization and Theming Improvements:

  • Introduced a tmaColors object to define consistent color families for TMA categories and their children, and updated the sunburst chart to use these colors for better visual distinction and clarity.
  • Refactored the sunburst chart configuration into a tmasunburst variable, improving maintainability and making it easier to update chart properties or data in the future. The chart rendering now uses this variable.

Chart Configuration Enhancements:

  • Updated the legend configuration for charts to improve readability: legends are now centered at the bottom, oriented horizontally, and series toggling is disabled for clarity.
  • Added explicit name properties to all line chart series (e.g., "CPU Utilization %", "CPI", "CPU Frequency (GHz)", "Remote DRAM Reads %", "Package Power (Watts)", "DRAM Power (Watts)") for clearer chart legends and tooltips. [1] [2] [3] [4] [5] [6]
  • Applied the new TMA color scheme to the TMA level-1 line chart for consistency across visualizations.

Documentation and Description Updates:

  • Clarified the description of "Remote DRAM reads %" to specify "remote DRAM" instead of "another socket's DRAM."
  • Updated the description of "Cache MPI" to specify "L1d" instead of just "L1," making the metric explanation more precise.

@harp-intel harp-intel merged commit d8d67a5 into main Sep 18, 2025
3 checks passed
@harp-intel harp-intel deleted the htmlups branch September 18, 2025 00:49
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