-
Notifications
You must be signed in to change notification settings - Fork 14
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
Garnett outdated libraries #301
Comments
@pcm32 I'm wondering if we should remove these tools from the https://singlecell.usegalaxy.eu instance given that they don't currently function as a workflow, the original tools are quite old, and there's not much resource for keeping them updated? |
I think that if we remove them, they will never get to be on a workflow. Very likely most galaxy tools have never made it to a very reused workflow for that matter. |
So the question here is what happens if we update those underlying tools, how much effort is required and can someone do it? |
I can submit it as an intern role like Morgan had last year, so if you were to update it, I can probs get someone testing it for the summer latest as a nice wee project |
Running the tool
Garnett - classify cells
in a Galaxy workflow produces the following error:Here is a Galaxy workflow replicating the issue:
https://humancellatlas.usegalaxy.eu/u/hexhowells/h/garnett-reduceddims-error
This appears to be an issue with an older version of the monocle3 library as it was raised in their Github a while ago: cole-trapnell-lab/monocle3#272 seems the issue was resolved in an updated version of monocle3.
I have run the
classify_cells
function locally on my machine with the same data used in the workflow using the most recent versions of monocle3 (monocle3_1.3.1) and garnett (garnett_0.2.20) so there seems no issue with the data.The text was updated successfully, but these errors were encountered: