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

Tools template #830

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

mira-miracoli
Copy link
Contributor

@mira-miracoli mira-miracoli commented Jul 5, 2023

@mira-miracoli mira-miracoli marked this pull request as draft July 5, 2023 08:07
@mira-miracoli mira-miracoli marked this pull request as ready for review July 5, 2023 08:19
@kysrpex
Copy link
Contributor

kysrpex commented Jul 5, 2023

I'll have a look at why TPV dry-run is failing and fix it.

@kysrpex
Copy link
Contributor

kysrpex commented Jul 5, 2023

For any other readers: this PR cannot be merged yet because we found no container for SRMA.

Co-authored-by: José Manuel Domínguez <[email protected]>
@bgruening
Copy link
Member

Are we trying to build a container for this tool? Should we just hide this tool? It seems it never worked, I can not find the JAR file.

@mira-miracoli
Copy link
Contributor Author

I think building a container would be the cleanest solution. However I don't know how we handle this outside of biocontainers.
Letting it run in a python container could fix it, but is of course not the cleanest and most reproducible stategy.

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.

3 participants