fix: fix calculateNodeValue() in vgrammar-hierrachy to support stri…

…ng value, fix #433
VisActor · Apr 18, 2024 · 054376d · 054376d
1 parent 54c0c28
commit 054376d
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/packages/vgrammar-hierarchy/src/utils.ts b/packages/vgrammar-hierarchy/src/utils.ts
@@ -1,4 +1,4 @@
-import { isNil } from '@visactor/vutils';
+import { isNil, toValidNumber } from '@visactor/vutils';
 import type { HierarchicalDatum, HierarchicalNodeElement } from './interface';
 
 export const calculateNodeValue = <Datum extends HierarchicalDatum, NodeElement extends HierarchicalNodeElement<Datum>>(
@@ -37,13 +37,13 @@ export const calculateNodeValue = <Datum extends HierarchicalDatum, NodeElement
         node,
         getNodeKey
       );
-      node.value = isNil(datum.value) ? res.sum : Math.max(res.sum, node.value);
+      node.value = isNil(datum.value) ? res.sum : Math.max(res.sum, toValidNumber(datum.value));
 
       prevFlattenIndex = res.flattenIndex;
       maxDepth = Math.max(res.maxDepth, maxDepth);
     } else {
       node.isLeaf = true;
-      node.value = isNil(datum.value) ? 0 : datum.value;
+      node.value = toValidNumber(datum.value);
     }
 
     sum += Math.abs(node.value);