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

Princess: No StringFormulaManager #340

Open
mkuijn-cb opened this issue Nov 10, 2023 · 4 comments · May be fixed by #391
Open

Princess: No StringFormulaManager #340

mkuijn-cb opened this issue Nov 10, 2023 · 4 comments · May be fixed by #391
Assignees

Comments

@mkuijn-cb
Copy link

The constructor for PrincessFormulaManager passes null for parameter strManager upwards to AbstractFormulaManager. My understanding is that Princess does support strings and regex if combined with the OSTRICH library. Is there a reason why support for this is infeasible or is it just a matter of simply not being implemented yet.

@kfriedberger
Copy link
Member

There even exists an older PR for that topic: #257 . That PR was blocked by some issues with Princess. Someone might want to look at the current state of Princess and update the PR to a current state.

@baierd
Copy link
Collaborator

baierd commented Sep 5, 2024

@daniel-raffler could you take a look at this?

@daniel-raffler
Copy link
Contributor

daniel-raffler commented Sep 7, 2024

I've resurrected the branch from the PR (257-more-theories-for-princess) and updated it to the latest version of JavaSMT. It's now also running the latest version of Ostrich, which fixes one of the issues that was reported by @kfriedberger (this is the bug report).

However, there are unfortunately a number of issues left as can be seen from the results of the test suite here. Support for String theory seems to be missing some key operations that are needed for our tests. I'll open some new tickets for Ostrich later today and will then link them in this thread. Rationals also have some issues left and @kfriedberger has has already opened two tickets for Princess here and here when he last worked on this PR.

All in all Rationals look better than String Theory, but at this point neither seems to be quite ready for inclusion yet. We'll have to wait and see if the remaining issues can be fixed before merging this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging a pull request may close this issue.

4 participants