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

Cost tracking for two-variable comprehensions and bindings #1104

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

TristonianJones
Copy link
Collaborator

@TristonianJones TristonianJones commented Jan 11, 2025

Cost tracking for two-variable comprehensions, cel.bind(), and optional_types

This update introduces the concept of a computed entry size for literals which helps
with cost propagation through chained computations. The cost-tracking has cannot
estimate further than one level of nesting within a literal value, but this should be suitable
for most cases.

This change also perform optional and dyn unwrapping, de-emphasizes dynamic size
computations by aggressively caching values though this be improved further for local
variables which require calls into the estimator.

Note, the cost tracking beyond a single level of depth for a literal is quite limited, but considerably improved from its former state.

@TristonianJones TristonianJones changed the title Cost tracking for two-variable comprehensions, cel.bind(), and optional_types Cost tracking for two-variable comprehensions and cel.bind() Jan 13, 2025
@TristonianJones TristonianJones changed the title Cost tracking for two-variable comprehensions and cel.bind() Cost tracking for two-variable comprehensions and bindings Jan 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant