@parca/profile 0.16.126 → 0.16.128

package/CHANGELOG.md

@@ -3,6 +3,16 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## 0.16.128 (2023-03-02)
+
+# 0.16.0 (2023-03-01)
+
+**Note:** Version bump only for package @parca/profile
+
+## [0.16.127](https://github.com/parca-dev/parca/compare/@parca/profile@0.16.126...@parca/profile@0.16.127) (2023-03-01)
+
+**Note:** Version bump only for package @parca/profile
+
 ## [0.16.126](https://github.com/parca-dev/parca/compare/@parca/profile@0.16.125...@parca/profile@0.16.126) (2023-02-27)
 
 **Note:** Version bump only for package @parca/profile

package/dist/ProfileExplorer/index.js

@@ -83,7 +83,7 @@ var ProfileExplorerApp = function (_a) {
         var profileA = ProfileSelectionFromParams(expression_a, from_a, to_a, mergeFrom, mergeTo, labels, filter_by_function);
         setProfileA(profileA);
         // eslint-disable-next-line react-hooks/exhaustive-deps
-    }, [merge_from_a, merge_to_a]);
+    }, [merge_from_a, merge_to_a, filter_by_function]);
     useEffect(function () {
         var _a;
         var mergeFrom = merge_from_b !== null && merge_from_b !== void 0 ? merge_from_b : undefined;
@@ -92,7 +92,7 @@ var ProfileExplorerApp = function (_a) {
         var profileB = ProfileSelectionFromParams(expression_b, from_b, to_b, mergeFrom, mergeTo, labels, filter_by_function);
         setProfileB(profileB);
         // eslint-disable-next-line react-hooks/exhaustive-deps
-    }, [merge_from_b, merge_to_b]);
+    }, [merge_from_b, merge_to_b, filter_by_function]);
     if (profileTypesLoading) {
         return _jsx(_Fragment, { children: loader });
     }

package/dist/ProfileIcicleGraph/IcicleGraph/useNodeColor.js

@@ -11,20 +11,20 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 import { useMemo } from 'react';
-import { diffColor } from '@parca/functions';
+import { COLOR_PROFILES, diffColor } from '@parca/functions';
 import { EVERYTHING_ELSE, selectDarkMode, selectStackColors, useAppSelector } from '@parca/store';
 var useNodeColor = function (_a) {
     var data = _a.data, compareMode = _a.compareMode;
     var colors = useAppSelector(selectStackColors);
     var isDarkMode = useAppSelector(selectDarkMode);
     var color = useMemo(function () {
-        var _a;
+        var _a, _b;
         if (compareMode) {
             var diff = parseFloat(data.diff);
             var cumulative = parseFloat(data.cumulative);
             return diffColor(diff, cumulative, isDarkMode);
         }
-        var color = colors[(_a = data.feature) !== null && _a !== void 0 ? _a : EVERYTHING_ELSE];
+        var color = (_b = colors[(_a = data.feature) !== null && _a !== void 0 ? _a : EVERYTHING_ELSE]) !== null && _b !== void 0 ? _b : (!isDarkMode ? COLOR_PROFILES.default.colors[0][0] : COLOR_PROFILES.default.colors[0][1]);
         return color;
     }, [data, colors, isDarkMode, compareMode]);
     return color;

package/package.json

@@ -1,14 +1,14 @@
 {
   "name": "@parca/profile",
-  "version": "0.16.126",
+  "version": "0.16.128",
   "description": "Profile viewing libraries",
   "dependencies": {
-    "@parca/client": "^0.16.64",
-    "@parca/components": "^0.16.105",
-    "@parca/dynamicsize": "^0.16.53",
-    "@parca/functions": "^0.16.65",
-    "@parca/parser": "^0.16.54",
-    "@parca/store": "^0.16.61",
+    "@parca/client": "^0.16.65",
+    "@parca/components": "^0.16.106",
+    "@parca/dynamicsize": "^0.16.54",
+    "@parca/functions": "^0.16.66",
+    "@parca/parser": "^0.16.55",
+    "@parca/store": "^0.16.62",
     "@types/react-beautiful-dnd": "^13.1.3",
     "d3": "7.8.2",
     "d3-scale": "^4.0.2",
@@ -45,5 +45,5 @@
     "access": "public",
     "registry": "https://registry.npmjs.org/"
   },
-  "gitHead": "8d91f2a4a87d24d92b906533df87b69f50b42cf2"
+  "gitHead": "dd68f2daa5f9656dd3c76c212960ebebe91ef832"
 }

package/src/ProfileExplorer/index.tsx

@@ -131,7 +131,7 @@ const ProfileExplorerApp = ({
 
     setProfileA(profileA);
     // eslint-disable-next-line react-hooks/exhaustive-deps
-  }, [merge_from_a, merge_to_a]);
+  }, [merge_from_a, merge_to_a, filter_by_function]);
 
   useEffect(() => {
     const mergeFrom = merge_from_b ?? undefined;
@@ -149,7 +149,7 @@ const ProfileExplorerApp = ({
 
     setProfileB(profileB);
     // eslint-disable-next-line react-hooks/exhaustive-deps
-  }, [merge_from_b, merge_to_b]);
+  }, [merge_from_b, merge_to_b, filter_by_function]);
 
   if (profileTypesLoading) {
     return <>{loader}</>;

package/src/ProfileIcicleGraph/IcicleGraph/useNodeColor.ts

@@ -13,7 +13,7 @@
 
 import {useMemo} from 'react';
 
-import {diffColor} from '@parca/functions';
+import {COLOR_PROFILES, diffColor} from '@parca/functions';
 import {EVERYTHING_ELSE, selectDarkMode, selectStackColors, useAppSelector} from '@parca/store';
 
 import type {ColoredFlamegraphNode} from './useColoredGraph';
@@ -34,7 +34,10 @@ const useNodeColor = ({data, compareMode}: Props): string => {
       return diffColor(diff, cumulative, isDarkMode);
     }
 
-    const color = colors[data.feature ?? EVERYTHING_ELSE];
+    const color =
+      colors[data.feature ?? EVERYTHING_ELSE] ??
+      (!isDarkMode ? COLOR_PROFILES.default.colors[0][0] : COLOR_PROFILES.default.colors[0][1]);
+
     return color;
   }, [data, colors, isDarkMode, compareMode]);