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

possible improvements #62

Open
3 of 4 tasks
bernt-matthias opened this issue May 29, 2018 · 1 comment
Open
3 of 4 tasks

possible improvements #62

bernt-matthias opened this issue May 29, 2018 · 1 comment

Comments

@bernt-matthias
Copy link
Collaborator

bernt-matthias commented May 29, 2018

Here some possible improvements (for now I have a solution for all of them):

  • remove spaces in tool name (problem comes for subcommands)
  • <command interpreter="python"><!\[CDATA\[ -> <command><!\[CDATA\[python
  • non optional positional arguments are currently treated as optional in the galaxy xml file
  • Is it possible to use the name of positional arguments instead of naming them positional_NUMBER? If there are multiple positionals with the same name one could still number them.

This would simplify automated postprocessing quite a bit.

Example: checkm outliers -h

positional arguments:
out_folder folder specified during qa command
bin_folder folder containing bins (fasta format)
tetra_profile tetranucleotide profiles for each sequence (see tetra command)
output_file print results to file

Are now positional1,..., positional 4

I would like to use sed/grep to replace the corresponding parts of the xml with appropriate content.

@hexylena
Copy link
Owner

I'd love to see all of this!

I'm afraid I've got a couple of other higher priority things for approximately the next month, so if you want most of this, I'd be happy to review any pull requests you make. Sorry I don't have more time for this repository.

bernt-matthias added a commit to bernt-matthias/argparse2tool that referenced this issue May 30, 2018
hexylena#62

- dropped interpreter in favour of `python command`
- replace spaces by underscore in tool id
- use name of positionals
@bernt-matthias bernt-matthias mentioned this issue May 30, 2018
5 tasks
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

No branches or pull requests

2 participants