Skip to content

Rebuild C++ API for Gurobi#3714

Merged
boegel merged 1 commit intoeasybuilders:developfrom
sven-hansen:20250508144651_new_pr_gurobi
May 22, 2025
Merged

Rebuild C++ API for Gurobi#3714
boegel merged 1 commit intoeasybuilders:developfrom
sven-hansen:20250508144651_new_pr_gurobi

Conversation

@sven-hansen
Copy link
Copy Markdown
Contributor

@sven-hansen sven-hansen commented May 8, 2025

(created using eb --new-pr)

We have had several users that faced problems using Gurobi's C++ API with our modules. This PR rebuilds the C++ library using the current toolchain's compiler as suggested by https://support.gurobi.com/hc/en-us/articles/360039093112-How-do-I-resolve-undefined-reference-errors-while-linking-Gurobi-in-C

I had this in the drawer for a while and figured it would be worth contributing.

@sven-hansen sven-hansen changed the title $TITLE Rebuild C++ API for Gurobi May 8, 2025
@sven-hansen
Copy link
Copy Markdown
Contributor Author

Test report by @sven-hansen

Overview of tested easyconfigs (in order)

  • SUCCESS Gurobi-11.0.0-GCCcore-12.3.0.eb
  • SUCCESS Gurobi-11.0.2-GCCcore-12.3.0.eb
  • SUCCESS Gurobi-10.0.1-GCCcore-11.3.0.eb

Build succeeded for 3 out of 3 (3 easyconfigs in total)
n23t0001.hpc.itc.rwth-aachen.de - Linux Rocky Linux 9.5, x86_64, Intel(R) Xeon(R) Platinum 8468 (sapphirerapids), Python 3.9.21
See https://gist.github.com/sven-hansen/4061c2e1b7f35084218e4e721f18a04e for a full test report.

@boegel boegel added the bug fix label May 21, 2025
@boegel boegel added this to the next release (5.1.0) milestone May 21, 2025
@boegel
Copy link
Copy Markdown
Member

boegel commented May 22, 2025

Test report by @boegel

Overview of tested easyconfigs (in order)

  • SUCCESS Gurobi-9.5.2-GCCcore-11.3.0.eb
  • SUCCESS Gurobi-10.0.1-GCCcore-12.2.0.eb
  • SUCCESS Gurobi-11.0.2-GCCcore-12.3.0.eb
  • SUCCESS Gurobi-12.0.1-GCCcore-13.3.0.eb

Build succeeded for 4 out of 4 (4 easyconfigs in total)
node3625.doduo.os - Linux RHEL 9.4, x86_64, AMD EPYC 7552 48-Core Processor (zen2), Python 3.9.18
See https://gist.github.com/boegel/bb9a2592f7f08b8a35f26e7433d7c8d8 for a full test report.

Copy link
Copy Markdown
Member

@boegel boegel left a comment

Choose a reason for hiding this comment

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

lgtm

@boegel boegel merged commit ed9324d into easybuilders:develop May 22, 2025
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants