Merge pull request #65 from concord-consortium/186999766-no-round-off…

…-value [#186999766] - Value rounding causes value to change with unit conversion calculations
concord-consortium · Feb 9, 2024 · 2ff5201 · 2ff5201
2 parents 3ed8808 + de437d3
commit 2ff5201
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/components/attribute-selector.tsx b/src/components/attribute-selector.tsx
@@ -33,15 +33,15 @@ export const AttributesSelector = () => {
         const fromUnits = dataType.units[units];
         const toUnits = dataType.units[newUnits];
         if (operator === "between") {
-          const lowerValue = Math.round(dataType.convertUnits(fromUnits, toUnits, filter.lowerValue.toString()));
-          const upperValue = Math.round(dataType.convertUnits(fromUnits, toUnits, filter.upperValue.toString()));
+          const lowerValue = parseFloat((dataType.convertUnits(fromUnits, toUnits, filter.lowerValue.toString())).toFixed(1));
+          const upperValue = parseFloat((dataType.convertUnits(fromUnits, toUnits, filter.upperValue.toString())).toFixed(1));
           return {
             ...filter,
             lowerValue,
             upperValue
           };
         } else if (operator !== "top" && operator !== "bottom" && operator !== "aboveMean" && operator !== "belowMean" && operator !== "all") {
-          const value = Math.round(dataType.convertUnits(fromUnits, toUnits, filter.value.toString()));
+          const value = parseFloat((dataType.convertUnits(fromUnits, toUnits, filter.value.toString())).toFixed(1));
           return {
             ...filter,
             value