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

SAT Competition #5

Open
ahorn opened this issue Mar 3, 2017 · 4 comments
Open

SAT Competition #5

ahorn opened this issue Mar 3, 2017 · 4 comments

Comments

@ahorn
Copy link

ahorn commented Mar 3, 2017

Hi,

Out of curiosity, have you considered submitting minisat-rust to the upcoming SAT Competition? If nothing else, it may provide some interesting data points how minisat-rust compares to the C++ implementation of minisat. It might also stir up some more interest in your project, although I don't know for sure of course. Either way, it's just a thought.

All best wishes,
A

@mishun
Copy link
Owner

mishun commented Mar 5, 2017

It's probably not very interesting to submit it as is because it is designed to generate the same output verbatim as original minisat for same input file and also still lacks a few important implementation details like proper clause allocation (Rust's heap API is still unstable). I actually thought about adding a few features from more modern solvers and submitting it just to show that "Rust can do it too", but, again, clause allocation, and me not being any kind of modern SAT solver expert (my experience with minisat is from my M.Sc. days more than 7 years ago).

@dralley
Copy link
Contributor

dralley commented Aug 8, 2019

Looks like the API has stabilized

@ahorn
Copy link
Author

ahorn commented Aug 9, 2019

For those interested, clause allocation in Rust is also discussed by @jix in a recent blog post.

@mishun
Copy link
Owner

mishun commented Aug 12, 2019

Yeah, I've seen std::alloc. I actually implemented custom allocator that is very similar to original minisat's one just a few days ago:
https://github.com/mishun/minisat-rust/blob/master/src/sat/formula/allocator.rs
Although, it still takes a few bytes per clause more

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants