Skip to content

OR Tools#2146

Open
sjanzou wants to merge 60 commits intodevelopfrom
ortools_build
Open

OR Tools#2146
sjanzou wants to merge 60 commits intodevelopfrom
ortools_build

Conversation

@sjanzou
Copy link
Collaborator

@sjanzou sjanzou commented Feb 6, 2026 •
edited
Loading

https://github.com/NatLabRockies/SAM-private/pull/140

Final issues being worked out on failing Windows runner...

sjanzou added 30 commits April 25, 2025 03:18
@sjanzou
C++20 updates for ortools
97e3df4
@sjanzou
Add Alt+F5 to run the lib_ptes_chp_dispatch_test test for ortools
7126714
@sjanzou
2025.4.29 ssc 303 OR Tools Windows beta release Alt+F6 to test
c72c628
@sjanzou
Merge branch 'patch' into ortool__full_release
e33a4a0
@sjanzou
SAMOS building and running in release mode with release build of orto…
94706c4 
…ols on Windows
@sjanzou
Working on release versions for wxWidgets libraries
2c6d69f
@sjanzou
wxWidgets in top level - all but export_config building on Windows
1b34342
@sjanzou
Merge remote-tracking branch 'origin/develop' into ortools_build
26ccbda
@sjanzou
Fix broken export_config post build command with ortools dependencies…
c4d1fa1 
… on Windows
@sjanzou
All 44 projects on Windows 11 building with ortools v9.12
7c325ba
@sjanzou
Merge remote-tracking branch 'origin/develop' into ortools_build
b105c0b
@sjanzou
lk script sample update before merging
2860965
@sjanzou
Merge branch 'develop' into ortools_build
3dd3fa1
@sjanzou
Update Ubuntu runner for ortools
275ea9d
@sjanzou
add ortools version and release numbers to workflow
d5e26c2
@sjanzou
Update ssc build path for Ubuntu runner
9e810ee
@sjanzou
Skip google tests for ssc build for SAM for Ubuntu runner
529e684
@sjanzou
update workflow for all runners
69ae2e4
@sjanzou
updated workflow for all runners and test results from Windows
1700983
@sjanzou
Merge remote-tracking branch 'origin/develop' into ortools_build
3abc478
@sjanzou
updated api files
e1e154b
@sjanzou
Update relative header link to ortools folder for GitHub Actions Linu…
1b58d3f 
…x runner
@sjanzou
Update ortools target_link_libraries and include path for GA runners
7839765
@sjanzou
Update issue with ortools package linking for GitHub Actions runners
789f6ef
@sjanzou
Update for binary ortools download for GitHub runners - all building …
d27853a 
…locally on Windows
@sjanzou
C++20 updates for Linux runners
cec2e6f
@sjanzou
Fix issue for test ortools function in main.cpp for GA runners OR_PRO…
568e2ec 
…TO_DLL
@sjanzou
Add ortools library to linker command for Linux runners
db36b06
@sjanzou
Update target_link_libraries for ortools
8ccbbbb
@sjanzou
Fix GitHub Actions syntax error in CMakeLists
a5a2fd6
@sjanzou sjanzou self-assigned this Feb 6, 2026
brtietz
brtietz requested changes Feb 11, 2026
View reviewed changes
Copy link
Collaborator

@brtietz brtietz left a comment

Choose a reason for hiding this comment

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

Looks like the version number is out of date. Other than that it looks good.

@sjanzou
Update all c++ cmake commands to be consistent with ortools v9.14
f8a03c3 
if(MSVC)
  set(CMAKE_CXX_STANDARD 20)
else()
  set(CMAKE_CXX_STANDARD 17)
endif()
@sjanzou sjanzou requested a review from brtietz February 16, 2026 10:00
@brtietz brtietz self-requested a review March 4, 2026 22:12
@brtietz
Copy link
Collaborator

brtietz commented Mar 4, 2026

Need to update post-build event in SAM repo to make sure that code generator brings in the or-tools dlls.

@sjanzou
Update code generator to include 9 required libraries for ssc to run
6ae1968 
 "abseil_dll", "bz2", "highs", "libprotobuf", "libscip", "libutf8_validity", "ortools", "re2", "ssc", "zlib1" in CodeGen_Base::PlatformFiles
Tested on Windows with VBA MSPT SO and MP and PVWatts
@sjanzou
Copy link
Collaborator Author

sjanzou commented Mar 5, 2026

Need to update post-build event in SAM repo to make sure that code generator brings in the or-tools dlls.

@qualand, please review the code generator changes...

The 9 ortools libraries required for ssc to run are
"abseil_dll", "bz2", "highs", "libprotobuf", "libscip", "libutf8_validity", "ortools", "re2", "ssc", "zlib1"

I tested the MSPT SO and MP in Excel (VBA) and matched the SAM results, e.g.

image

@qualand
Copy link
Collaborator

qualand commented Mar 9, 2026

Need to update post-build event in SAM repo to make sure that code generator brings in the or-tools dlls.

@qualand, please review the code generator changes...

The 9 ortools libraries required for ssc to run are "abseil_dll", "bz2", "highs", "libprotobuf", "libscip", "libutf8_validity", "ortools", "re2", "ssc", "zlib1"

I tested the MSPT SO and MP in Excel (VBA) and matched the SAM results, e.g.

image

I tested LK for SDKTool code generation and SAM results matched (using OR-Tools).

image

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

Reviewers

@cpaulgilman cpaulgilman Awaiting requested review from cpaulgilman

@janinefreeman janinefreeman Awaiting requested review from janinefreeman

@qualand qualand Awaiting requested review from qualand

@tyneises tyneises Awaiting requested review from tyneises

@brtietz brtietz Awaiting requested review from brtietz

Assignees

@sjanzou sjanzou

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@sjanzou @brtietz @qualand