Skip to content

Adapted MMUPHin for workflow use #6857

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

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

SantaMcCloud
Copy link
Contributor

@SantaMcCloud SantaMcCloud commented Mar 17, 2025

FOR CONTRIBUTOR:

  • I have read the CONTRIBUTING.md document and this tool is appropriate for the tools-iuc repo.
  • License permits unrestricted use (educational + commercial)
  • This PR adds a new tool or tool collection
  • This PR updates an existing tool or tool collection
  • This PR does something else (explain below)

@SantaMcCloud
Copy link
Contributor Author

SantaMcCloud commented Mar 17, 2025

I was ask to change this wrapper such that this tool can work in a workflow since the data_column type is not good to in a workflow. In this case all param which used this type and multiple column can be choosed was set into a conditional which either use the data_column type or a text input is used where a list in comma seperated column number can be inputed.

@paulzierep
Copy link
Contributor

Same issue as here: #6858

@SantaMcCloud
Copy link
Contributor Author

Same issue as here: #6858

all issues should be done herer and the validator is now there for only digits input in a comma seperated list

@paulzierep
Copy link
Contributor

if we could fix galaxyproject/galaxy#19832 this tool fix would not be needed

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