Skip to content

starknet_api: copy resource bounds from mempool test utils #9748

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: arni/test_utils/remove_consts_from_mempool_test_utils
Choose a base branch
from
Open

starknet_api: copy resource bounds from mempool test utils #9748

wants to merge 1 commit into from

Conversation

@ArniStarkware
Copy link
Contributor

No description provided.

@reviewable-StarkWare
Copy link

This change is Reviewable

@github-actions
Copy link

github-actions bot commented Oct 23, 2025 •
edited
Loading

Artifacts upload workflows:

@ArniStarkware Graphite App
Copy link
Contributor Author

ArniStarkware commented Oct 23, 2025 •
edited
Loading

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

This was referenced Oct 23, 2025
Open
Open
@ArniStarkware ArniStarkware marked this pull request as ready for review October 23, 2025 17:38
@github-actions
Copy link

github-actions bot commented Oct 23, 2025 •
edited
Loading

Benchmark movements: tree_computation_flow performance improved 😺 tree_computation_flow time: [14.516 ms 14.639 ms 14.773 ms] change: [-19.857% -18.577% -17.155%] (p = 0.00 < 0.05) Performance has improved. Found 7 outliers among 100 measurements (7.00%) 4 (4.00%) high mild 3 (3.00%) high severe full_committer_flow performance regressed! full_committer_flow time: [15.426 ms 15.534 ms 15.646 ms] change: [+2.3670% +3.7237% +5.1333%] (p = 0.00 < 0.05) Performance has regressed. Found 5 outliers among 100 measurements (5.00%) 5 (5.00%) high mild

@ArniStarkware ArniStarkware mentioned this pull request Oct 23, 2025
Open
@ArniStarkware ArniStarkware changed the base branch from main-v0.14.1 to graphite-base/9748 October 28, 2025 16:31
@ArniStarkware ArniStarkware force-pushed the arni/test_utils/move_values_from_mempool_test_utils branch from be6be0c to ab3dffe Compare October 28, 2025 16:32
@ArniStarkware ArniStarkware changed the base branch from graphite-base/9748 to arni/test_utils/remove_consts_from_mempool_test_utils October 28, 2025 16:32
@ArniStarkware ArniStarkware mentioned this pull request Oct 28, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ArniStarkware @reviewable-StarkWare