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

Update dependency: deps/pyk_release #2041

Merged
merged 49 commits into from
Aug 31, 2023

Conversation

rv-jenkins
Copy link
Contributor

@rv-jenkins rv-jenkins commented Aug 24, 2023

raise ValueError(
'Simplifying initial node led to #Bottom, are you sure your LHS is defined?'
)
if is_bottom(_new_target):
if new_target.top():
Copy link
Contributor

@PetarMax PetarMax Aug 30, 2023

Choose a reason for hiding this comment

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

.bottom()?

Copy link
Contributor

Choose a reason for hiding this comment

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

Yep, that's the new signature of the CTerm and should be slightly more clever to be able to handle #Bottom #And #Bottom recursively

Copy link
Contributor

Choose a reason for hiding this comment

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

Okay, I see what you mean now @PetarMax :D Thanks

kevm-pyk/src/kevm_pyk/__main__.py Outdated Show resolved Hide resolved
kevm-pyk/src/tests/integration/test_prove.py Outdated Show resolved Hide resolved
@dwightguth
Copy link
Collaborator

Can someone please let me know when this is merged? My maude backend work is blocked on this.

@rv-jenkins rv-jenkins merged commit 2709f49 into master Aug 31, 2023
14 checks passed
@rv-jenkins rv-jenkins deleted the _update-deps/runtimeverification/pyk branch August 31, 2023 21:41
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.

6 participants