Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New Filter, Plot, and Explore Tutorial Using Seurat Tools! #4759

Merged
merged 79 commits into from
Apr 9, 2024

Conversation

Camila-goclowski
Copy link
Contributor

@Camila-goclowski Camila-goclowski commented Feb 28, 2024

New filter, plot, and explore tutorial (mimicking currently available Scanpy and Seurat RStudio ones) using Seurat Tools by Pablo!

@Camila-goclowski Camila-goclowski marked this pull request as ready for review March 26, 2024 15:13
@Camila-goclowski Camila-goclowski requested a review from a team as a code owner March 26, 2024 15:13
hexylena
hexylena previously approved these changes Apr 5, 2024
@hexylena
Copy link
Member

hexylena commented Apr 5, 2024

I pushed some changes to make the CYOA work (you needed hyphens in the class names) and make the branches more consistent but now it looks good formatting-wise. @pavanvidem is the technical side ok as well?

@pavanvidem
Copy link
Member

If I understood it correctly, there are two ways of generating Seurat objects based on the input data:

  1. EBI-Data-Retrieval and Importing-from-a-history should use Read10X tool
  2. Uploading-from-Zenodo should Filter Cells tool

@Camila-goclowski please correct me if I'm wrong.

In the last commit, I created a new div with both the EBI-Data-Retrieval and Importing-from-a-history classes. Hope that works as needed.

@pavanvidem
Copy link
Member

@hexylena how to know which div is visible by default on page load? When I served on my laptop, content from all the divs is shown by default.

@pavanvidem
Copy link
Member

pavanvidem commented Apr 5, 2024

@Camila-goclowski It would be nice to provide a link to a workflow or a history to compare the results. We have them in most of the single-cell tutorials.

Copy link
Member

@pavanvidem pavanvidem left a comment

Choose a reason for hiding this comment

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

We can add the workflow/history later. Otherwise, everything is good from my side.

Great efforts @Camila-goclowski! Many thanks for your contribution!!

@pavanvidem pavanvidem merged commit 7b45001 into galaxyproject:main Apr 9, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants