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

Fingerprints seemingly ignored for some proofs #85

Closed
ahelwer opened this issue Jun 6, 2023 · 5 comments
Closed

Fingerprints seemingly ignored for some proofs #85

ahelwer opened this issue Jun 6, 2023 · 5 comments

Comments

@ahelwer
Copy link
Collaborator

ahelwer commented Jun 6, 2023

Using the 202210041448 release, the following proofs in the tlaplus/examples repo take a long time to check regardless of whether they were previously checked and their fingerprints stored in the .tlacache directory:

specifications/LoopInvariance/Quicksort.tla
specifications/LoopInvariance/SumSequence.tla
specifications/bcastByz/bcastByz.tla

This proof only takes six seconds to check but it still takes six seconds to check even if its fingerprints exist:

specifications/MisraReachability/ReachabilityProofs.tla

To help with differentiation, checking the following proofs is correctly greatly sped up when their fingerprints exist, going from ~10 seconds to ~0.5 seconds:

specifications/LoopInvariance/BinarySearch.tla
specifications/MisraReachability/ReachableProofs.tla
specifications/TeachingConcurrency/SimpleRegular.tla
@lemmy
Copy link
Member

lemmy commented Jun 6, 2023

Possibly related #84

@ahelwer
Copy link
Collaborator Author

ahelwer commented Jun 6, 2023

Wow weird coincidence, I had been sitting on this issue for months!

@ahelwer
Copy link
Collaborator Author

ahelwer commented Aug 30, 2023

Oops reopening because the PR isn't yet merged I guess

@ahelwer ahelwer reopened this Aug 30, 2023
@kape1395
Copy link
Collaborator

#84 is merged now. Can we close this issue?

@ahelwer
Copy link
Collaborator Author

ahelwer commented Aug 28, 2024

Probably!

@ahelwer ahelwer closed this as completed Aug 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

3 participants