feat: Only load values for non measure dimension and non standard err…

…or dimension
visualize-admin · Feb 18, 2022 · 897cc13 · 897cc13
1 parent ef8f131
commit 897cc13
Show file tree

Hide file tree

Showing 13 changed files with 32 additions and 12 deletions.
diff --git a/app/charts/area/chart-area.tsx b/app/charts/area/chart-area.tsx
@@ -45,7 +45,7 @@ export const ChartAreasVisualization = ({
     variables: {
       locale,
       iri: dataSetIri,
-      measures: [chartConfig.fields.y.componentIri], // FIXME: Other fields may also be measures
+      dimensions: [chartConfig.fields.y.componentIri], // FIXME: Other fields may also be measures
       filters: queryFilters,
     },
   });

diff --git a/app/charts/bar/chart-bar.tsx b/app/charts/bar/chart-bar.tsx
@@ -43,7 +43,7 @@ export const ChartBarsVisualization = ({
     variables: {
       locale,
       iri: dataSetIri,
-      measures: [chartConfig.fields.x.componentIri], // FIXME: Other fields may also be measures
+      dimensions: [chartConfig.fields.x.componentIri], // FIXME: Other fields may also be measures
       filters: queryFilters,
     },
   });

diff --git a/app/charts/column/chart-column.tsx b/app/charts/column/chart-column.tsx
@@ -52,7 +52,7 @@ export const ChartColumnsVisualization = ({
     variables: {
       locale,
       iri: dataSetIri,
-      measures: [chartConfig.fields.y.componentIri], // FIXME: Other fields may also be measures
+      dimensions: [chartConfig.fields.y.componentIri], // FIXME: Other fields may also be measures
       filters: queryFilters,
     },
   });

diff --git a/app/charts/line/chart-lines.tsx b/app/charts/line/chart-lines.tsx
@@ -48,7 +48,7 @@ export const ChartLinesVisualization = ({
     variables: {
       locale,
       iri: dataSetIri,
-      measures: [chartConfig.fields.y.componentIri], // FIXME: Other fields may also be measures
+      dimensions: [chartConfig.fields.y.componentIri], // FIXME: Other fields may also be measures
       filters: queryFilters,
     },
   });

diff --git a/app/charts/map/chart-map.tsx b/app/charts/map/chart-map.tsx
@@ -50,7 +50,7 @@ export const ChartMapVisualization = ({
     variables: {
       locale,
       iri: dataSetIri,
-      measures: [
+      dimensions: [
         chartConfig.fields.areaLayer.measureIri,
         chartConfig.fields.symbolLayer.measureIri,
       ],

diff --git a/app/charts/pie/chart-pie.tsx b/app/charts/pie/chart-pie.tsx
@@ -42,7 +42,7 @@ export const ChartPieVisualization = ({
     variables: {
       locale,
       iri: dataSetIri,
-      measures: [chartConfig.fields.y.componentIri], // FIXME: Other fields may also be measures
+      dimensions: [chartConfig.fields.y.componentIri], // FIXME: Other fields may also be measures
       filters: queryFilters,
     },
   });

diff --git a/app/charts/scatterplot/chart-scatterplot.tsx b/app/charts/scatterplot/chart-scatterplot.tsx
@@ -50,7 +50,7 @@ export const ChartScatterplotVisualization = ({
     variables: {
       locale,
       iri: dataSetIri,
-      measures: [
+      dimensions: [
         chartConfig.fields.x.componentIri,
         chartConfig.fields.y.componentIri,
       ], // FIXME: Other fields may also be measures

diff --git a/app/charts/table/chart-table.tsx b/app/charts/table/chart-table.tsx
@@ -36,7 +36,7 @@ export const ChartTableVisualization = ({
     variables: {
       locale,
       iri: dataSetIri,
-      measures,
+      dimensions: measures,
       filters: chartConfig.filters,
     },
   });

diff --git a/app/components/data-download.tsx b/app/components/data-download.tsx
@@ -56,7 +56,7 @@ export const DataDownload = memo(
       variables: {
         locale,
         iri: dataSetIri,
-        measures, // FIXME: Other fields may also be measures
+        dimensions: measures, // FIXME: Other fields may also be measures
         filters,
       },
     });

diff --git a/app/configurator/components/chart-options-selector.tsx b/app/configurator/components/chart-options-selector.tsx
@@ -75,7 +75,7 @@ export const ChartOptionsSelector = ({
     variables: {
       locale,
       iri: state.dataSet,
-      measures,
+      dimensions: measures,
       filters: state.chartConfig.filters,
     },
   });

diff --git a/app/domain/data.ts b/app/domain/data.ts
@@ -7,6 +7,7 @@ import {
   NominalDimension,
   OrdinalDimension,
 } from "../graphql/query-hooks";
+import { ResolvedDimension } from "../graphql/shared-types";
 
 export type RawObservationValue = Literal | NamedNode;
 
@@ -202,3 +203,15 @@ export const isGeoShapesDimension = (
 ): dimension is GeoShapesDimension => {
   return dimension?.__typename === "GeoShapesDimension";
 };
+
+export const isStandardErrorResolvedDimension = (dim: ResolvedDimension) => {
+  return dim.data?.related.some((x) => x.type === "StandardError");
+};
+
+export const shouldValuesBeLoadedForResolvedDimension = (
+  dim: ResolvedDimension
+) => {
+  return !(
+    dim.data.isMeasureDimension || isStandardErrorResolvedDimension(dim)
+  );
+};
diff --git a/app/rdf/queries.ts b/app/rdf/queries.ts
@@ -1,4 +1,5 @@
 import { descending, group, index } from "d3";
+import { Maybe } from "graphql-tools";
 import {
   Cube,
   CubeDimension,
@@ -14,6 +15,7 @@ import {
   DimensionValue,
   Observation,
   parseObservationValue,
+  shouldValuesBeLoadedForResolvedDimension,
 } from "../domain/data";
 import { SPARQL_EDITOR, SPARQL_ENDPOINT } from "../domain/env";
 import { DataCubeSearchFilter, DataCubeTheme } from "../graphql/query-hooks";
@@ -250,9 +252,10 @@ export const createCubeDimensionValuesLoader =
   };
 
 export const getCubeDimensionValues = async (
-  { dimension, cube, locale, data }: ResolvedDimension,
+  rdimension: ResolvedDimension,
   filters?: Filters
 ): Promise<DimensionValue[]> => {
+  const { dimension, cube, locale, data } = rdimension;
   if (
     typeof dimension.minInclusive !== "undefined" &&
     typeof dimension.maxInclusive !== "undefined" &&
@@ -267,6 +270,10 @@ export const getCubeDimensionValues = async (
     ];
   }
 
+  if (!shouldValuesBeLoadedForResolvedDimension(rdimension)) {
+    return [];
+  }
+
   return await getCubeDimensionValuesWithLabels({
     dimension,
     cube,

diff --git a/app/utils/dimension-hierarchy.ts b/app/utils/dimension-hierarchy.ts
@@ -277,7 +277,7 @@ export const useHierarchicalDimensionValuesQuery = ({
     variables: {
       iri: dataSetIri,
       locale,
-      measures: path,
+      dimensions: path,
     },
   });