UI: fix show-in-order for histogram plot type (#1195)

evidentlyai · Jul 9, 2024 · cd0ad77 · cd0ad77
1 parent ab9ea17
commit cd0ad77
Show file tree

Hide file tree

Showing 9 changed files with 15 additions and 12 deletions.
diff --git a/src/evidently/nbextension/static/index.js b/src/evidently/nbextension/static/index.js
diff --git a/src/evidently/ui/assets/index.html b/src/evidently/ui/assets/index.html
@@ -11,7 +11,7 @@
       content="Evidently - ML Monitoring Demo. Hosted example to monitor the performance of a demand forecasting model on a toy dataset."
     />
     <title>Evidently - ML Monitoring Demo</title>
-    <script type="module" crossorigin src="/static/js/index-C5cNhiiF.js"></script>
+    <script type="module" crossorigin src="/static/js/index-D7UWIwGk.js"></script>
     <link rel="modulepreload" crossorigin href="/static/js/vendor-DRRvL2bZ.js">
     <link rel="stylesheet" crossorigin href="/static/css/index-maTzSdYn.css">
   </head>

diff --git a/...ts/static/js/DashboardContent-C9df8NyG.js → ...ts/static/js/DashboardContent-D3SFZTsO.js b/...ts/static/js/DashboardContent-C9df8NyG.js → ...ts/static/js/DashboardContent-D3SFZTsO.js
diff --git a/...ts/static/js/DashboardWidgets-CzYedL-x.js → ...ts/static/js/DashboardWidgets-Bui21iWY.js b/...ts/static/js/DashboardWidgets-CzYedL-x.js → ...ts/static/js/DashboardWidgets-Bui21iWY.js
diff --git a/...tly/ui/assets/static/js/index-B6U_Sdu1.js → ...tly/ui/assets/static/js/index-8XYbUoWV.js b/...tly/ui/assets/static/js/index-B6U_Sdu1.js → ...tly/ui/assets/static/js/index-8XYbUoWV.js
diff --git a/...tly/ui/assets/static/js/index-JkvV_Cuw.js → ...tly/ui/assets/static/js/index-CfH0L5ET.js b/...tly/ui/assets/static/js/index-JkvV_Cuw.js → ...tly/ui/assets/static/js/index-CfH0L5ET.js
diff --git a/...tly/ui/assets/static/js/index-C5cNhiiF.js → ...tly/ui/assets/static/js/index-D7UWIwGk.js b/...tly/ui/assets/static/js/index-C5cNhiiF.js → ...tly/ui/assets/static/js/index-D7UWIwGk.js
diff --git a/ui/packages/evidently-ui-lib/src/widgets/BigGraphWidgetContent.tsx b/ui/packages/evidently-ui-lib/src/widgets/BigGraphWidgetContent.tsx
@@ -10,6 +10,7 @@ interface BigGraphWidgetProps extends AdditionalGraphInfo {
 
 const BigGraphWidgetContent: React.FunctionComponent<BigGraphWidgetProps> = (props) => {
   const viewParams = useDashboardViewParams()
+  const isHistogram = props.data.some(({ type }) => type === 'histogram')
 
   return (
     <div>
@@ -20,7 +21,7 @@ const BigGraphWidgetContent: React.FunctionComponent<BigGraphWidgetProps> = (pro
           title: undefined,
           xaxis: {
             ...props.layout?.xaxis,
-            type: viewParams?.isXaxisAsCategorical ? 'category' : undefined
+            type: viewParams?.isXaxisAsCategorical && !isHistogram ? 'category' : undefined
           }
         }}
         config={{ responsive: true }}

diff --git a/ui/packages/evidently-ui-lib/src/widgets/RichDataWidget.tsx b/ui/packages/evidently-ui-lib/src/widgets/RichDataWidget.tsx
@@ -26,6 +26,8 @@ const RichDataWidget: React.FunctionComponent<RichDataParams & { widgetSize: num
 ) => {
   const [details, setDetails] = useState<boolean>(false)
   const viewParams = useDashboardViewParams()
+  const isHistogram = props.graph?.data.some(({ type }) => type === 'histogram')
+
   return (
     <React.Fragment>
       <Grid container spacing={2} justifyContent="center" alignItems="center">
@@ -66,7 +68,7 @@ const RichDataWidget: React.FunctionComponent<RichDataParams & { widgetSize: num
                 title: undefined,
                 xaxis: {
                   ...props.graph.layout?.xaxis,
-                  type: viewParams?.isXaxisAsCategorical ? 'category' : undefined
+                  type: viewParams?.isXaxisAsCategorical && !isHistogram ? 'category' : undefined
                 }
               }}
               config={{ responsive: true }}