Skip to content

Conversation

@arnavk23
Copy link

@arnavk23 arnavk23 commented Nov 21, 2025

  • Adds human and machine-readable documentation for Uno options and a CLI flag to dump options for automated configuration tools.
  • Documents available runtime options in options.md
  • Adds --dump-options to the AMPL frontend to print option name, type and default (TSV).
    -Generates machine-readable exports (TSV, JSON, irace-style) under docs so tools like irace can consume defaults programmatically.

Related issues

#420 (Document the options)
#166 (Automatic configuration)

@arnavk23
Copy link
Author

@cvanaret Was just browsing around, found your repor and loved it! Tried to fix your documentation issue, take a look.

@cvanaret
Copy link
Owner

cvanaret commented Nov 21, 2025

Hi @arnavk23, thanks a lot for your contribution! You may have broken a few things though 😅
May I suggest that you add the function dump_default_options() in Options without changing the original formatting/indentation in Options.cpp and uno_ampl.cpp?

@arnavk23
Copy link
Author

@cvanaret All are passing except one and when I looked into it, I think it is not due to the change I added

@cvanaret
Copy link
Owner

@arnavk23 absolutely, this is an unrelated issue that appeared recently.
I've added a couple of comments on your submission.

@arnavk23
Copy link
Author

arnavk23 commented Nov 23, 2025

@arnavk23 absolutely, this is an unrelated issue that appeared recently. I've added a couple of comments on your submission.

@cvanaret Made the changes and returned back to the original formatting, please take a look now on the change proposed.

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;
Copy link
Owner

@cvanaret cvanaret Nov 24, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

" missing after true

@cvanaret
Copy link
Owner

cvanaret commented Nov 24, 2025

@cvanaret Made the changes and returned back to the original formatting, please take a look now on the change proposed.

Looks great, thanks! There's just a " missing (see comment).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants