-
Notifications
You must be signed in to change notification settings - Fork 8
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
Performance issues with colibri2 installed #212
Comments
I can look into it, how do I reproduce this? |
Great! There's been a couple changes. So I'll try to be as thorough as possible when describing how to reproduce this. 1. Setup benchmarking environment
2. Reproducing the first results
3. Reproducing the second results
4. Verifying the performance hit (optional)
SummaryI think the slowdown is probably due to us creating some terms at the top level of the module, which is going to have a startup cost and will end up accumulating across multiple benchmarks. It would be good if we could somehow reduce this, but we can probably account for the startup cost by running an empty benchmark and then adjusting the runtimes by subtracting the startup time. |
Thanks for the detailed response! |
Thanks for #216. Do you also have any insights why the time for |
I am not sure tbh. (Also there are still unknown in the bench marks, but that's just colibri2 lacking in reasoning power, will see if I can easily do something about it) |
I'm not sure what happens when both z3 and colibri2 are installed together, but that's when the performance issue occurs, it also seems to occur with When z3, colibri2 and bitwuzla are installed:
When z3 and bitwuzla are installed:
When colibri2 and bitwuzla are installed:
|
Yeah, I agree it might be something related with the dependencies. But, I don't think we will have time to address this for the submission on friday so we're currently evaluating solvers in separate opam switches so that their dependencies don't interfere with each other. |
Some performance issues noticed when colibri2 is installed. Probably due to additional start up costs?
With colibri2 installed:
The text was updated successfully, but these errors were encountered: