Skip to content

build(evalBranchQuota): dynamically calculate a minimum value for evalBranchQuota#181

Merged
Arwalk merged 1 commit into
masterfrom
feature/dynamic_evalbranchquota
Jun 21, 2026
Merged

build(evalBranchQuota): dynamically calculate a minimum value for evalBranchQuota#181
Arwalk merged 1 commit into
masterfrom
feature/dynamic_evalbranchquota

Conversation

@Arwalk

@Arwalk Arwalk commented Jun 20, 2026

Copy link
Copy Markdown
Owner

@setEvalBranchQuota accepts any comptime value. That means it can be computed at comptime too. By (ab)using this, we can parse the protobuf structures at the entry point of all public functions and set a value for @setEvalBranchQuota depending on the number of fields that can be reached in a structure and its sub structures.

closes #152

…lBranchQuota

@setEvalBranchQuota accepts any comptime value. That means it can be computed at comptime too. By (ab)using this, we can parse the protobuf structures at the entry point of all public functions and set a value for @setEvalBranchQuota depending on the number of fields that can be reached in a structure and its sub structures.

closes #152
@Arwalk Arwalk requested a review from menduz June 20, 2026 20:30
@menduz

menduz commented Jun 21, 2026

Copy link
Copy Markdown
Collaborator

I think the gauss curve meme applies for this PR

@Arwalk Arwalk merged commit 97fd323 into master Jun 21, 2026
14 checks passed
@Arwalk

Arwalk commented Jun 21, 2026

Copy link
Copy Markdown
Owner Author

I think the gauss curve meme applies for this PR

I won't ask on which side of the curve i am, just to be safe.

@Arwalk Arwalk deleted the feature/dynamic_evalbranchquota branch June 21, 2026 15:32
@Arwalk

Arwalk commented Jun 21, 2026

Copy link
Copy Markdown
Owner Author

Fine, here it is.
image

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.

Handle eval branches quota more gracefully

2 participants