-
Notifications
You must be signed in to change notification settings - Fork 52
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
UWA test fails on linux build #590
Comments
Interesting. We have a build script run on pull requests to do a Z3 build on a Linux server and run the unit tests, so this shouldn't happen. Any unusual configuration - hardware, Z3 version, etc? I'll do a CI run on master now to check. |
Shouldn't be anything unusual. I just built the thing using the plain instructions, as provided, on a fairly fresh install of Xubuntu. Might be I have a package somewhere that's on a newer version than vampire expects, or something?
|
Thanks for the details. That's really odd, you're slap-bang in the middle of what we test on. Could you run |
If anything you're on the newer side: maybe with 24.04 they're shipping a new compiler or something that's breaking us. I sure hope not, though. |
It's possible? It seems weird that only this one thing would break, but I don't know my C++ that well. |
Also see the text file in the first post - it has the terminal output for the failures, including a bunch of errors that popped up. |
And in case it's relevant, my GCC version:
|
@joe-hauns , would be able to glean what could be going on? (Especially from the UWATestErrors.txt mentioned earlier?) |
I'm trying to reproduce the bug. From the error logs i cannot tell what the issue is, but I can see that it's only triggered for higher order problems, which our current master isn't super stable for anyways. So if you're running on non-higher order there should probably not be any issues. not sure whether that helps.. |
I also don't run into any issues when running |
Hmm. Lemme try and see if I can upgrade gcc, then. :/ |
I am not surprised that we are hitting some issues with higher-order reasoning on master branch. As @joe-hauns mentioned, higher-order on current master is not very well supported. @Angular-Angel , if you are interested in running on higher-order problems, please use this branch for the time being. We are in the process of merging it into master, but for the time being it is the most stable branch to support higher-order I think. |
@Angular-Angel could you tell me a little more about the "intro problems from ASE 2017"? How can this problem set be accessed? |
It's right here! |
Somehow I wasn't even aware of these! These are certainly not higher-order, so we shoudn't be failing on them. |
This is the one it fails on - I think there's some higher order logic? :/ https://github.com/vprover/ase17tutorial/blob/master/intro/MSC023%3D2.p |
I built vampire, and was trying to run it on the intro problems from ASE 2017, and it failed on the Fahrenheit to Celsius conversion - so I built a dev version and ran the tests, and sure enough, the UWA test failed. I had the ASE into problem fail with both the z3 and non z3 versions, but I only tested the z3 version of vampire.
UWATestErrors.txt
The text was updated successfully, but these errors were encountered: