Skip to content

Commit

Permalink
Bug 1164898 - A few more fixups to display of confidence warning
Browse files Browse the repository at this point in the history
* Don't display it if we don't have any runs at all for either original or new
* Some capitalization tweaks
  • Loading branch information
wlach committed Jul 16, 2015
1 parent 9222612 commit f105542
Showing 1 changed file with 8 additions and 7 deletions.
15 changes: 8 additions & 7 deletions ui/partials/perf/compareconfidence.html
@@ -1,8 +1,9 @@
<span ng-if="result.confidenceText && result.confidence">
{{result.confidence|displayPrecision}} (<abbr title="Result of running t-test on base versus new result distribution: higher values indicate greater confidence that the indicated performance difference is statistically meaningful">{{result.confidenceText}}</abbr>)
</span>&nbsp;<span ng-if="result.originalRuns.length < 5 || result.newRuns.length < 5"
class="glyphicon glyphicon-warning-sign text-warning"
tooltip="6 Base / New runs recommended for increased confidence in comparison"
tooltip-placement="left"
style="cursor:default">
</span>
{{result.confidence|displayPrecision}} (<abbr title="Result of running t-test on base versus new result distribution: higher values indicate greater confidence that the indicated performance difference is statistically meaningful">{{result.confidenceText}}</abbr>)</span>&nbsp;
<span ng-if="(result.originalRuns.length &&
result.newRuns.length) &&
(result.originalRuns.length < 6 || result.newRuns.length < 6)"
class="glyphicon glyphicon-warning-sign text-warning"
tooltip="6 base / new runs recommended for increased confidence in comparison"
tooltip-placement="left"
style="cursor:default"/>

0 comments on commit f105542

Please sign in to comment.