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

[Certora] Remove munging #494

Merged
merged 5 commits into from
Sep 20, 2023
Merged

[Certora] Remove munging #494

merged 5 commits into from
Sep 20, 2023

Conversation

QGarchery
Copy link
Contributor

No description provided.

@QGarchery QGarchery added the verif Modifies the formal verification label Sep 15, 2023
@QGarchery QGarchery self-assigned this Sep 15, 2023
@QGarchery QGarchery force-pushed the certora/remove-munging branch from 957758d to b3c7038 Compare September 15, 2023 13:31
Copy link
Contributor

@MathisGD MathisGD left a comment

Choose a reason for hiding this comment

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

amazing!

@QGarchery
Copy link
Contributor Author

amazing!

Only one thing left to elucidate is why when run by the CI, it takes 1h41 min to complete, whereas when I run the script from my machine (processed by the same server), it takes something like 14 minutes. Both are on the exact same code, and on the same script

@MathisGD
Copy link
Contributor

The Certora team is probably able to help you

@QGarchery
Copy link
Contributor Author

QGarchery commented Sep 18, 2023

The Certora team is probably able to help you

Already on it :)

@QGarchery QGarchery merged commit abc93ac into certora/dev Sep 20, 2023
15 checks passed
@QGarchery QGarchery deleted the certora/remove-munging branch September 20, 2023 14:02
@QGarchery QGarchery mentioned this pull request Sep 20, 2023
@QGarchery QGarchery mentioned this pull request Oct 11, 2023
32 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
verif Modifies the formal verification
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants