Fixes null run ID bug

README.md

@@ -195,6 +195,7 @@ Then, you can run the following code to start the UI:
 
 ```bash
 hamilton ui
+# python -m hamilton.cli.__main__ ui # on windows
 ```
 
 This will start the UI at [localhost:8241](https://localhost:8241). You can then navigate to the UI to see your dataflows.

ui/backend/setup.py

@@ -1,4 +1,4 @@
-#!/usr/bin/env python
+# /usr/bin/env python
 # -*- coding: utf-8 -*-
 
 """The setup script."""
@@ -18,7 +18,7 @@ def load_requirements():
 
 setup(
     name="sf-hamilton-ui",  # there's already a hamilton in pypi
-    version="0.0.4",
+    version="0.0.5",
     description="Hamilton, the micro-framework for creating dataframes.",
     long_description="""Hamilton tracking server, see [the docs for more](https://github.com/dagworks-inc/hamilton/tree/main/ui/)""",
     long_description_content_type="text/markdown",

ui/frontend/src/components/dashboard/Runs/Task/ErrorView.tsx

@@ -34,6 +34,7 @@ export const ErrorView = (props: {
 
   const errorAttributes = getNodeRunAttributes<AttributeError1>(
     props.nodeRunData.flatMap((i) => i?.attributes || []),
+    props.nodeRunData.map((i) => i?.dag_run || 0),
     "AttributeError1"
   );
   const [priorRunIndex, setPriorRunIndex] = useState(

ui/frontend/src/components/dashboard/Runs/Task/result-summaries/DataObservability.tsx

@@ -149,6 +149,7 @@ export const ResultsSummaryView = (props: {
 
   const primitive1Views = getNodeRunAttributes<AttributePrimitive1>(
     props.nodeRunData.flatMap((i) => i?.attributes || []),
+    props.runIds,
     "AttributePrimitive1"
   );
   if (primitive1Views.length > 0) {
@@ -162,6 +163,7 @@ export const ResultsSummaryView = (props: {
     );
   }
 
+
   // ... [similar blocks for other attributes]
 
   // const error1Views = getNodeRunAttributes<AttributeError1>(
@@ -180,6 +182,7 @@ export const ResultsSummaryView = (props: {
 
   const dict1Views = getNodeRunAttributes<AttributeDict1>(
     props.nodeRunData.flatMap((i) => i?.attributes || []),
+    props.runIds,
     "AttributeDict1"
   );
 
@@ -195,6 +198,7 @@ export const ResultsSummaryView = (props: {
 
   const dict2Views = getNodeRunAttributes<AttributeDict2>(
     props.nodeRunData.flatMap((i) => i?.attributes || []),
+    props.runIds,
     "AttributeDict2"
   );
   if (dict2Views.length > 0) {
@@ -210,6 +214,7 @@ export const ResultsSummaryView = (props: {
   const dagworksDescribe3Views =
     getNodeRunAttributes<AttributeDagworksDescribe3>(
       props.nodeRunData.flatMap((i) => i?.attributes || []),
+      props.runIds,
       "AttributeDagworksDescribe3"
     );
   if (dagworksDescribe3Views.length > 0) {
@@ -225,6 +230,7 @@ export const ResultsSummaryView = (props: {
 
   const pandasDescribe1View = getNodeRunAttributes<AttributePandasDescribe1>(
     props.nodeRunData.flatMap((i) => i?.attributes || []),
+    props.runIds,
     "AttributePandasDescribe1"
   );
 
@@ -241,6 +247,7 @@ export const ResultsSummaryView = (props: {
 
   const unsupportedViews = getNodeRunAttributes<AttributeUnsupported1>(
     props.nodeRunData.flatMap((i) => i?.attributes || []),
+    props.runIds,
     "AttributeUnsupported1"
   );
 

ui/frontend/src/components/dashboard/Visualize/DAGViz.tsx

@@ -971,13 +971,6 @@ const GroupedNodeComponent = (props: {
                 {groupName}
               </span>
             </div>
-            {/* <CollapseExpandIcon
-                className="font-extrabold hover:scale-110 cursor-pointer"
-                onClick={() => {
-                  //eslint-disable-next-line no-debugger
-                  toggleCollapseExpand();
-                }}
-              /> */}
           </>
         )}
       </span>

ui/frontend/src/state/api/friendlyApi.ts

@@ -216,17 +216,18 @@ export type DAGWorksDescribeColumn =
 
 export function getNodeRunAttributes<T>(
   allAttributes: NodeRunAttribute[],
+  dagRunIds: number[],
   cls: keyof typeof nodeAttributeTypeMap
 ): { name: string; value: T; runId: number }[] {
   const { version, type } = nodeAttributeTypeMap[cls];
   return allAttributes
-    .map((item) => {
+    .map((item, i) => {
       return {
         name: item.name,
         value: item.value as T,
         schema_version: item.schema_version,
         type: item.type,
-        runId: item.dag_run as number,
+        runId: dagRunIds[i],
       };
     })
     .filter((attr) => attr.schema_version === version && attr.type === type);