Merge pull request #980 from hms-dbmi-cellenics/fix-download-rds

fix download processed
hms-dbmi-cellenics · Feb 13, 2024 · becdb3b · becdb3b
2 parents 58a2f3f + 3c09366
commit becdb3b
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/src/__test__/components/data-management/__snapshots__/DownloadDataButton.test.jsx.snap b/src/__test__/components/data-management/__snapshots__/DownloadDataButton.test.jsx.snap
@@ -23,7 +23,7 @@ exports[`DownloadDataButton Downloads processed matrix properly 1`] = `
     "testae48e318dab9a1bd0bexperiment-0",
     {
       "embeddingMethod": "umap",
-      "isSeurat": undefined,
+      "isSeurat": false,
       "name": "DownloadAnnotSeuratObject",
     },
     [Function],

diff --git a/src/utils/extraActionCreators/downloadProcessedMatrix.js b/src/utils/extraActionCreators/downloadProcessedMatrix.js
@@ -15,12 +15,13 @@ const downloadProcessedMatrix = (experimentId) => async (dispatch, getState) =>
     const {
       method: embeddingMethod,
       // embedding is saved in R object if downloading from project created by Seurat object upload
-      useSaved: isSeurat,
+      useSaved,
     } = getState().experimentSettings.processing.configureEmbedding.embeddingSettings;
 
     await dispatch(loadEmbedding(experimentId, embeddingMethod, true));
 
     const taskName = 'DownloadAnnotSeuratObject';
+    const isSeurat = useSaved || false;
 
     // the request needs the embeddingETag to merge that data with the rds
     // the embeddingETag is added by the API to this body