-
Notifications
You must be signed in to change notification settings - Fork 17
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
Let dolmen fix that for you #65
Comments
Here is a list of things that come to mind:
|
Another example: some problems may contain solver-reserved identifiers, particularly when they are created from the output of some solvers (see e.g. https://clc-gitlab.cs.uiowa.edu:2443/SMT-LIB-benchmarks/QF_BV/-/blob/master/20190429-UltimateAutomizerSvcomp2019/byte_add_1_true-unreach-call_true-no-overflow_true-termination.i_6.smt2 ), it would be useful to be able to parse (and type) them to then print them with correct names. |
Does dolmen produce an error for those? I have some preliminary work on doing large scale back experiments on SMT-LIB and it would be a good opportunity to mark such. |
It's fairly recent addition indeed: PR #193 made it so that it is an error to use symbols that start with an It might be a problem for problems generated by solver-like programs indeed, and I should probably try and run a check on the whole smtlib to see whether this is a frequent occurrence. That being said, I think that such generated problems should respect the spec, since solvers might assume that all of the identifiers starting with an |
In the meantime, and depending on how many problems in the smtlib use such solver-reserved identifiers, I'll consider changing the error into a warning, which would be fatal by default, but that could be turned off/not fatal if wanted/required by users. |
Long-term thread to list the various little errors that dolmen could accept during typing, in order to eventually allow to "fix" problems once problem export in dolmen has been implemented. This would be done either when using
--strict=false
or even another option (--strict=fix
?).The text was updated successfully, but these errors were encountered: