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

Add support for options which are sets #35

Open
ewu63 opened this issue Jan 19, 2021 · 0 comments
Open

Add support for options which are sets #35

ewu63 opened this issue Jan 19, 2021 · 0 comments
Labels
enhancement New feature or request

Comments

@ewu63
Copy link
Collaborator

ewu63 commented Jan 19, 2021

Description of feature

Similar to the list of choices, we should also allow for sets. This would be useful for things like monitorVariables in ADflow, where we want the default options to list out all the possible values. The user options would then be checked that it is a subset of the full set. This would also allow for a description of each value in the options table.

One question is whether these values should be ordered. For safety under parallel execution, we probably want to cast the final set (after error checking) into a sorted list, in case the solvers expect a list to iterate over.

@ewu63 ewu63 added the enhancement New feature or request label Jan 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant