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

fix xpress custom install #734

Closed
wants to merge 3 commits into from
Closed

Conversation

a-zakir
Copy link
Contributor

@a-zakir a-zakir commented Dec 15, 2023

No description provided.

Watermelon AI Summary

AI Summary deactivated by a-zakir

GitHub PRs

No results found in Jira Tickets :(

No results found in Confluence Docs :(

No results found in Slack Threads :(

No results found in Notion Pages :(

No results found in Linear Tickets :(

No results found in Asana Tasks :(

antares-xpansion is an open repo and Watermelon will serve it for free.
🍉🫶

Copy link

@@ -336,7 +336,7 @@ bool initXpressEnv(bool verbose, int xpress_oem_license_key) {
return false;
}
} else {
xpresspath = xpress_from_env;
xpresspath = (std::filesystem::path(xpress_from_env) / "bin").string();
Copy link
Contributor

Choose a reason for hiding this comment

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

The Xpress documentation advises to call XPRSinit (line 351) with a NULL argument instead of xpresspath. And if we have to use xpresspath it should be preferable to use the environment variable XPAUTH_PATH. As XPAUTH_PATH gives the full path of the license file, we should just get the directory from the full path.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

the XPRSinit is called with the null argument when the solver is really called(see SolverXpress.cpp)
this function searches Xpress lib :

  • in "standard" place like /opt/xpressmp
  • by looking in the env variable XPRESSDIR
    the whole thing is to not be tied with an xpress version so no need to set the XPAUTH_PATH from xpvars.sh script.
    there is a scenario where the license and binaries are in different places, how would you get the lib, bin from license file?

@JasonMarechal25
Copy link
Contributor

No change left. Changes were integrated in another PR

@JasonMarechal25 JasonMarechal25 deleted the fix/xpress-custom-install branch February 29, 2024 15:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants