-
Notifications
You must be signed in to change notification settings - Fork 185
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
Python- Re-implementation of command-line parsing #1608
Comments
while we're at it:
|
While I fully support the idea to look at these aspects, may I carefully suggest we keep it on a wish list type of level for now? It'd be neat if we can close off an RC4 before ISMRM; given what's already ready to "ship" for some time (and the amount of it), and how long it's been since RC3. Lots of users will be looking forward to RC4, and we'll be facing many of them at ISMRM. Disclaimer: no hidden intentions above. Could've made this into an issue by itself, so there's no reason why I added it specifically to this thread, other than this thread having the |
Note that in retrospect, while writing this I've realized it may be a better investment of effort to investigate a potential tailored solution to #1392 rather than attempting to compete with |
This keeps getting raised as an issue, e.g. #1392, #1607.
Ideally a custom command-line parsing implementation would use a comparable syntax to the C++ binaries, with the same capabilities (and possibly also remove some capabilities of
argparse
that are technically ambiguous according to the MRtrix3 command-line option ordering philosophy), with corresponding re-implementation of the help page / documentation generation, and would also require some kind of implementation that would satisfy what is currently achieved using Subparsers withinargparse
and thealgorithm
module.The text was updated successfully, but these errors were encountered: