Bug 971649 - d3 Graph High values are sometimes too large by quite a bit
Summary: d3 Graph High values are sometimes too large by quite a bit
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: RHQ Project
Classification: Other
Component: Core UI
Version: 4.7
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: ---
: RHQ 4.9
Assignee: Mike Thompson
QA Contact: Mike Foley
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-06-07 05:01 UTC by Mike Thompson
Modified: 2014-03-26 08:31 UTC (History)
1 user (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2014-03-26 08:31:38 UTC
Embargoed:


Attachments (Terms of Use)
graph showing erroneous max value (87.66 KB, image/png)
2013-06-07 05:03 UTC, Mike Thompson
no flags Details
json error raw data (9.74 KB, application/json)
2013-06-07 05:07 UTC, Mike Thompson
no flags Details

Description Mike Thompson 2013-06-07 05:01:31 UTC
Description of problem:
Some graphs that show the legend at the right with min, max, avg have an erroneous value in the max.

Looks like an error of 100x in some highs.

Instead of a 9Gb max it is a 900Gb max.

Reproduce:
Take a look at the file system graphs for a single hard disk.

An error in org.rhq.enterprise.gui.coregui.client.inventory.common.charttype.MetricGraphData#normalizeUnitsAndValues

private MeasurementNumericValueAndUnits normalizeUnitsAndValues(double value, MeasurementUnits measurementUnits) {
        MeasurementNumericValueAndUnits newValue = MeasurementConverterClient.fit(value, measurementUnits);
        MeasurementNumericValueAndUnits returnValue;

        // adjust for percentage numbers
        if (measurementUnits.equals(MeasurementUnits.PERCENTAGE)) {
            returnValue = new MeasurementNumericValueAndUnits(newValue.getValue() * 100, newValue.getUnits());
        } else {
            returnValue = new MeasurementNumericValueAndUnits(newValue.getValue(), newValue.getUnits());
        }

        return returnValue;
    }

Which should work but needs further investigation why it does not.

Comment 1 Mike Thompson 2013-06-07 05:03:23 UTC
Created attachment 757984 [details]
graph showing erroneous max value

Comment 2 Mike Thompson 2013-06-07 05:07:02 UTC
Created attachment 757985 [details]
json error raw data

Comment 3 Mike Thompson 2013-09-10 16:35:51 UTC
The graphs have changed much since this BZ. In fact, the only place you can see this legend is on a wide portlet and inventory detail summary graph. I have verified this fix in file system resources of latest build.

Comment 4 Heiko W. Rupp 2014-03-26 08:31:38 UTC
Bulk closing now that 4.10 is out.

If you think an issue is not resolved, please open a new BZ and link to the existing one.


Note You need to log in before you can comment on or make changes to this bug.