Compute average costs more correctly in Multimeter
ClosedPublic
Actions

Authored by epriestley on May 2 2015, 11:45 AM.

Tags

None

Referenced Files

	F15532175: D12655.diff
	Wed, Apr 23, 2:38 PM

	F15524551: D12655.id30399.diff
	Mon, Apr 21, 8:48 AM

	F15522795: D12655.id30485.diff
	Sun, Apr 20, 9:27 PM

	F15516607: D12655.id.diff
	Fri, Apr 18, 7:58 PM

	F15512612: D12655.diff
	Thu, Apr 17, 12:00 PM

	F15489010: D12655.id30399.diff
	Fri, Apr 11, 5:14 AM

	F15464939: D12655.id30485.diff
	Wed, Apr 2, 1:09 PM

	F15439589: D12655.id30485.diff
	Wed, Mar 26, 8:09 AM

View All Files

Subscribers

epriestley

Details

Reviewers

btrahan

Maniphest Tasks

T6930: Build Multimeter, a "why is Phabricator slow?" toolset

Commits

Restricted Diffusion Commit
rPe6e0df6aff4c: Compute average costs more correctly in Multimeter

Summary

Ref T6930. We were dividing total cost by number of samples, but should more accurately divide it by the sum of the sample rates. This produces a more meaningful number.

Test Plan

Saw a more meaningful number in the UI.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 30399.May 2 2015, 11:45 AM

epriestley retitled this revision from to Compute average costs more correctly in Multimeter.

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added a reviewer: btrahan.

epriestley added a task: T6930: Build Multimeter, a "why is Phabricator slow?" toolset.

Herald added a subscriber: epriestley. · View Herald TranscriptMay 2 2015, 11:45 AM

btrahan accepted this revision.May 4 2015, 4:43 PM

btrahan edited edge metadata.

This revision is now accepted and ready to land.May 4 2015, 4:43 PM

Closed by commit rPe6e0df6aff4c: Compute average costs more correctly in Multimeter (authored by epriestley, committed by epriestley). · Explain WhyMay 4 2015, 5:04 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

multimeter/

controller/

MultimeterSampleController.php

7 lines

Diff 30485

View Options

src/applications/multimeter/controller/MultimeterSampleController.php

Compute average costs more correctly in MultimeterClosedPublicActions