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

Intergrate CI #24

Draft
wants to merge 9 commits into
base: master
Choose a base branch
from
Draft

Intergrate CI #24

wants to merge 9 commits into from

Conversation

VietAnh1010
Copy link
Collaborator

Merge changes from ci branch into master.

@VietAnh1010
Copy link
Collaborator Author

We need to cache the dependencies - installing them takes too long!

Why3 on my local machine is 1.7.2. Bump to 1.8.0 and make sure
that the build works.
The problem is inside `simplify_pure`, which is quite surprising.
The problem is that, `simplify_pure` will keep simplify our formula
until convergence, which is denoted by a boolean. There is a mistake
somewhere with the boolean, such that the loop never exits.

The current fix is to bound the loop with another parameter "fuel",
which is decreased by 1 after each iteration. Once the fuel is exhausted
we will stop the loop (this is a technique that is used in Coq to write
non-structurally recursive function).

By the way, the way we write "simplify_term" and "simplify_pure" is too
messy, but it is not our top priority now to fix that.
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.

2 participants