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

Mcsat check model with hint API method #472

Merged
merged 5 commits into from Nov 4, 2023
Merged

Conversation

ahmed-irfan
Copy link
Contributor

  • Adds the new api method
  • Adds a test to check the api method
  • Updates the documentation

@coveralls
Copy link

coveralls commented Nov 1, 2023

Coverage Status

coverage: 65.066% (-0.03%) from 65.098% when pulling bdd5e40 on mcsat-check-model-with-hint into 30dd2c1 on master.

mcsat_value_destruct(&value);
}

mcsat_pop(mcsat);
Copy link
Contributor

Choose a reason for hiding this comment

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

Does the use of push/pop mechanism mean that we lose all of the lemmas we have learnt prior to the call to yices_check_context_with_model_and_hint ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No, we will not lose the lemmas learned before calling the yices_check_context_with_model_and_hint method. The mcsat_pop method removes those lemmas that were learned after the last call to mcsat_push. Here we don't remove any lemmas because we didn't learn any between the push/pop call.

Copy link
Contributor

Choose a reason for hiding this comment

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

OK, awesome.

@disteph disteph merged commit 827cd9c into master Nov 4, 2023
24 of 26 checks passed
@ahmed-irfan ahmed-irfan deleted the mcsat-check-model-with-hint branch November 4, 2023 02:02
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

Successfully merging this pull request may close these issues.

None yet

3 participants