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

Add baredSC #225

Merged
merged 12 commits into from
Oct 20, 2023
Merged

Add baredSC #225

merged 12 commits into from
Oct 20, 2023

Conversation

lldelisle
Copy link
Contributor

No description provided.

@lldelisle
Copy link
Contributor Author

As it is related to single-cell, it may be a good idea to put all single-cell RNA-seq into a category different from transcriptomics.

@mvdbeek
Copy link
Member

mvdbeek commented Oct 3, 2023

Agreed, we may not expect them there.

@lldelisle
Copy link
Contributor Author

I thought galaxyproject/planemo#1371 was fixing the following case but apparently I was wrong.
I have a collection of outputs datasets.
All datasets have the same name as they come from the same input (by the way my output naming was not good).
When planemo tests the assertion, it applies the test to the last output with the same name.

@lldelisle lldelisle changed the title [WIP] add baredSC Add baredSC Oct 18, 2023
@lldelisle
Copy link
Contributor Author

Reviews are welcome

@lldelisle
Copy link
Contributor Author

@mvdbeek, tell me if the description is clear enough. Also I could add the same workflows with Scanpy inputs however the problem is that I do not know where to find scanpy inputs to make the tests...

Copy link
Member

@mvdbeek mvdbeek left a comment

Choose a reason for hiding this comment

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

Awesome, this looks great!

@lldelisle
Copy link
Contributor Author

Youhou. I allow myself to bypass protection because last commit motifying something else than README passed tests.

@lldelisle lldelisle merged commit 200970b into galaxyproject:main Oct 20, 2023
@lldelisle lldelisle deleted the add_baredSC branch October 20, 2023 08:33
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