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

missing method to map back the solution #7

Closed
ale-depi opened this issue Mar 1, 2023 · 2 comments
Closed

missing method to map back the solution #7

ale-depi opened this issue Mar 1, 2023 · 2 comments

Comments

@ale-depi
Copy link

ale-depi commented Mar 1, 2023

Hi @meelgroup

There is no way to map back the solution found by a SAT solver to the original problem. The tool is very useful because simplifies a lot CNFs but sometimes it is needed the interpretation rather than just the satisfiability.

This could help since CryptoMiniSat do not support anymore the preprocessing as one could see from this.

Remarks.

  • In the Arjun README at the end of "How to Build" commands it could be useful an ldconfig.
  • Since you are suggesting sudo make install, also ./arjun ... can be just arjun ...
  • I see that you are also the developer of ApproxMC. The above suggestion for ldconfig works as well.
  • Note that in the section How to use the Python interface, the double ** is get as a Markdown delimiter messing the text.

Thank you for your work.

Alessandro

msoos added a commit that referenced this issue Mar 3, 2023
@msoos
Copy link
Collaborator

msoos commented Mar 3, 2023

Thanks for spotting these and opening an issue! I have written a mini-description of what's needed in #8 for mapping back solutions. I have remedied the sudo ldconfig missing in the README. I also did this for ApproxMC. I also fixed the ** :) I'm going to keep open #8, but close this because I believe everything you mentioned has now been addressed except #8. Please do re-open it in case I'm wrong!

Thanks again,

Mate

@msoos msoos closed this as completed Mar 3, 2023
@msoos
Copy link
Collaborator

msoos commented Mar 6, 2023

Please check #8 to see how the new back-mapping works. Please check if it works for you.

Cheers,

Mate

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

2 participants