Skip to content

Feature seurat upload#1273

Draft
dlesper wants to merge 23 commits into
develfrom
feature_seurat_upload
Draft

Feature seurat upload#1273
dlesper wants to merge 23 commits into
develfrom
feature_seurat_upload

Conversation

@dlesper

@dlesper dlesper commented May 28, 2026

Copy link
Copy Markdown
Contributor

Seurat Uploader

  • Note bioconductor was updated to 3.22 due to anndataR requiring the newer version
  • Changes to Dockerfile ubuntu packages were due to R dependencies
  • Changes to CGI and HTML to allow RDS to be processed
  • Creation of SeuratUploader in lib, and function calls throughout CGI to process RDS

@dlesper dlesper requested review from adkinsrs and jorvis May 28, 2026 18:26
Comment thread docker/Dockerfile

@adkinsrs adkinsrs left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I committed various changes, so for this particular review request I am just going to mark it as "request changes" since that was what was needed.

Comment thread www/cgi/process_uploaded_expression_dataset.cgi
Comment thread www/cgi/process_uploaded_expression_dataset.cgi
Comment thread www/cgi/process_uploaded_expression_dataset.cgi
Comment thread lib/gear/seuratuploader.py
@adkinsrs adkinsrs marked this pull request as draft June 4, 2026 17:09
@adkinsrs

adkinsrs commented Jun 4, 2026

Copy link
Copy Markdown
Member

Converted the pull request into a draft, as @dlesper needs to add a retry request mechanism to the MyGene call to address intermittent 500-level responses.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants