Merge pull request #979 from biomage-org/fix-leiden-hms

Set louvain as main key for leiden
hms-dbmi-cellenics · Feb 5, 2024 · a4c16b5 · a4c16b5
2 parents 0bb8bd2 + 3a05323
commit a4c16b5
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 10 deletions.
diff --git a/src/components/data-processing/ConfigureEmbedding/CalculationConfig.jsx b/src/components/data-processing/ConfigureEmbedding/CalculationConfig.jsx
@@ -358,16 +358,17 @@ const CalculationConfig = (props) => {
             <Select
               value={clusteringMethod}
               disabled={disabled}
-              onChange={(value) => updateSettings(
-                { clusteringSettings: { method: value } },
-              )}
+              onChange={(value) => {
+                updateSettings({
+                  clusteringSettings: {
+                    method: value,
+                  },
+                });
+                dispatch(runCellSetsClustering(experimentId, resolution));
+              }}
             >
               <Option value='louvain'>Louvain</Option>
-              <Option value='leiden' disabled>
-                <Tooltip title='Leiden metric is going to be supported on a future version of the platform.'>
-                  Leiden
-                </Tooltip>
-              </Option>
+              <Option value='leiden'>Leiden</Option>
               <Option value='slm' disabled>
                 <Tooltip title='SLM metric is going to be supported on a future version of the platform.'>
                   SLM

diff --git a/src/redux/actions/componentConfig/getTrajectoryPlotPseudoTime.js b/src/redux/actions/componentConfig/getTrajectoryPlotPseudoTime.js
@@ -32,7 +32,7 @@ const getTrajectoryPlotPseudoTime = (
     },
     clustering: {
       method: clusteringSettings.method,
-      resolution: clusteringSettings.methodSettings[clusteringSettings.method].resolution,
+      resolution: clusteringSettings.methodSettings.louvain.resolution,
     },
     cellSets: selectedCellSets,
     rootNodes,

diff --git a/src/redux/actions/componentConfig/getTrajectoryPlotStartingNodes.js b/src/redux/actions/componentConfig/getTrajectoryPlotStartingNodes.js
@@ -31,7 +31,7 @@ const getTrajectoryPlotStartingNodes = (
     },
     clustering: {
       method: clusteringSettings.method,
-      resolution: clusteringSettings.methodSettings[clusteringSettings.method].resolution,
+      resolution: clusteringSettings.methodSettings.louvain.resolution,
     },
     cellSets: selectedCellSets,
   };