Skip to content

The Solvers section on the https://www.minizinc.org/downloads/ page is inaccurate #75

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

Closed
yurivict opened this issue May 26, 2024 · 1 comment

Comments

@yurivict
Copy link

yurivict commented May 26, 2024

It lists solvers as included that aren't required by cmake.

A number of solvers are included out of the box:

    [Gecode](https://www.gecode.org/)
    [Chuffed](https://github.com/chuffed/chuffed)
    [Google OR-Tools CP-SAT](https://developers.google.com/optimization)
    [HiGHS](https://highs.dev/)
    [COIN-OR CBC](https://github.com/coin-or/Cbc)
  • Google OR-Tools isn't looked for by the cmake scripts.
  • Chuffed is looked for but without the REQUIRED keyword which makes it not really included.
  • Atlantis is also present in the cmake script without REQUIRED but isn't listed anywhere in any lists.

I suggest that only 2-3 solvers are left in the "included-out-of-the-box" list (for example Gecode, COIN-OR CBC). Such solvers should have corresponding find_package(xx REQUIRED).

More rare solvers should be made optional, not listed anywhere in cmake, and only listed in the MiniZinc can also interface with the following solvers if installed: list.

It's also not clear what is the difference between "Additionally, many other compatible solvers can be installed, including:" and "MiniZinc can also interface with the following solvers if installed:" lists.

@yurivict yurivict changed the title The Downloads section on the https://www.minizinc.org/downloads/ page is inaccurate The Solvers section on the https://www.minizinc.org/downloads/ page is inaccurate May 26, 2024
@guidotack
Copy link
Member

The text on this page refers to the binary packages, which include the solvers listed there (and it includes pre-built interfaces to the second list of solvers). We will update it to clarify that the source code repository doesn’t include those solver out of the box.

@cyderize cyderize transferred this issue from MiniZinc/libminizinc May 28, 2024
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

No branches or pull requests

2 participants