-
Notifications
You must be signed in to change notification settings - Fork 83
feat: Additional Pipeline Expressions #1115
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
feat: Additional Pipeline Expressions #1115
Conversation
bhshkh
left a comment
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.
Minor nits. Overall LGTM.
|
|
||
| @expose_as_static | ||
| def map_remove(self, key: str | Constant[str]) -> "Expr": | ||
| """Remove a key from a the map produced by evaluating this expression. |
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.
| """Remove a key from a the map produced by evaluating this expression. | |
| """Remove a key from the map produced by evaluating this expression. |
| >>> Field.of("address").map_remove("city") | ||
| Args: | ||
| key: The key to ewmove in the map. |
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.
| key: The key to ewmove in the map. | |
| key: The key to remove in the map. |
| >>> Expr.array(["bar", Field.of("baz")]) | ||
| Args: | ||
| elements: THe input list to evaluate in the expression |
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.
| elements: THe input list to evaluate in the expression | |
| elements: The input list to evaluate in the expression |
| >>> Expr.map({"foo": "bar", "baz": Field.of("baz")}) | ||
| Args: | ||
| elements: THe input dict to evaluate in the expression |
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.
| elements: THe input dict to evaluate in the expression | |
| elements: The input dict to evaluate in the expression |
|
Thanks, good catches! |
Adds remaining pipeline expressions