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.

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

fix: flamegraph script (and enable > 1 circuit) #10065

Merged
merged 7 commits into from
Nov 27, 2024
Merged

fix: flamegraph script (and enable > 1 circuit) #10065

merged 7 commits into from
Nov 27, 2024

Conversation

iAmMichaelConnor
Copy link
Contributor

The flamegraph script was broken.
Then I added some extra features:

  • The ability to specify >1 circuit name.
  • -l list the names of all the available, compiled circuits, because I can't be bothered typing them.
  • -a create flamegraphs for all the circuits at once.
  • -n don't try to create flamegraphs, just act as a proxy to the server and serve me my flamegraphs.

Look at them all muhahahahahaha:
image

iAmMichaelConnor
iAmMichaelConnor commented Nov 20, 2024
View reviewed changes
noir/noir-repo/tooling/profiler/src/cli/gates_flamegraph_cmd.rs Outdated

/// The output name for the flamegraph svg files
#[clap(long, short = 'f', default_value = "main")]
output_filename: String
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Not sure if this rust is acceptable, but I needed each flamegraph to have a distinct name, rather than overwriting main_gates.svg repeatedly.

@iAmMichaelConnor
Copy link
Contributor Author

Ohh, @sirasistant does CI get angry if I try to touch Noir in here instead of in the Noir repo?

@sirasistant
Copy link
Collaborator

It's just formatting errors: you need to run cargo fmt :D

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Nov 20, 2024 •
edited
Loading

Changes to circuit sizes

Generated at commit: 128a5403335c69acf6e4b79f93881ffa3b8bfc19, compared to commit: 49b4a6c07f39711ad2a0477e1fad11e11b8ee23c

🧾 Summary (100% most significant diffs)

Program ACIR opcodes (+/-) % Circuit size (+/-) %
rollup_base_private +137,564 ❌ +69.59% +750,997 ❌ +28.25%
rollup_base_public +25,379 ❌ +5.68% -251,515 ✅ -6.29%
Full diff report 👇
Program ACIR opcodes (+/-) % Circuit size (+/-) %
rollup_base_private 335,237 (+137,564) +69.59% 3,409,576 (+750,997) +28.25%
rollup_base_public 472,504 (+25,379) +5.68% 3,745,928 (-251,515) -6.29%

@github-actions GitHub Actions
Copy link
Contributor

Changes to public function bytecode sizes

Generated at commit: 128a5403335c69acf6e4b79f93881ffa3b8bfc19, compared to commit: 49b4a6c07f39711ad2a0477e1fad11e11b8ee23c

🧾 Summary (100% most significant diffs)

Program Bytecode size in bytes (+/-) %
AvmTest::set_read_storage_single +590 ❌ +421.43%
DocsExample::initialize_public_immutable +722 ❌ +320.89%
AvmTest::read_storage_single +399 ❌ +302.27%
EasyPrivateVoting::end_vote +586 ❌ +257.02%
NFT::set_admin +586 ❌ +255.90%
Token::set_admin +586 ❌ +255.90%
FeeJuice::set_portal +775 ❌ +255.78%
Child::pub_inc_value +581 ❌ +242.08%
StaticChild::pub_inc_value +581 ❌ +242.08%
AvmInitializerTest::read_storage_immutable +399 ❌ +214.52%
StaticChild::pub_illegal_inc_value +581 ❌ +206.76%
Child::pub_inc_value_internal +581 ❌ +203.15%
NFT::public_get_symbol +403 ❌ +180.72%
TokenBridge::get_token +403 ❌ +180.72%
Token::get_admin +403 ❌ +180.72%
NFT::public_get_name +399 ❌ +171.98%
NFT::get_admin +399 ❌ +171.98%
Token::public_get_symbol +399 ❌ +171.98%
Token::public_get_name +399 ❌ +171.98%
Token::public_get_decimals +399 ❌ +161.54%
TokenBridge::_assert_token_is_same +399 ❌ +151.14%
TokenBridge::get_portal_address_public +399 ❌ +151.14%
AvmTest::read_storage_list +416 ❌ +146.48%
AvmTest::set_storage_single +184 ❌ +143.75%
AvmTest::set_storage_list +187 ❌ +128.97%
TokenBlacklist::total_supply +399 ❌ +115.65%
Token::total_supply +399 ❌ +115.65%
Token::_reduce_total_supply +586 ❌ +108.92%
TokenBlacklist::_reduce_total_supply +581 ❌ +107.99%
AvmTest::variable_base_msm +241 ❌ +107.59%
Child::set_value_twice_with_nested_last +635 ❌ +107.08%
Child::set_value_twice_with_nested_first +635 ❌ +107.08%
AvmTest::external_call_to_divide_by_zero +448 ❌ +95.73%
DocsExample::update_leader +187 ❌ +92.12%
Child::pub_set_value +179 ❌ +78.17%
StaticChild::pub_set_value +179 ❌ +78.17%
Lending::get_assets +399 ❌ +77.03%
AvmTest::nested_call_to_add_with_gas +406 ❌ +74.09%
StaticParent::public_get_value_from_child +406 ❌ +72.63%
AvmTest::nested_static_call_to_add +406 ❌ +70.86%
AvmTest::nested_call_to_add +406 ❌ +70.86%
AvmTest::nested_call_to_assert_same +406 ❌ +70.86%
TestLog::emit_unencrypted_events +736 ❌ +64.06%
Benchmarking::increment_balance +1,323 ❌ +55.36%
Uniswap::_assert_token_is_same +397 ❌ +55.06%
ImportTest::public_dispatch +550 ❌ +51.64%
StaticChild::public_dispatch +1,066 ❌ +50.47%
Crowdfunding::_publish_donation_receipts +450 ❌ +47.22%
DocsExample::public_dispatch +1,852 ❌ +43.68%
Benchmarking::public_dispatch +1,688 ❌ +42.50%
TokenBridge::constructor +993 ❌ +42.13%
StatefulTest::public_constructor +1,091 ❌ +39.09%
TestLog::public_dispatch +904 ❌ +38.00%
FPC::constructor +834 ❌ +35.98%
EasyPrivateVoting::constructor +843 ❌ +35.10%
Crowdfunding::public_dispatch +1,603 ❌ +34.30%
Claim::constructor +816 ❌ +33.27%
StatefulTest::public_dispatch +2,034 ❌ +33.12%
Auth::set_authorized +1,500 ❌ +32.94%
Claim::public_dispatch +1,053 ❌ +32.38%
AvmInitializerTest::constructor +615 ❌ +31.51%
PriceFeed::public_dispatch +974 ❌ +31.44%
AppSubscription::constructor +878 ❌ +30.98%
Crowdfunding::init +841 ❌ +30.96%
Uniswap::constructor +713 ❌ +30.15%
Auth::constructor +713 ❌ +29.63%
Child::public_dispatch +1,367 ❌ +28.63%
Auth::set_authorized_delay +1,323 ❌ +28.48%
StaticParent::public_dispatch +1,593 ❌ +27.54%
TokenBridge::exit_to_l1_public +1,737 ❌ +27.40%
AvmInitializerTest::public_dispatch +742 ❌ +27.38%
TokenBlacklist::constructor +1,696 ❌ +27.17%
AppSubscription::public_dispatch +1,065 ❌ +26.66%
TokenBlacklist::update_roles +1,552 ❌ +25.25%
Token::constructor +1,035 ❌ +25.06%
Auth::public_dispatch +2,108 ❌ +23.95%
EasyPrivateVoting::add_to_tally_public +489 ❌ +23.76%
Token::mint_to_public +657 ❌ +22.91%
StatefulTest::increment_public_value_no_init_check +391 ❌ +22.91%
StatefulTest::increment_public_value +391 ❌ +22.27%
NFT::set_minter +396 ❌ +21.89%
Token::set_minter +391 ❌ +21.61%
AvmTest::add_storage_map +410 ❌ +21.51%
Lending::_borrow +1,592 ❌ +21.46%
FeeJuice::public_dispatch +1,057 ❌ +19.97%
NFT::_store_payload_in_transient_storage_unsafe +230 ❌ +19.41%
Token::_store_payload_in_transient_storage_unsafe +230 ❌ +19.41%
AuthWitTest::public_dispatch +321 ❌ +19.30%
Lending::_deposit +491 ❌ +19.25%
NFT::constructor +754 ❌ +18.59%
Child::set_value_with_two_nested_calls +174 ❌ +18.55%
Spam::public_dispatch +594 ❌ +18.28%
NFT::mint +519 ❌ +18.24%
StaticParent::public_nested_static_call +370 ❌ +18.00%
EasyPrivateVoting::public_dispatch +931 ❌ +17.89%
Auth::get_authorized_delay +347 ❌ +16.92%
FPC::public_dispatch +1,215 ❌ +16.74%
FeeJuice::_increase_public_balance +362 ❌ +16.28%
TokenBlacklist::mint_public +632 ❌ +15.80%
Test::emit_unencrypted +230 ❌ +15.50%
AvmTest::keccak_hash +371 ❌ +15.47%
TokenBridge::claim_public +1,754 ❌ +15.41%
TokenBlacklist::mint_private +574 ❌ +15.34%
NFT::finalize_transfer_to_private +841 ❌ +15.03%
NFT::_finalize_transfer_to_private_unsafe +841 ❌ +14.91%
NFT::owner_of +362 ❌ +14.87%
Spam::public_spam +362 ❌ +14.87%
InclusionProofs::constructor +342 ❌ +14.86%
TokenBlacklist::_increase_public_balance +362 ❌ +14.52%
Token::_increase_public_balance +362 ❌ +14.52%
Token::transfer_in_public +589 ❌ +13.92%
Token::finalize_mint_to_private +823 ❌ +13.10%
PriceFeed::get_price +230 ❌ +13.08%
Token::_finalize_mint_to_private_unsafe +823 ❌ +13.00%
AvmTest::read_storage_map +218 ❌ +12.74%
Token::burn_public +531 ❌ +12.70%
TokenBlacklist::transfer_public +713 ❌ +12.69%
TokenBlacklist::burn_public +660 ❌ +12.60%
StatefulTest::get_public_value +218 ❌ +12.51%
InclusionProofs::public_dispatch +470 ❌ +12.49%
AuthRegistry::is_reject_all +218 ❌ +12.44%
NFT::is_minter +218 ❌ +12.07%
Token::is_minter +218 ❌ +12.07%
Benchmarking::broadcast +214 ❌ +12.02%
NFT::transfer_in_public +396 ❌ +11.86%
FeeJuice::balance_of_public +218 ❌ +11.78%
AuthRegistry::is_consumable +224 ❌ +11.71%
TokenBlacklist::balance_of_public +218 ❌ +11.45%
Token::balance_of_public +218 ❌ +11.45%
Router::public_dispatch +264 ❌ +11.18%
NFT::public_dispatch +2,641 ❌ +11.09%
TokenBlacklist::shield +630 ❌ +10.85%
AuthRegistry::consume +310 ❌ +10.57%
Parent::public_dispatch +791 ❌ +10.24%
Token::finalize_transfer_to_private +676 ❌ +10.18%
Token::_finalize_transfer_to_private_unsafe +676 ❌ +10.11%
AvmTest::returndata_copy_oracle +177 ❌ +9.66%
FeeJuice::check_balance +214 ❌ +9.61%
TokenBlacklist::public_dispatch +2,153 ❌ +9.27%
Uniswap::public_dispatch +2,120 ❌ +9.16%
Lending::_repay +491 ❌ +8.63%
AvmTest::set_storage_map +152 ❌ +8.43%
AvmTest::emit_unencrypted_log +230 ❌ +8.28%
AuthRegistry::set_reject_all +138 ❌ +8.08%
TokenBridge::public_dispatch +1,672 ❌ +8.06%
AuthRegistry::set_authorized +148 ❌ +7.96%
Test::public_dispatch +1,517 ❌ +7.92%
Lending::init +234 ❌ +7.91%
AuthRegistry::_set_authorized +148 ❌ +7.76%
NFT::_finish_transfer_to_public +138 ❌ +7.71%
PriceFeed::set_price +136 ❌ +7.51%
AuthRegistry::public_dispatch +572 ❌ +7.20%
Auth::get_scheduled_authorized +128 ❌ +7.06%
Lending::_withdraw +609 ❌ +7.03%
Auth::get_authorized +128 ❌ +6.69%
Token::complete_refund +356 ❌ +6.48%
Token::public_dispatch +2,019 ❌ +6.38%
Lending::get_position +229 ❌ +4.57%
Lending::public_dispatch +1,183 ❌ +4.33%
AvmTest::bulk_testing +971 ❌ +4.20%
Uniswap::swap_public +633 ❌ +4.15%
TokenBlacklist::get_roles +108 ❌ +4.02%
Uniswap::_approve_bridge_and_exit_input_asset_to_L1 +249 ❌ +4.00%
Test::consume_mint_to_public_message +400 ❌ +3.68%
CardGame::public_dispatch +447 ❌ +2.76%
Test::consume_message_from_arbitrary_sender_public +206 ❌ +2.60%
CardGame::on_cards_claimed +155 ❌ +2.12%
AvmTest::public_dispatch +979 ❌ +1.61%
Lending::get_asset +4 ❌ +0.18%
CardGame::on_card_played -13 ✅ -0.22%
AvmTest::sha256_hash -13 ✅ -0.33%
AvmTest::pedersen_commit -8 ✅ -0.63%
Parent::public_nested_static_call -69 ✅ -2.01%
Lending::deposit_public -57 ✅ -2.18%
Lending::repay_public -57 ✅ -2.28%
Full diff report 👇
Program Bytecode size in bytes (+/-) %
AvmTest::set_read_storage_single 730 (+590) +421.43%
DocsExample::initialize_public_immutable 947 (+722) +320.89%
AvmTest::read_storage_single 531 (+399) +302.27%
EasyPrivateVoting::end_vote 814 (+586) +257.02%
NFT::set_admin 815 (+586) +255.90%
Token::set_admin 815 (+586) +255.90%
FeeJuice::set_portal 1,078 (+775) +255.78%
Child::pub_inc_value 821 (+581) +242.08%
StaticChild::pub_inc_value 821 (+581) +242.08%
AvmInitializerTest::read_storage_immutable 585 (+399) +214.52%
StaticChild::pub_illegal_inc_value 862 (+581) +206.76%
Child::pub_inc_value_internal 867 (+581) +203.15%
NFT::public_get_symbol 626 (+403) +180.72%
TokenBridge::get_token 626 (+403) +180.72%
Token::get_admin 626 (+403) +180.72%
NFT::public_get_name 631 (+399) +171.98%
NFT::get_admin 631 (+399) +171.98%
Token::public_get_symbol 631 (+399) +171.98%
Token::public_get_name 631 (+399) +171.98%
Token::public_get_decimals 646 (+399) +161.54%
TokenBridge::_assert_token_is_same 663 (+399) +151.14%
TokenBridge::get_portal_address_public 663 (+399) +151.14%
AvmTest::read_storage_list 700 (+416) +146.48%
AvmTest::set_storage_single 312 (+184) +143.75%
AvmTest::set_storage_list 332 (+187) +128.97%
TokenBlacklist::total_supply 744 (+399) +115.65%
Token::total_supply 744 (+399) +115.65%
Token::_reduce_total_supply 1,124 (+586) +108.92%
TokenBlacklist::_reduce_total_supply 1,119 (+581) +107.99%
AvmTest::variable_base_msm 465 (+241) +107.59%
Child::set_value_twice_with_nested_last 1,228 (+635) +107.08%
Child::set_value_twice_with_nested_first 1,228 (+635) +107.08%
AvmTest::external_call_to_divide_by_zero 916 (+448) +95.73%
DocsExample::update_leader 390 (+187) +92.12%
Child::pub_set_value 408 (+179) +78.17%
StaticChild::pub_set_value 408 (+179) +78.17%
Lending::get_assets 917 (+399) +77.03%
AvmTest::nested_call_to_add_with_gas 954 (+406) +74.09%
StaticParent::public_get_value_from_child 965 (+406) +72.63%
AvmTest::nested_static_call_to_add 979 (+406) +70.86%
AvmTest::nested_call_to_add 979 (+406) +70.86%
AvmTest::nested_call_to_assert_same 979 (+406) +70.86%
TestLog::emit_unencrypted_events 1,885 (+736) +64.06%
Benchmarking::increment_balance 3,713 (+1,323) +55.36%
Uniswap::_assert_token_is_same 1,118 (+397) +55.06%
ImportTest::public_dispatch 1,615 (+550) +51.64%
StaticChild::public_dispatch 3,178 (+1,066) +50.47%
Crowdfunding::_publish_donation_receipts 1,403 (+450) +47.22%
DocsExample::public_dispatch 6,092 (+1,852) +43.68%
Benchmarking::public_dispatch 5,660 (+1,688) +42.50%
TokenBridge::constructor 3,350 (+993) +42.13%
StatefulTest::public_constructor 3,882 (+1,091) +39.09%
TestLog::public_dispatch 3,283 (+904) +38.00%
FPC::constructor 3,152 (+834) +35.98%
EasyPrivateVoting::constructor 3,245 (+843) +35.10%
Crowdfunding::public_dispatch 6,277 (+1,603) +34.30%
Claim::constructor 3,269 (+816) +33.27%
StatefulTest::public_dispatch 8,175 (+2,034) +33.12%
Auth::set_authorized 6,054 (+1,500) +32.94%
Claim::public_dispatch 4,305 (+1,053) +32.38%
AvmInitializerTest::constructor 2,567 (+615) +31.51%
PriceFeed::public_dispatch 4,072 (+974) +31.44%
AppSubscription::constructor 3,712 (+878) +30.98%
Crowdfunding::init 3,557 (+841) +30.96%
Uniswap::constructor 3,078 (+713) +30.15%
Auth::constructor 3,119 (+713) +29.63%
Child::public_dispatch 6,141 (+1,367) +28.63%
Auth::set_authorized_delay 5,968 (+1,323) +28.48%
StaticParent::public_dispatch 7,378 (+1,593) +27.54%
TokenBridge::exit_to_l1_public 8,076 (+1,737) +27.40%
AvmInitializerTest::public_dispatch 3,452 (+742) +27.38%
TokenBlacklist::constructor 7,938 (+1,696) +27.17%
AppSubscription::public_dispatch 5,060 (+1,065) +26.66%
TokenBlacklist::update_roles 7,699 (+1,552) +25.25%
Token::constructor 5,165 (+1,035) +25.06%
Auth::public_dispatch 10,908 (+2,108) +23.95%
EasyPrivateVoting::add_to_tally_public 2,547 (+489) +23.76%
Token::mint_to_public 3,525 (+657) +22.91%
StatefulTest::increment_public_value_no_init_check 2,098 (+391) +22.91%
StatefulTest::increment_public_value 2,147 (+391) +22.27%
NFT::set_minter 2,205 (+396) +21.89%
Token::set_minter 2,200 (+391) +21.61%
AvmTest::add_storage_map 2,316 (+410) +21.51%
Lending::_borrow 9,009 (+1,592) +21.46%
FeeJuice::public_dispatch 6,349 (+1,057) +19.97%
NFT::_store_payload_in_transient_storage_unsafe 1,415 (+230) +19.41%
Token::_store_payload_in_transient_storage_unsafe 1,415 (+230) +19.41%
AuthWitTest::public_dispatch 1,984 (+321) +19.30%
Lending::_deposit 3,041 (+491) +19.25%
NFT::constructor 4,809 (+754) +18.59%
Child::set_value_with_two_nested_calls 1,112 (+174) +18.55%
Spam::public_dispatch 3,843 (+594) +18.28%
NFT::mint 3,364 (+519) +18.24%
StaticParent::public_nested_static_call 2,425 (+370) +18.00%
EasyPrivateVoting::public_dispatch 6,135 (+931) +17.89%
Auth::get_authorized_delay 2,398 (+347) +16.92%
FPC::public_dispatch 8,473 (+1,215) +16.74%
FeeJuice::_increase_public_balance 2,586 (+362) +16.28%
TokenBlacklist::mint_public 4,633 (+632) +15.80%
Test::emit_unencrypted 1,714 (+230) +15.50%
AvmTest::keccak_hash 2,769 (+371) +15.47%
TokenBridge::claim_public 13,136 (+1,754) +15.41%
TokenBlacklist::mint_private 4,316 (+574) +15.34%
NFT::finalize_transfer_to_private 6,435 (+841) +15.03%
NFT::_finalize_transfer_to_private_unsafe 6,482 (+841) +14.91%
NFT::owner_of 2,796 (+362) +14.87%
Spam::public_spam 2,796 (+362) +14.87%
InclusionProofs::constructor 2,643 (+342) +14.86%
TokenBlacklist::_increase_public_balance 2,855 (+362) +14.52%
Token::_increase_public_balance 2,855 (+362) +14.52%
Token::transfer_in_public 4,819 (+589) +13.92%
Token::finalize_mint_to_private 7,105 (+823) +13.10%
PriceFeed::get_price 1,988 (+230) +13.08%
Token::_finalize_mint_to_private_unsafe 7,152 (+823) +13.00%
AvmTest::read_storage_map 1,929 (+218) +12.74%
Token::burn_public 4,711 (+531) +12.70%
TokenBlacklist::transfer_public 6,333 (+713) +12.69%
TokenBlacklist::burn_public 5,900 (+660) +12.60%
StatefulTest::get_public_value 1,960 (+218) +12.51%
InclusionProofs::public_dispatch 4,232 (+470) +12.49%
AuthRegistry::is_reject_all 1,970 (+218) +12.44%
NFT::is_minter 2,024 (+218) +12.07%
Token::is_minter 2,024 (+218) +12.07%
Benchmarking::broadcast 1,994 (+214) +12.02%
NFT::transfer_in_public 3,736 (+396) +11.86%
FeeJuice::balance_of_public 2,068 (+218) +11.78%
AuthRegistry::is_consumable 2,137 (+224) +11.71%
TokenBlacklist::balance_of_public 2,122 (+218) +11.45%
Token::balance_of_public 2,122 (+218) +11.45%
Router::public_dispatch 2,625 (+264) +11.18%
NFT::public_dispatch 26,461 (+2,641) +11.09%
TokenBlacklist::shield 6,438 (+630) +10.85%
AuthRegistry::consume 3,244 (+310) +10.57%
Parent::public_dispatch 8,514 (+791) +10.24%
Token::finalize_transfer_to_private 7,315 (+676) +10.18%
Token::_finalize_transfer_to_private_unsafe 7,362 (+676) +10.11%
AvmTest::returndata_copy_oracle 2,010 (+177) +9.66%
FeeJuice::check_balance 2,441 (+214) +9.61%
TokenBlacklist::public_dispatch 25,366 (+2,153) +9.27%
Uniswap::public_dispatch 25,256 (+2,120) +9.16%
Lending::_repay 6,179 (+491) +8.63%
AvmTest::set_storage_map 1,956 (+152) +8.43%
AvmTest::emit_unencrypted_log 3,007 (+230) +8.28%
AuthRegistry::set_reject_all 1,845 (+138) +8.08%
TokenBridge::public_dispatch 22,412 (+1,672) +8.06%
AuthRegistry::set_authorized 2,007 (+148) +7.96%
Test::public_dispatch 20,664 (+1,517) +7.92%
Lending::init 3,192 (+234) +7.91%
AuthRegistry::_set_authorized 2,054 (+148) +7.76%
NFT::_finish_transfer_to_public 1,929 (+138) +7.71%
PriceFeed::set_price 1,946 (+136) +7.51%
AuthRegistry::public_dispatch 8,515 (+572) +7.20%
Auth::get_scheduled_authorized 1,941 (+128) +7.06%
Lending::_withdraw 9,273 (+609) +7.03%
Auth::get_authorized 2,041 (+128) +6.69%
Token::complete_refund 5,848 (+356) +6.48%
Token::public_dispatch 33,653 (+2,019) +6.38%
Lending::get_position 5,238 (+229) +4.57%
Lending::public_dispatch 28,533 (+1,183) +4.33%
AvmTest::bulk_testing 24,075 (+971) +4.20%
Uniswap::swap_public 15,903 (+633) +4.15%
TokenBlacklist::get_roles 2,795 (+108) +4.02%
Uniswap::_approve_bridge_and_exit_input_asset_to_L1 6,473 (+249) +4.00%
Test::consume_mint_to_public_message 11,284 (+400) +3.68%
CardGame::public_dispatch 16,643 (+447) +2.76%
Test::consume_message_from_arbitrary_sender_public 8,137 (+206) +2.60%
CardGame::on_cards_claimed 7,458 (+155) +2.12%
AvmTest::public_dispatch 61,741 (+979) +1.61%
Lending::get_asset 2,209 (+4) +0.18%
CardGame::on_card_played 5,987 (-13) -0.22%
AvmTest::sha256_hash 3,910 (-13) -0.33%
AvmTest::pedersen_commit 1,255 (-8) -0.63%
Parent::public_nested_static_call 3,364 (-69) -2.01%
Lending::deposit_public 2,555 (-57) -2.18%
Lending::repay_public 2,438 (-57) -2.28%

@iAmMichaelConnor iAmMichaelConnor merged commit 0c3b7ef into master Nov 27, 2024
65 checks passed
@iAmMichaelConnor iAmMichaelConnor deleted the mc/update-flamegraph branch November 27, 2024 18:31
This was referenced Nov 27, 2024
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sirasistant sirasistant sirasistant approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@iAmMichaelConnor @sirasistant