Closed Bug 1372714 Opened 7 years ago Closed 7 years ago

[AWSY] Add an option to output parse_about_memory.py results in mebibytes

Categories

(Testing :: AWSY, enhancement)

Version 3
enhancement
Not set
normal

Tracking

(firefox56 fixed)

RESOLVED FIXED
mozilla56
Tracking Status
firefox56 --- fixed

People

(Reporter: mccr8, Assigned: mccr8)

Details

Attachments

(2 files)

about:memory uses mebibytes, not bytes, which makes comparing the contents of about:memory to parse_about_memory.py very tedious.
This makes it easier to compare results to about:memory.
Attachment #8877305 - Flags: review?(erahm)
(In reply to Andrew McCreight [:mccr8] from comment #1)
> Created attachment 8877305 [details] [diff] [review]
> Add an option to output parse_about_memory.py results in mebibytes.
> 
> This makes it easier to compare results to about:memory.

FWIW I just check 'verbose' when loading the report.
Comment on attachment 8877305 [details] [diff] [review]
Add an option to output parse_about_memory.py results in mebibytes.

Review of attachment 8877305 [details] [diff] [review]:
-----------------------------------------------------------------

Not super stoked on '--mebi' but meh. |ls| has '-h' for 'human readable'

::: testing/awsy/awsy/parse_about_memory.py
@@ +156,4 @@
>      for (k, v) in sorted_totals:
>          if v:
> +            if args.mebi:
> +                print "{0:.2f}".format(v / bytes_per_mebibyte),

Maybe include MiB (or just MB like about:memory) at the end of the value?
Attachment #8877305 - Flags: review?(erahm) → review+
(In reply to Eric Rahm [:erahm] (please no mozreview requests) from comment #3)
> Maybe include MiB (or just MB like about:memory) at the end of the value?

That's a good idea. I'll also add "bytes" as the units in the default case.
This makes it easier to compare results to about:memory.
Attachment #8877691 - Flags: review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/70e68a66dd9b8b46a47324d5bffe6a27cf01f337
Bug 1372714 - Add an option to output parse_about_memory.py results in mebibytes. r=erahm
https://hg.mozilla.org/mozilla-central/rev/70e68a66dd9b
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
Component: General → AWSY
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: