-
Notifications
You must be signed in to change notification settings - Fork 284
Update CaDiCaL from 2.0.0 to 3.0.0 #8736
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
base: develop
Are you sure you want to change the base?
Conversation
2f01b69 to
0fe1341
Compare
0fe1341 to
183bed1
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #8736 +/- ##
========================================
Coverage 80.00% 80.00%
========================================
Files 1700 1700
Lines 188257 188257
Branches 73 73
========================================
Hits 150615 150615
Misses 37642 37642 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
TGWDB
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be nicer to review if the separate changes/fixes were split into separate commits, but not worth retroactively fixing.
Thank you, feedback noted. On this occasion I wasn't entirely sure along which axis to split them up for they were intimately tied to the version (and: ensuing API) change. But I do acknowledge that, as of late, I have starting cramming more changes into single commits, I'll try to adjust my course on that. |
Update to the latest release.
183bed1 to
7f0322d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR updates the CaDiCaL SAT solver dependency from version 2.0.0 to version 3.0.0. The update includes adaptations to API changes in the newer version, build system updates to accommodate new source files and compiler flags, and commented-out code for potential future features related to bounded variable addition.
- API changes in CaDiCaL requiring updates to value retrieval logic
- Build system modifications to support C source files and additional compiler flags
- New patch file for version 3.0.0 to handle VERSION file renaming
Reviewed changes
Copilot reviewed 7 out of 8 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| src/solvers/sat/satcheck_cadical.h | Adds commented-out declarations for new_variable and new_variables methods |
| src/solvers/sat/satcheck_cadical.cpp | Updates API calls from val(dimacs()) to val(var_no(), true) with manual sign handling; adds commented-out implementations of variable creation methods; disables bounded variable addition feature |
| src/solvers/Makefile | Updates version to 3.0.0, adds -DNCLOSEFROM flag and filters out -Wswitch-enum warning |
| src/solvers/CMakeLists.txt | Updates download URL and patch file reference from version 2.0.0 to 3.0.0, updates MD5 checksum |
| src/Makefile | Updates version reference and patch file from 2.0.0 to 3.0.0 |
| scripts/cadical_CMakeLists.txt | Adds "src/*.c" pattern to include C source files alongside C++ files |
| scripts/cadical-3.0.0-patch | New patch file for version 3.0.0 to rename VERSION to VERSION.txt |
| .github/workflows/pull-request-checks.yaml | Adds -DCMAKE_C_FLAGS=-m32 to ensure C files are compiled with 32-bit flag |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Updates to the latest CaDiCaL release.
7f0322d to
fffb703
Compare
Also enable checksum-checking in Makefile-based builds.
fffb703 to
afed131
Compare
Update to the latest release.