Skip to content

Add the kWh unit for tooltips of power tracks.#4622

Merged
julienw merged 1 commit into
firefox-devtools:mainfrom
fqueze:kwh-unit
May 16, 2023
Merged

Add the kWh unit for tooltips of power tracks.#4622
julienw merged 1 commit into
firefox-devtools:mainfrom
fqueze:kwh-unit

Conversation

@fqueze

@fqueze fqueze commented May 16, 2023

Copy link
Copy Markdown
Contributor

When power profiling something that uses more power than a browser on a laptop (could be an application using all cores for multiple hours on a powerful server, or in my most recent case kitchen appliances or washing machines), having a Wh as the largest unit means we display numbers in the thousands in tooltips of the power track. Adding the 'kilo' version of the existing units is straightforward.

Example profile: https://share.firefox.dev/430KJTk (this is using the washing machine, then the dryer)

@fqueze fqueze requested a review from canova May 16, 2023 06:01
@fqueze fqueze requested a review from a team as a code owner May 16, 2023 06:01
@codecov

codecov Bot commented May 16, 2023

Copy link
Copy Markdown

Codecov Report

Patch coverage: 40.00% and project coverage change: -0.01 ⚠️

Comparison is base (054e70b) 88.63% compared to head (319b98b) 88.62%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4622      +/-   ##
==========================================
- Coverage   88.63%   88.62%   -0.01%     
==========================================
  Files         294      294              
  Lines       26086    26090       +4     
  Branches     7030     7031       +1     
==========================================
+ Hits        23121    23122       +1     
- Misses       2759     2762       +3     
  Partials      206      206              
Impacted Files Coverage Δ
src/components/tooltip/TrackPower.js 94.33% <40.00%> (-5.67%) ⬇️

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@julienw julienw left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

looks good to me!

@julienw julienw merged commit 6d11f8b into firefox-devtools:main May 16, 2023
@canova canova mentioned this pull request May 30, 2023
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.

3 participants