Skip to content

Conversation

KristofferC
Copy link
Member

@KristofferC KristofferC commented Jul 26, 2025

This makes the SAT resolver (https://github.com/StefanKarpinski/Resolver.jl) a new default (so that people actually try it). We still need to make libpicosat available from julia itself (JuliaLang/julia#59119).

This adds an extra CI run that runs with the maxsum resolver by default so that we properly test that.

@github-project-automation github-project-automation bot moved this to New in Pkg.jl Jul 26, 2025
@IanButterworth IanButterworth moved this from New to In review in Pkg.jl Jul 26, 2025
@IanButterworth IanButterworth moved this from In review to In progress in Pkg.jl Jul 26, 2025
@KristofferC KristofferC requested a review from a team as a code owner July 26, 2025 17:37
@DilumAluthge DilumAluthge removed the request for review from a team July 26, 2025 17:42
@KristofferC KristofferC added the DO NOT MERGE Do not merge this PR! label Jul 26, 2025
@DilumAluthge
Copy link
Member

Should we make this "pluggable"? E.g., instead of special-casing StefanKarpinski/Resolver.jl, could we have a way for the user to provide a Resolver struct, and then Pkg uses that? Obviously we'd need to write out the interface spec for Resolver objects. But that would allow anyone to experiment with their own Resolver and then drop it in?

@KristofferC
Copy link
Member Author

I think that is kind of a premature generalisation.

@KristofferC KristofferC changed the title WIP: integrate the SAT-based resolver into Pkg integrate the SAT-based resolver into Pkg Jul 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DO NOT MERGE Do not merge this PR!
Projects
Status: In progress
Development

Successfully merging this pull request may close these issues.

2 participants