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

Temporarily disable changing Timeout to I_dont_know in satML #950

Merged
merged 1 commit into from
Nov 20, 2023

Conversation

bclement-ocp
Copy link
Collaborator

This is a hotfix while we figure out a proper way forward for #946 so that Alt-Ergo doesn't randomly crash due to timeouts in the meantime.

This is a hotfix while we figure out a proper way forward for
OCamlPro#946 so that Alt-Ergo doesn't
randomly crash due to timeouts in the meantime.
@bclement-ocp
Copy link
Collaborator Author

@Halbaroth @Stevendeo could I get a quick review on this one? Would be great to not have spurious crashes when running benchmarks on next (it fills up the sqlite database with garbage).

Longer term the solution is probably along the lines of #936 but for timeouts.

@Halbaroth Halbaroth merged commit 2dc85ad into OCamlPro:next Nov 20, 2023
14 checks passed
Halbaroth added a commit to Halbaroth/alt-ergo that referenced this pull request Nov 20, 2023
bclement-ocp pushed a commit that referenced this pull request Nov 20, 2023
@bclement-ocp bclement-ocp deleted the bclement/hotfix-timeout branch January 23, 2024 08:49
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