-
Notifications
You must be signed in to change notification settings - Fork 19
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
TPE Lean model #548
TPE Lean model #548
Conversation
Signed-off-by: Shaobo He <shaobohe@amazon.com>
Signed-off-by: Shaobo He <shaobohe@amazon.com>
Signed-off-by: Shaobo He <shaobohe@amazon.com>
Signed-off-by: Shaobo He <shaobohe@amazon.com>
Signed-off-by: Shaobo He <shaobohe@amazon.com>
Signed-off-by: Shaobo He <shaobohe@amazon.com>
Signed-off-by: Shaobo He <shaobohe@amazon.com>
Signed-off-by: Shaobo He <shaobohe@amazon.com>
Signed-off-by: Shaobo He <shaobohe@amazon.com>
Signed-off-by: Shaobo He <shaobohe@amazon.com>
Signed-off-by: Shaobo He <shaobohe@amazon.com>
Signed-off-by: Shaobo He <shaobohe@amazon.com>
(NoneIsTrue req.resource.asEntityUID λ resource ↦ | ||
instanceOfEntityType resource resource.ty env.ets.entityTypeMembers?) && | ||
(NoneIsTrue req.context λ m ↦ | ||
instanceOfType (.record m) (.record env.reqty.context) env.ets) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is because we perform typechecking first to obtain a TypedExpr prior to TPE, and bad actions are rejected at that point. Worth leaving a comment here to that effect.
Signed-off-by: Shaobo He <shaobohe@amazon.com>
Signed-off-by: Shaobo He <shaobohe@amazon.com>
Signed-off-by: Shaobo He <shaobohe@amazon.com>
Signed-off-by: Shaobo He <shaobohe@amazon.com>
Signed-off-by: Shaobo He <shaobohe@amazon.com>
Signed-off-by: Shaobo He <shaobohe@amazon.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great
commit 5e64bec Author: John Kastner <jkastner@amazon.com> Date: Thu Mar 6 18:56:34 2025 +0000 Extend typechecker inversion lemma for type annotation Signed-off-by: John Kastner <jkastner@amazon.com> commit a33231a Author: shaobo-he-aws <130499339+shaobo-he-aws@users.noreply.github.com> Date: Thu Mar 6 07:22:38 2025 -0800 TPE Lean model (#548) Signed-off-by: Shaobo He <shaobohe@amazon.com> commit 582a8cb Author: Adrian Palacios <73246657+adpaco-aws@users.noreply.github.com> Date: Wed Mar 5 11:48:14 2025 -0500 Cases for `datetime` types in `extExprToValue` (#568) Signed-off-by: Adrian Palacios <accorell@amazon.com> commit ee3715f Author: Adrian Palacios <73246657+adpaco-aws@users.noreply.github.com> Date: Tue Mar 4 16:05:17 2025 -0500 Add types and methods from `datetime` extension to parsers (#565) Signed-off-by: Adrian Palacios <accorell@amazon.com> commit 183be85 Author: Craig Disselkoen <cdiss@amazon.com> Date: Tue Mar 4 13:01:15 2025 -0800 [lean] remove JSON deserializer for internal JSON format (#567) Signed-off-by: Craig Disselkoen <cdiss@amazon.com> commit 71e13dc Author: Craig Disselkoen <cdiss@amazon.com> Date: Tue Mar 4 12:20:55 2025 -0800 remove partial-auth and partial-eval DRT targets (#566) Signed-off-by: Craig Disselkoen <cdiss@amazon.com> commit 434f09f Author: Craig Disselkoen <cdiss@amazon.com> Date: Tue Mar 4 06:04:25 2025 -0800 use Protobuf for additional DRT targets (#562) Signed-off-by: Craig Disselkoen <cdiss@amazon.com>
Issue #, if available:
Description of changes: