Skip to content

Allow specifying lemmas in pyks KProve.proveClaim#2434

Merged
rv-jenkins merged 10 commits intomasterfrom
pyk-prove-lemmas
Feb 15, 2022
Merged

Allow specifying lemmas in pyks KProve.proveClaim#2434
rv-jenkins merged 10 commits intomasterfrom
pyk-prove-lemmas

Conversation

@ehildenb
Copy link
Copy Markdown
Member

@ehildenb ehildenb commented Feb 12, 2022

Part of: https://github.com/runtimeverification/ksummarize/issues/39

Blocked on: #2438

Thanks to #2345 we are able to add lemmas to the specification module.

This PR:

  • Allows threading through list of lemmas to pyk's KProve.proveClaim() method.
  • Adds a test to demonstrate a failing proof that passes when a lemma is added.
  • Prints out rules/claims without all the extra space between rule/claim body and attributes if the requires/ensures clauses are empty.

@ehildenb ehildenb requested a review from tothtamas28 February 12, 2022 19:08
@ehildenb ehildenb self-assigned this Feb 12, 2022
Comment thread k-distribution/tests/pyk/emit-json-spec-tests/emit_json_spec_test.py Outdated
@ehildenb ehildenb requested a review from tothtamas28 February 15, 2022 17:11
Copy link
Copy Markdown
Contributor

@tothtamas28 tothtamas28 left a comment

Choose a reason for hiding this comment

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

LGTM (except a few small issues, see below).

Comment thread k-distribution/tests/pyk/simple_proofs_test.py Outdated
Comment thread k-distribution/tests/pyk/simple_proofs_test.py Outdated
ehildenb and others added 2 commits February 15, 2022 17:31
@rv-jenkins rv-jenkins merged commit cba885f into master Feb 15, 2022
@rv-jenkins rv-jenkins deleted the pyk-prove-lemmas branch February 15, 2022 18:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants