-
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
Term printers #8
Comments
I'd say printing unicode in languages not designed for it is going to be tough. You can try "u1234" instead of the corresponding codepoint, though. |
To be clear, the point of escaping was precisely to not print unicode in languages that don't support it. Basically, the idea of escaping would be to map every unicode character to a list of unicode character (kind of like a I asked about that because, as I recall you objected to adding dependencies such as uutf and uucp, which means the printer would have to be part of a sub-package with additional dependencies. |
Regarding the question "should statements also be printable ?" I would say, Yes.
To be safe, I think we should't do this translations yet unless we know that it is safe to do. If you try to prety print features in one language that are not supported you should tell to the user exactly that.
For now, we may skip the print of the terms for iCNF and dimacs. Later on we may use a map as you sugested. |
@Gbury I will start with the pretty-printers. Do you want to provide some skeleton? |
I already started some work on branch |
This will be greatly benefit from the typed terms now that there is a type-checker. I'll upstream some code from archsat soon to cover this feature. |
See #211 for an implementation of printers for smtlib v2.6. |
Implement printers for terms in each language.
This brings some questions that need answers:
#1
,#2
, etc... So should we just not give printer for them, or have the printer maintain a map of terms to numbered propositional atoms ?cc @anmaped
cc @c-cube for the unicode-related escaping question
The text was updated successfully, but these errors were encountered: