-
Notifications
You must be signed in to change notification settings - Fork 6
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
Add a CustomCommandLineArgument interface #130
Comments
Would this enable also to programmatically to restrict enums? For example, with an use-case from HTSJDK, remove support for |
Yes, perhaps custom validation or parsing could be another use for this. |
The CommandLineProgram base class should include methods that handle some of the warning messages as implemented in broadinstitute/gatk#4429. Barclay should also include the TerminalColor some variant of class mentioned in broadinstitute/gatk#4429 (comment). |
Yeah. I'll add it there. |
This would be useful when the doc and possible values for an argument are not static, but derived programmatically from code. For example, an argument where the values are drawn from the cross product of two existing enums, or some subset thereof:
where the values of
crossProductArgs
are drawn from the set { "A1-B1", "A1-B2", "A2-B1", "A2-B2", "B1-A1", ... }, with hyphen as a separator.This link is to a private repository, but for those who can navigate to it, a real life (and slightly more complicated) version of this came up in https://github.com/broadinstitute/picard-private/pull/1218.
The text was updated successfully, but these errors were encountered: