Merge remote-tracking branch 'origin/main' into map-collection-variab…

…le-map-type
VEuPathDB · Aug 23, 2023 · 7f7167b · 7f7167b
2 parents 8d9f506 + c90abd2
commit 7f7167b
Show file tree

Hide file tree

Showing 6 changed files with 35 additions and 9 deletions.
diff --git a/Dockerfile b/Dockerfile
@@ -33,7 +33,10 @@ COPY package.json package.json
 COPY yarn.lock yarn.lock
 COPY packages packages
 
+ARG NODE_OPTIONS=--max-old-space-size=4096
+
 # Build the client bundles
+RUN echo "Building with NODE_OPTIONS=$NODE_OPTIONS"
 RUN yarn \
     && yarn nx bundle:npm @veupathdb/clinepi-site \
     && yarn nx bundle:npm @veupathdb/genomics-site \

diff --git a/docker-compose.yml b/docker-compose.yml
@@ -1,7 +1,5 @@
 version: "3.5"
 
-name: "web-client"
-
 networks:
   traefik:
     external: true

diff --git a/packages/libs/components/src/plots/VolcanoPlot.tsx b/packages/libs/components/src/plots/VolcanoPlot.tsx
@@ -363,11 +363,17 @@ function VolcanoPlot(props: VolcanoPlotProps, ref: Ref<HTMLDivElement>) {
                   }}
                 >
                   <ul>
-                    {data?.pointIDs?.map((id) => (
-                      <li key={id}>
-                        <span>{id}</span>
-                      </li>
-                    ))}
+                    {data?.displayLabels
+                      ? data.displayLabels.map((label) => (
+                          <li key={label}>
+                            <span>{label}</span>
+                          </li>
+                        ))
+                      : data?.pointIDs?.map((id) => (
+                          <li key={id}>
+                            <span>{id}</span>
+                          </li>
+                        ))}
                   </ul>
                   <div
                     className="pseudo-hr"

diff --git a/packages/libs/components/src/types/plots/volcanoplot.ts b/packages/libs/components/src/types/plots/volcanoplot.ts
@@ -11,6 +11,8 @@ export type VolcanoPlotDataPoint = {
   pointIDs?: string[];
   // Used to determine color of data point in the plot
   significanceColor?: string;
+  // Optional user-friendly label. One for each pointID
+  displayLabels?: string[];
 };
 
 export type VolcanoPlotData = Array<VolcanoPlotDataPoint>;
diff --git a/...s/eda/src/lib/core/components/visualizations/implementations/VolcanoPlotVisualization.tsx b/...s/eda/src/lib/core/components/visualizations/implementations/VolcanoPlotVisualization.tsx
@@ -52,6 +52,7 @@ import SliderWidget, {
 } from '@veupathdb/components/lib/components/widgets/Slider';
 import { ResetButtonCoreUI } from '../../ResetButton';
 import AxisRangeControl from '@veupathdb/components/lib/components/plotControls/AxisRangeControl';
+import { fixVarIdLabel } from '../../../utils/visualization';
 // end imports
 
 const DEFAULT_SIG_THRESHOLD = 0.05;
@@ -270,9 +271,18 @@ function VolcanoPlotViz(props: VisualizationProps<Options>) {
          */
         .map((d) => {
           const { pointID, ...remainingProperties } = d;
+          // Try to find a user-friendly label for the point. Note that pointIDs are in entityID.variableID format.
+          const displayLabel =
+            pointID &&
+            fixVarIdLabel(
+              pointID.split('.')[1],
+              pointID.split('.')[0],
+              entities
+            );
           return {
             ...remainingProperties,
             pointIDs: pointID ? [pointID] : undefined,
+            displayLabels: displayLabel ? [displayLabel] : undefined,
             significanceColor: assignSignificanceColor(
               Number(d.log2foldChange),
               Number(d.pValue),
@@ -299,19 +309,24 @@ function VolcanoPlotViz(props: VisualizationProps<Options>) {
         if (foundIndex === -1) {
           aggregatedData.push(entry);
         } else {
-          const { pointIDs } = aggregatedData[foundIndex];
+          const { pointIDs, displayLabels } = aggregatedData[foundIndex];
           if (pointIDs) {
             aggregatedData[foundIndex] = {
               ...aggregatedData[foundIndex],
               pointIDs: [
                 ...pointIDs,
                 ...(entry.pointIDs ? entry.pointIDs : []),
               ],
+              displayLabels: displayLabels && [
+                ...displayLabels,
+                ...(entry.displayLabels ? entry.displayLabels : []),
+              ],
             };
           } else {
             aggregatedData[foundIndex] = {
               ...aggregatedData[foundIndex],
               pointIDs: entry.pointIDs,
+              displayLabels: entry.displayLabels,
             };
           }
         }

diff --git a/packages/libs/wdk-client/src/Utils/UserPreferencesUtils.ts b/packages/libs/wdk-client/src/Utils/UserPreferencesUtils.ts
@@ -65,7 +65,9 @@ export async function getResultTableColumnsPref(
 
   const [knownColumns, unknownColumns] = partition(
     columns,
-    (columnName) => columnName in recordClass.attributesMap
+    (columnName) =>
+      recordClass.attributes.some((a) => a.name === columnName) ||
+      question.dynamicAttributes.some((a) => a.name === columnName)
   );
 
   if (unknownColumns.length > 0) {