Attachment #113240 for bug #207579

View | Details | Raw Unified | Return to bug 207579
Collapse All | Expand All




		BuildResults currentBuild = configResults.getCurrentBuildResults();
		BuildResults baselineBuild = configResults.getBaselineBuildResults();

		// Compute difference values
		double baselineValue = baselineBuild.getValue(dim_id);
		double diffValue = baselineValue - currentBuild.getValue(dim_id);
		double diffPercentage =  baselineValue == 0 ? 0 : Math.round(diffValue / baselineValue * 1000) / 10.0;
		String diffDisplayValue = currentDim.getDisplayValue(diffValue);

		// Set colors
		String fontColor = "";
		if (diffPercentage > 10) {
			fontColor = "#006600";	// green
		}
		if (diffPercentage < -10) {
			fontColor = "#FF0000";	// red
		}

		// Print line
		String percentage = (diffPercentage == 0) ? "" : "<br>" + diffPercentage + " %";

		if (diffPercentage > 10 || diffPercentage < -10) {
			stream.print("<td><FONT COLOR=\"" + fontColor + "\"><b>" + diffDisplayValue + percentage + "</b></FONT></td>");
		} else {




    private final int fId;
	private final Unit fUnit;
	private final int fMultiplier;
	private final boolean fLargerIsBetter= false;	// true indicates that larger values are better
	
	public static Dim getDimension(int id) {
        InternalDimensions.COMITTED.getId();	// trigger loading class InternalDimensions

		return fMultiplier;
	}

	public boolean largerIsBetter() {
	    return fLargerIsBetter;
	}

	public String getName() {
		return DimensionMessages.getString(fId);
	}

Return to bug 207579

Lines 258-277 Link Here

(-)src/org/eclipse/test/performance/ui/ScenarioData.java (-8 / +11 lines)
258	BuildResults currentBuild = configResults.getCurrentBuildResults();	258	BuildResults currentBuild = configResults.getCurrentBuildResults();
259	BuildResults baselineBuild = configResults.getBaselineBuildResults();	259	BuildResults baselineBuild = configResults.getBaselineBuildResults();
260		260
		261	// Compute difference values
261	double baselineValue = baselineBuild.getValue(dim_id);	262	double baselineValue = baselineBuild.getValue(dim_id);
262	double diffValue = currentBuild.getValue(dim_id) - baselineValue;	263	double diffValue = baselineValue - currentBuild.getValue(dim_id);
263	double diffPercentage = baselineValue == 0 ? 0 : Math.round(diffValue / baselineValue * 1000) / 10.0;	264	double diffPercentage = baselineValue == 0 ? 0 : Math.round(diffValue / baselineValue * 1000) / 10.0;
264	String diffDisplayValue = currentDim.getDisplayValue(diffValue);	265	String diffDisplayValue = currentDim.getDisplayValue(diffValue);
265	// green	266
		267	// Set colors
266	String fontColor = "";	268	String fontColor = "";
267	if ((diffPercentage < -10 && !currentDim.largerIsBetter()) \|\| (diffPercentage > 10 && currentDim.largerIsBetter()))	269	if (diffPercentage > 10) {
268	fontColor = "#006600";	270	fontColor = "#006600"; // green
269	if ((diffPercentage < -10 && currentDim.largerIsBetter()) \|\| (diffPercentage > 10 && !currentDim.largerIsBetter()))	271	}
270	fontColor = "#FF0000";	272	if (diffPercentage < -10) {
		273	fontColor = "#FF0000"; // red
		274	}
271		275
272	diffPercentage = Math.abs(diffPercentage);	276	// Print line
273	String percentage = (diffPercentage == 0) ? "" : "<br>" + diffPercentage + " %";	277	String percentage = (diffPercentage == 0) ? "" : "<br>" + diffPercentage + " %";
274
275	if (diffPercentage > 10 \|\| diffPercentage < -10) {	278	if (diffPercentage > 10 \|\| diffPercentage < -10) {
276	stream.print("<td><FONT COLOR=\"" + fontColor + "\"><b>" + diffDisplayValue + percentage + "</b></FONT></td>");	279	stream.print("<td><FONT COLOR=\"" + fontColor + "\"><b>" + diffDisplayValue + percentage + "</b></FONT></td>");
277	} else {	280	} else {

Lines 24-30 Link Here

(-)src/org/eclipse/test/internal/performance/data/Dim.java (-5 lines)
24	private final int fId;	24	private final int fId;
25	private final Unit fUnit;	25	private final Unit fUnit;
26	private final int fMultiplier;	26	private final int fMultiplier;
27	private final boolean fLargerIsBetter= false; // true indicates that larger values are better
28		27
29	public static Dim getDimension(int id) {	28	public static Dim getDimension(int id) {
30	InternalDimensions.COMITTED.getId(); // trigger loading class InternalDimensions	29	InternalDimensions.COMITTED.getId(); // trigger loading class InternalDimensions
Lines 66-75 Link Here
66	return fMultiplier;	65	return fMultiplier;
67	}	66	}
68		67
69	public boolean largerIsBetter() {
70	return fLargerIsBetter;
71	}
72
73	public String getName() {	68	public String getName() {
74	return DimensionMessages.getString(fId);	69	return DimensionMessages.getString(fId);
75	}	70	}