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

test(blockifier): l1 bounds -> all bounds in versioned state test #1308

Conversation

dorimedini-starkware
Copy link
Collaborator

@dorimedini-starkware dorimedini-starkware commented Oct 10, 2024

This change is Reviewable

Copy link

codecov bot commented Oct 10, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 56.57%. Comparing base (b0cfe82) to head (aeb522a).
Report is 394 commits behind head on main.

❗ There is a different number of reports uploaded between BASE (b0cfe82) and HEAD (aeb522a). Click for more details.

HEAD has 2 uploads less than BASE
Flag BASE (b0cfe82) HEAD (aeb522a)
3 1
Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1308       +/-   ##
===========================================
- Coverage   74.18%   56.57%   -17.62%     
===========================================
  Files         359      316       -43     
  Lines       36240    35488      -752     
  Branches    36240    35488      -752     
===========================================
- Hits        26886    20078     -6808     
- Misses       7220    13820     +6600     
+ Partials     2134     1590      -544     
Flag Coverage Δ
?

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@dorimedini-starkware dorimedini-starkware force-pushed the 10-10-test_blockifier_add_all_resource_bounds_to_stateful_validator_test branch from 43f4163 to 541e3ca Compare October 10, 2024 15:11
@dorimedini-starkware dorimedini-starkware force-pushed the 10-10-test_blockifier_l1_bounds_-_all_bounds_in_versioned_state_test branch from 6c693dd to 50c5c89 Compare October 10, 2024 15:11
@dorimedini-starkware dorimedini-starkware force-pushed the 10-10-test_blockifier_add_all_resource_bounds_to_stateful_validator_test branch from 541e3ca to ae3e6b3 Compare October 13, 2024 08:13
@dorimedini-starkware dorimedini-starkware force-pushed the 10-10-test_blockifier_l1_bounds_-_all_bounds_in_versioned_state_test branch from 50c5c89 to 5e19234 Compare October 13, 2024 08:13
@dorimedini-starkware dorimedini-starkware force-pushed the 10-10-test_blockifier_add_all_resource_bounds_to_stateful_validator_test branch from ae3e6b3 to 23e75f6 Compare October 13, 2024 08:15
@dorimedini-starkware dorimedini-starkware force-pushed the 10-10-test_blockifier_l1_bounds_-_all_bounds_in_versioned_state_test branch from 5e19234 to c4458ec Compare October 13, 2024 08:16
@dorimedini-starkware dorimedini-starkware self-assigned this Oct 13, 2024
@dorimedini-starkware dorimedini-starkware force-pushed the 10-10-test_blockifier_add_all_resource_bounds_to_stateful_validator_test branch from 23e75f6 to 920351b Compare October 13, 2024 12:38
@dorimedini-starkware dorimedini-starkware force-pushed the 10-10-test_blockifier_l1_bounds_-_all_bounds_in_versioned_state_test branch from c4458ec to 9610494 Compare October 13, 2024 12:38
@dorimedini-starkware dorimedini-starkware force-pushed the 10-10-test_blockifier_add_all_resource_bounds_to_stateful_validator_test branch from 920351b to ce63edc Compare October 14, 2024 08:06
@dorimedini-starkware dorimedini-starkware force-pushed the 10-10-test_blockifier_l1_bounds_-_all_bounds_in_versioned_state_test branch from 9610494 to e458074 Compare October 14, 2024 08:06
@dorimedini-starkware dorimedini-starkware force-pushed the 10-10-test_blockifier_add_all_resource_bounds_to_stateful_validator_test branch from ce63edc to af2eed1 Compare October 14, 2024 09:14
@dorimedini-starkware dorimedini-starkware force-pushed the 10-10-test_blockifier_l1_bounds_-_all_bounds_in_versioned_state_test branch from e458074 to 5e46120 Compare October 14, 2024 09:14
@dorimedini-starkware dorimedini-starkware force-pushed the 10-10-test_blockifier_add_all_resource_bounds_to_stateful_validator_test branch from af2eed1 to 556c943 Compare October 14, 2024 12:12
@dorimedini-starkware dorimedini-starkware force-pushed the 10-10-test_blockifier_l1_bounds_-_all_bounds_in_versioned_state_test branch 2 times, most recently from e7581c7 to 343b9d0 Compare October 14, 2024 13:08
Copy link
Contributor

@yoavGrs yoavGrs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed 1 of 1 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @dorimedini-starkware)

@dorimedini-starkware dorimedini-starkware force-pushed the 10-10-test_blockifier_add_all_resource_bounds_to_stateful_validator_test branch from 556c943 to 6e4825a Compare October 15, 2024 07:52
@dorimedini-starkware dorimedini-starkware force-pushed the 10-10-test_blockifier_l1_bounds_-_all_bounds_in_versioned_state_test branch from 343b9d0 to 750d497 Compare October 15, 2024 07:52
@dorimedini-starkware dorimedini-starkware force-pushed the 10-10-test_blockifier_add_all_resource_bounds_to_stateful_validator_test branch 2 times, most recently from 1de9c83 to f69343d Compare October 15, 2024 08:33
@dorimedini-starkware dorimedini-starkware force-pushed the 10-10-test_blockifier_l1_bounds_-_all_bounds_in_versioned_state_test branch from 750d497 to dca6687 Compare October 15, 2024 08:33
@dorimedini-starkware dorimedini-starkware changed the base branch from 10-10-test_blockifier_add_all_resource_bounds_to_stateful_validator_test to graphite-base/1308 October 15, 2024 08:58
@dorimedini-starkware dorimedini-starkware force-pushed the 10-10-test_blockifier_l1_bounds_-_all_bounds_in_versioned_state_test branch from dca6687 to 64067d2 Compare October 15, 2024 08:59
@dorimedini-starkware dorimedini-starkware changed the base branch from graphite-base/1308 to main October 15, 2024 08:59
@dorimedini-starkware dorimedini-starkware force-pushed the 10-10-test_blockifier_l1_bounds_-_all_bounds_in_versioned_state_test branch from 64067d2 to aeb522a Compare October 15, 2024 08:59
@dorimedini-starkware dorimedini-starkware merged commit a76c12e into main Oct 15, 2024
11 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Oct 18, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants