generated from amazon-archives/__template_Apache-2.0
-
Notifications
You must be signed in to change notification settings - Fork 135
Open
Labels
internal-improvementRefactoring, minor performance improvement, or other changes that Cedar users may never noticeRefactoring, minor performance improvement, or other changes that Cedar users may never notice
Description
Describe the improvement you'd like to request
The LocalSolver implementation in cedar-policy-symcc/src/symcc/solver.rs extracts solver models by looking for a specific text shape "(\n<the actual model>\n)\n".
A more robust implementation could instead parse the S-expression returned by the solver.
Reusing the implementation in decoder.rs would also make it easier to have a single implementation of what the solver output is expected to be.
Describe alternatives you've considered
Leave the lightweight implementation in place, since only CVC5 models that follow this convention are supported.
Additional context
No response
Is this something that you'd be interested in working on?
- ๐ I may be able to implement this internal improvement
-
โ ๏ธ This feature might incur a breaking change
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
internal-improvementRefactoring, minor performance improvement, or other changes that Cedar users may never noticeRefactoring, minor performance improvement, or other changes that Cedar users may never notice