-
-
Notifications
You must be signed in to change notification settings - Fork 37
docs: add options documentation + CLI dump and machine-readable exports #450
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
base: main
Are you sure you want to change the base?
Conversation
|
@cvanaret Was just browsing around, found your repor and loved it! Tried to fix your documentation issue, take a look. |
|
Hi @arnavk23, thanks a lot for your contribution! You may have broken a few things though 😅 |
|
@cvanaret All are passing except one and when I looked into it, I think it is not due to the change I added |
|
@arnavk23 absolutely, this is an unrelated issue that appeared recently. |
Updated comments for clarity and consistency.
uno/options/Options.cpp
Outdated
| case OptionType::BOOL: | ||
| std::cout << "BOOL\t"; | ||
| if (defaults.bool_options.find(option_name) != defaults.bool_options.end()) | ||
| std::cout << (defaults.bool_options.at(option_name) ? "true; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
" missing after true
Looks great, thanks! There's just a |
options.md--dump-optionsto the AMPL frontend to print option name, type and default (TSV).-Generates machine-readable exports (TSV, JSON, irace-style) under
docsso tools like irace can consume defaults programmatically.Related issues
#420 (Document the options)
#166 (Automatic configuration)