-
Notifications
You must be signed in to change notification settings - Fork 10
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
subparser parents #60
Comments
Wow you were not kidding about a lot of argparse options. Oh my word. Thanks for the example. I see that it's under GPL3, would you mind if I included it into the |
Sure, I took it from bioconda checkm-genome=1.0.11 (most recent version in https://github.com/Ecogenomics/CheckM/blob/master/bin/checkm) and adapted it to include only the argparse stuff (and replaced some imports by the actual code). |
Perfect, thanks! |
since I just stumbled across this again, still no time for development work, but
I've often considered how nice it would be if I could find a good AST parser (or three), and then identify large common subsets (using the same code structure/same XML structure even if different parameter names) across N xml files and automatically rewrite it into a macro. But that's almost a masters-time/level project and hard to fit that in. edit; @bgruening just in case you have any masters students who need another project? But I guess this isn't a very 'fun' or attractive project. |
I now "solved" this in a shell script generating the macros. Made small progress here: https://github.com/bernt-matthias/mb-galaxy-tools/tree/master/tools/checkm |
Great idea! I would like to see this as a planemo feature I think. It should be useful to a lot of tools. A master student for planemo development would be great! |
one more request on this way .. the tool I want to wrap uses the parent feature of subparser, i.e. it defines subparsers that are reused in many other subbarsers.
check for the "parent" argument (in combination with add_help=False) here :
checkm.py.txt
I guess macros would be a great way to do this.
for now I will do this manually
The text was updated successfully, but these errors were encountered: