Extend TPE to support attribute level partial information - #2547
john-h-kastner-aws wants to merge 5 commits into
Conversation
a3b7318 to
c203887
Compare
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
c4452d3 to
387be0e
Compare
387be0e to
3c21bbe
Compare
Coverage ReportHead Commit: Base Commit: Download the full coverage report. Coverage of Added or Modified Lines of Rust CodeRequired coverage: 80.00% Actual coverage: 91.31% Status: PASSED ✅ Details
Coverage of All Lines of Rust CodeRequired coverage: 80.00% Actual coverage: 88.42% Status: PASSED ✅ Details
|
Signed-off-by: jkastner <jkastner@amazon.com>
Signed-off-by: jkastner <jkastner@amazon.com>
Signed-off-by: jkastner <jkastner@amazon.com>
Signed-off-by: jkastner <jkastner@amazon.com>
3c21bbe to
4594d04
Compare
Coverage ReportHead Commit: Base Commit: Download the full coverage report. Coverage of Added or Modified Lines of Rust CodeRequired coverage: 80.00% Actual coverage: 91.04% Status: PASSED ✅ Details
Coverage of All Lines of Rust CodeRequired coverage: 80.00% Actual coverage: 88.42% Status: PASSED ✅ Details
|
4594d04 to
b877242
Compare
Signed-off-by: jkastner <jkastner@amazon.com>
b877242 to
db3c840
Compare
Coverage ReportHead Commit: Base Commit: Download the full coverage report. Coverage of Added or Modified Lines of Rust CodeRequired coverage: 80.00% Actual coverage: 92.21% Status: PASSED ✅ Details
Coverage of All Lines of Rust CodeRequired coverage: 80.00% Actual coverage: 88.43% Status: PASSED ✅ Details
|
Description of changes
This is the core implementation of "TPE infinity" extension. It adds the internal implementation details required to support partial evaluation with attribute level partial information.
It doesn't include the public API, which will come in a PR on top of this for separate review.
The internal representation of partial values now supports the following states for attributes.
Issue #, if available
Checklist for requesting a review
The change in this PR is (choose one, and delete the other options):
cedar-policy(e.g., changes to the signature of an existing API).cedar-policy(e.g., addition of a new API).cedar-policy.cedar-policy-core,cedar-validator, etc.)I confirm that this PR (choose one, and delete the other options):
I confirm that
cedar-spec(choose one, and delete the other options):cedar-spec, and how you have tested that your updates are correct.)cedar-spec. (Post your PR anyways, and we'll discuss in the comments.)I confirm that
docs.cedarpolicy.com(choose one, and delete the other options):cedar-docs. PRs should be targeted at astaging-X.Ybranch, notmain.)