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(docs): Update debugging docs #9200

Merged
merged 4 commits into from
Oct 20, 2024
Merged

fix(docs): Update debugging docs #9200

merged 4 commits into from
Oct 20, 2024

Conversation

critesjosh
Copy link
Contributor

Updates the debugging docs to actually work.

closes: AztecProtocol/dev-rel#396

@critesjosh critesjosh requested review from catmcgee and a team October 11, 2024 16:42
@AztecBot
Copy link
Collaborator

AztecBot commented Oct 11, 2024 •
edited
Loading

Docs Preview

Hey there! 👋 You can check your preview at https://6713ebd9809f46ef0525cdd3--aztec-docs-dev.netlify.app

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Oct 14, 2024 •
edited
Loading

Changes to public function bytecode sizes

Generated at commit: f721b3a9ac24fcfb0234f61de7ecb58011dd11f8, compared to commit: ab0c80d7493e6bdbc58dcd517b248de6ddd6fd67

🧾 Summary (100% most significant diffs)

Program Bytecode size in bytes (+/-) %
Test::create_l2_to_l1_message_public +369 ❌ +18.45%
Token::complete_refund +402 ❌ +14.73%
Token::mint_private +415 ❌ +14.03%
AvmTest::pedersen_hash_with_index +412 ❌ +13.62%
AvmTest::pedersen_hash +412 ❌ +13.62%
TokenBlacklist::mint_private +454 ❌ +12.07%
NFT::finalize_transfer_to_private +581 ❌ +11.10%
CardGame::on_cards_claimed +707 ❌ +10.59%
Token::shield +512 ❌ +10.37%
TokenBlacklist::shield +563 ❌ +9.36%
CardGame::public_dispatch +1,190 ❌ +8.12%
AvmTest::debug_logging +99 ❌ +8.09%
AvmTest::u128_from_integer_overflow +52 ❌ +7.49%
AvmTest::bulk_testing +1,764 ❌ +7.33%
AvmTest::u128_addition_overflow +85 ❌ +7.23%
Test::public_dispatch +1,231 ❌ +7.15%
Delegator::public_delegate_set_value +60 ❌ +7.14%
AvmTest::emit_unencrypted_log +191 ❌ +6.99%
CardGame::start_game +421 ❌ +6.92%
CardGame::on_card_played +339 ❌ +6.69%
TokenBridge::exit_to_l1_public +444 ❌ +6.59%
Delegator::public_dispatch +125 ❌ +6.56%
CardGame::on_game_joined +279 ❌ +6.46%
Uniswap::swap_public +917 ❌ +6.41%
Test::emit_unencrypted +98 ❌ +6.41%
TestLog::public_dispatch +188 ❌ +6.37%
DelegatedOn::public_dispatch +72 ❌ +6.06%
TestLog::emit_unencrypted_events +101 ❌ +6.01%
AvmTest::keccak_hash +153 ❌ +5.99%
Test::consume_message_from_arbitrary_sender_public +383 ❌ +5.90%
ImportTest::public_dispatch +96 ❌ +5.90%
Parent::public_nested_static_call +215 ❌ +5.84%
AvmTest::sha256_hash +171 ❌ +5.84%
Test::consume_mint_public_message +494 ❌ +5.78%
AuthWitTest::public_dispatch +117 ❌ +5.75%
Parent::public_dispatch +461 ❌ +5.75%
DocsExample::public_dispatch +285 ❌ +5.70%
TokenBridge::claim_public +563 ❌ +5.69%
StaticChild::public_dispatch +157 ❌ +5.68%
DocsExample::get_shared_immutable_constrained_public_multiple +39 ❌ +5.67%
AvmTest::nested_call_to_add_with_gas +45 ❌ +5.67%
PriceFeed::public_dispatch +199 ❌ +5.64%
Crowdfunding::_publish_donation_receipts +68 ❌ +5.63%
Child::pub_inc_value +36 ❌ +5.54%
StaticChild::pub_inc_value +36 ❌ +5.54%
StaticParent::public_dispatch +310 ❌ +5.52%
StaticChild::pub_illegal_inc_value +37 ❌ +5.52%
AuthWitTest::consume_public +46 ❌ +5.50%
AvmTest::nested_call_to_add +45 ❌ +5.49%
AvmTest::nested_call_to_assert_same +45 ❌ +5.49%
AvmTest::nested_static_call_to_add +45 ❌ +5.49%
Child::pub_inc_value_internal +37 ❌ +5.48%
Child::public_dispatch +294 ❌ +5.46%
Child::set_value_twice_with_nested_first +47 ❌ +5.40%
Child::set_value_twice_with_nested_last +47 ❌ +5.40%
Parent::public_static_call +50 ❌ +5.38%
StaticParent::public_static_call +50 ❌ +5.38%
Benchmarking::public_dispatch +254 ❌ +5.38%
Uniswap::_approve_bridge_and_exit_input_asset_to_L1 +304 ❌ +5.36%
TokenBridge::_call_mint_on_token +63 ❌ +5.34%
StatefulTest::increment_public_value_no_init_check +94 ❌ +5.31%
AvmTest::variable_base_msm +23 ❌ +5.30%
DocsExample::initialize_shared_immutable +40 ❌ +5.30%
DocsExample::initialize_public_immutable +40 ❌ +5.30%
Spam::public_dispatch +177 ❌ +5.29%
Lending::get_asset +100 ❌ +5.28%
Benchmarking::broadcast +90 ❌ +5.28%
Router::public_dispatch +143 ❌ +5.26%
StatefulTest::increment_public_value +94 ❌ +5.23%
PriceFeed::get_price +88 ❌ +5.20%
AvmTest::read_storage_list +31 ❌ +5.16%
Benchmarking::increment_balance +150 ❌ +5.15%
FeeJuice::balance_of_public +90 ❌ +5.14%
NFT::set_minter +94 ❌ +5.13%
Token::set_minter +94 ❌ +5.13%
NFT::transfer_in_public +181 ❌ +5.13%
Lending::_deposit +113 ❌ +5.12%
Child::pub_set_value +16 ❌ +5.11%
StaticChild::pub_set_value +16 ❌ +5.11%
StatefulTest::get_public_value +85 ❌ +5.11%
StaticParent::public_nested_static_call +64 ❌ +5.09%
AuthRegistry::set_reject_all +80 ❌ +5.09%
PriceFeed::set_price +84 ❌ +5.08%
AvmTest::add_storage_map +100 ❌ +5.08%
AvmTest::read_storage_map +84 ❌ +5.08%
AuthRegistry::is_reject_all +85 ❌ +5.07%
AvmTest::set_read_storage_single +29 ❌ +5.06%
Router::_check_block_number +62 ❌ +5.05%
TokenBlacklist::balance_of_public +90 ❌ +5.04%
Token::balance_of_public +90 ❌ +5.04%
AuthRegistry::set_authorized +87 ❌ +5.04%
StaticParent::public_get_value_from_child +34 ❌ +5.03%
AuthRegistry::_set_authorized +88 ❌ +5.02%
AuthRegistry::is_consumable +92 ❌ +5.02%
NFT::_finish_transfer_to_public +81 ❌ +5.02%
Token::assert_minter_and_mint +111 ❌ +5.01%
FeeJuice::_increase_public_balance +112 ❌ +5.00%
AvmTest::assert_calldata_copy +21 ❌ +4.99%
Router::_check_timestamp +62 ❌ +4.98%
NFT::is_minter +85 ❌ +4.98%
Token::is_minter +85 ❌ +4.98%
Lending::init +138 ❌ +4.97%
Parent::pub_entry_point +34 ❌ +4.96%
StaticParent::public_call +34 ❌ +4.96%
AvmTest::create_different_nullifier_in_nested_call +33 ❌ +4.96%
Child::set_value_with_two_nested_calls +62 ❌ +4.96%
Auth::set_authorized_delay +265 ❌ +4.95%
Lending::repay_public +78 ❌ +4.94%
AvmTest::nested_static_call_to_set_storage +32 ❌ +4.93%
FPC::pay_refund +80 ❌ +4.92%
ImportTest::pub_call_public_fn +32 ❌ +4.92%
Lending::deposit_public +83 ❌ +4.92%
Auth::set_authorized +267 ❌ +4.92%
AuthRegistry::consume +140 ❌ +4.91%
Spam::public_spam +119 ❌ +4.91%
FeeJuice::public_dispatch +275 ❌ +4.91%
TokenBlacklist::_reduce_total_supply +43 ❌ +4.90%
Token::_reduce_total_supply +43 ❌ +4.90%
AvmTest::set_storage_map +82 ❌ +4.89%
AvmTest::create_same_nullifier_in_nested_call +32 ❌ +4.89%
AuthRegistry::public_dispatch +374 ❌ +4.88%
Auth::get_authorized +85 ❌ +4.88%
AvmTest::elliptic_curve_add +24 ❌ +4.88%
EasyPrivateVoting::end_vote +30 ❌ +4.88%
NFT::set_admin +30 ❌ +4.87%
Token::set_admin +30 ❌ +4.87%
FPC::pay_refund_with_shielded_rebate +76 ❌ +4.87%
DocsExample::get_shared_immutable_constrained_public_indirect +20 ❌ +4.85%
Auth::get_scheduled_authorized +80 ❌ +4.85%
Parent::pub_entry_point_twice +55 ❌ +4.85%
Lending::borrow_public +54 ❌ +4.84%
Lending::withdraw_public +54 ❌ +4.84%
TokenBlacklist::_increase_public_balance +119 ❌ +4.84%
Token::_increase_public_balance +119 ❌ +4.84%
AvmTest::poseidon2_hash +72 ❌ +4.83%
EasyPrivateVoting::add_to_tally_public +105 ❌ +4.81%
Token::transfer_public +217 ❌ +4.74%
TokenBlacklist::get_roles +115 ❌ +4.73%
FPC::prepare_fee +56 ❌ +4.73%
AvmTest::get_args_hash +74 ❌ +4.73%
TokenBlacklist::burn_public +262 ❌ +4.72%
Token::burn_public +211 ❌ +4.72%
FeeJuice::check_balance +97 ❌ +4.69%
DocsExample::get_shared_immutable_constrained_public +24 ❌ +4.66%
TokenBlacklist::transfer_public +275 ❌ +4.62%
AvmTest::public_dispatch +2,513 ❌ +4.61%
NFT::public_dispatch +983 ❌ +4.61%
AvmTest::read_storage_single +20 ❌ +4.58%
TokenBlacklist::update_roles +318 ❌ +4.58%
AvmTest::return_oracle +15 ❌ +4.55%
TokenBlacklist::total_supply +26 ❌ +4.43%
Token::total_supply +26 ❌ +4.43%
Lending::public_dispatch +1,123 ❌ +4.42%
NFT::mint +129 ❌ +4.42%
Lending::get_assets +34 ❌ +4.42%
NFT::owner_of +106 ❌ +4.41%
FeeJuice::set_portal +40 ❌ +4.40%
AvmTest::set_storage_list +11 ❌ +4.38%
Uniswap::public_dispatch +1,028 ❌ +4.37%
Auth::get_authorized_delay +89 ❌ +4.35%
TokenBridge::_assert_token_is_same +22 ❌ +4.35%
AvmTest::set_storage_single +10 ❌ +4.33%
Token::mint_public +134 ❌ +4.33%
NFT::public_get_symbol +21 ❌ +4.29%
TokenBridge::get_token +21 ❌ +4.29%
Token::get_admin +21 ❌ +4.29%
TokenBlacklist::mint_public +176 ❌ +4.28%
AvmInitializerTest::read_storage_immutable +20 ❌ +4.26%
NFT::get_admin +21 ❌ +4.24%
NFT::public_get_name +21 ❌ +4.24%
Token::public_get_name +21 ❌ +4.24%
Token::public_get_symbol +21 ❌ +4.24%
Uniswap::_assert_token_is_same +19 ❌ +4.24%
Lending::_withdraw +312 ❌ +4.23%
DelegatedOn::public_set_value +10 ❌ +4.22%
Lending::_borrow +308 ❌ +4.21%
AvmTest::elliptic_curve_add_and_double +26 ❌ +4.20%
Lending::update_accumulator +214 ❌ +4.18%
Lending::get_position +194 ❌ +4.18%
Token::public_dispatch +1,140 ❌ +4.18%
DocsExample::update_leader +12 ❌ +4.17%
Token::public_get_decimals +21 ❌ +4.12%
Lending::_repay +206 ❌ +4.03%
TokenBlacklist::public_dispatch +976 ❌ +4.00%
TokenBridge::get_portal_address_public +21 ❌ +3.98%
TokenBridge::public_dispatch +764 ❌ +3.88%
NFT::_store_point_in_transient_storage +13 ❌ +3.86%
StaticChild::pub_get_value +6 ❌ +3.70%
Child::pub_get_value +5 ❌ +3.65%
AvmTest::add_u128 +7 ❌ +3.48%
AvmTest::check_selector +6 ❌ +3.43%
AvmTest::to_radix_le +8 ❌ +3.40%
AvmTest::get_function_selector +5 ❌ +3.11%
Test::assert_public_global_vars +7 ❌ +2.88%
AvmTest::assertion_failure +4 ❌ +2.58%
AvmTest::keccak_f1600 +11 ❌ +2.36%
Test::dummy_public_call +2 ❌ +1.90%
AvmTest::add_args_return +2 ❌ +1.87%
AvmTest::assert_timestamp +2 ❌ +1.77%
AvmTest::assert_same +2 ❌ +1.69%
Test::is_time_equal +2 ❌ +1.68%
Auth::public_dispatch +150 ❌ +1.41%
FPC::public_dispatch +104 ❌ +1.14%
AvmTest::new_note_hash +1 ❌ +1.11%
AvmTest::new_nullifier +1 ❌ +1.11%
Test::emit_nullifier_public +1 ❌ +1.11%
AvmTest::get_fee_per_da_gas +1 ❌ +1.10%
AvmTest::get_sender +1 ❌ +1.10%
AvmTest::get_fee_per_l2_gas +1 ❌ +1.10%
AvmTest::get_version +1 ❌ +1.10%
AvmTest::get_transaction_fee +1 ❌ +1.10%
AvmTest::get_address +1 ❌ +1.10%
AvmTest::get_block_number +1 ❌ +1.10%
AvmTest::get_chain_id +1 ❌ +1.10%
AvmTest::get_timestamp +1 ❌ +1.10%
AvmTest::get_storage_address +1 ❌ +1.10%
AvmTest::set_opcode_u8 +1 ❌ +1.10%
AvmTest::get_l2_gas_left +1 ❌ +1.10%
AvmTest::get_da_gas_left +1 ❌ +1.10%
AvmTest::nullifier_collision +1 ❌ +1.06%
AvmTest::set_opcode_u32 +1 ❌ +1.05%
AvmTest::send_l2_to_l1_msg +1 ❌ +1.02%
Test::create_l2_to_l1_message_arbitrary_recipient_public +1 ❌ +1.02%
AvmTest::set_opcode_u64 +1 ❌ +1.01%
DocsExample::spend_public_authwit +1 ❌ +0.99%
AvmTest::set_opcode_small_field +1 ❌ +0.93%
AvmTest::nullifier_exists +1 ❌ +0.92%
AvmTest::l1_to_l2_msg_exists +1 ❌ +0.91%
AvmTest::note_hash_exists +1 ❌ +0.91%
AvmTest::modulo2 +1 ❌ +0.88%
AvmTest::assert_nullifier_exists +1 ❌ +0.88%
AvmTest::emit_nullifier_and_check +1 ❌ +0.85%
AvmTest::set_opcode_big_field +1 ❌ +0.81%
AvmTest::set_opcode_really_big_field +1 ❌ +0.81%
InclusionProofs::push_nullifier_public +1 ❌ +0.81%
InclusionProofs::test_nullifier_inclusion_from_public +1 ❌ +0.70%
TokenBlacklist::constructor +36 ❌ +0.45%
StatefulTest::public_dispatch +31 ❌ +0.40%
Crowdfunding::public_dispatch -28 ✅ -0.44%
EasyPrivateVoting::public_dispatch -58 ✅ -0.93%
AppSubscription::public_dispatch -103 ✅ -1.92%
Token::constructor -124 ✅ -2.27%
Claim::public_dispatch -112 ✅ -2.44%
PrivateFPC::public_dispatch -112 ✅ -2.54%
NFT::constructor -133 ✅ -2.60%
InclusionProofs::public_dispatch -131 ✅ -2.93%
AppSubscription::constructor -174 ✅ -4.23%
Crowdfunding::init -178 ✅ -4.49%
StatefulTest::public_constructor -172 ✅ -4.49%
AvmInitializerTest::public_dispatch -169 ✅ -4.50%
Claim::constructor -170 ✅ -4.63%
TokenBridge::constructor -170 ✅ -4.66%
PrivateFPC::constructor -170 ✅ -4.85%
EasyPrivateVoting::constructor -186 ✅ -5.06%
Auth::constructor -188 ✅ -5.32%
FPC::constructor -190 ✅ -5.41%
Uniswap::constructor -190 ✅ -5.41%
AvmInitializerTest::constructor -207 ✅ -6.83%
InclusionProofs::constructor -211 ✅ -6.85%
AvmTest::test_get_contract_instance -306 ✅ -26.20%
AvmTest::test_get_contract_instance_raw -421 ✅ -52.04%
AvmTest::pedersen_commit -729 ✅ -66.15%
Full diff report 👇
Program Bytecode size in bytes (+/-) %
Test::create_l2_to_l1_message_public 2,369 (+369) +18.45%
Token::complete_refund 3,131 (+402) +14.73%
Token::mint_private 3,373 (+415) +14.03%
AvmTest::pedersen_hash_with_index 3,438 (+412) +13.62%
AvmTest::pedersen_hash 3,438 (+412) +13.62%
TokenBlacklist::mint_private 4,215 (+454) +12.07%
NFT::finalize_transfer_to_private 5,817 (+581) +11.10%
CardGame::on_cards_claimed 7,383 (+707) +10.59%
Token::shield 5,447 (+512) +10.37%
TokenBlacklist::shield 6,581 (+563) +9.36%
CardGame::public_dispatch 15,849 (+1,190) +8.12%
AvmTest::debug_logging 1,322 (+99) +8.09%
AvmTest::u128_from_integer_overflow 746 (+52) +7.49%
AvmTest::bulk_testing 25,822 (+1,764) +7.33%
AvmTest::u128_addition_overflow 1,260 (+85) +7.23%
Test::public_dispatch 18,459 (+1,231) +7.15%
Delegator::public_delegate_set_value 900 (+60) +7.14%
AvmTest::emit_unencrypted_log 2,924 (+191) +6.99%
CardGame::start_game 6,507 (+421) +6.92%
CardGame::on_card_played 5,409 (+339) +6.69%
TokenBridge::exit_to_l1_public 7,182 (+444) +6.59%
Delegator::public_dispatch 2,031 (+125) +6.56%
CardGame::on_game_joined 4,601 (+279) +6.46%
Uniswap::swap_public 15,214 (+917) +6.41%
Test::emit_unencrypted 1,626 (+98) +6.41%
TestLog::public_dispatch 3,141 (+188) +6.37%
DelegatedOn::public_dispatch 1,261 (+72) +6.06%
TestLog::emit_unencrypted_events 1,781 (+101) +6.01%
AvmTest::keccak_hash 2,708 (+153) +5.99%
Test::consume_message_from_arbitrary_sender_public 6,874 (+383) +5.90%
ImportTest::public_dispatch 1,723 (+96) +5.90%
Parent::public_nested_static_call 3,897 (+215) +5.84%
AvmTest::sha256_hash 3,100 (+171) +5.84%
Test::consume_mint_public_message 9,047 (+494) +5.78%
AuthWitTest::public_dispatch 2,151 (+117) +5.75%
Parent::public_dispatch 8,482 (+461) +5.75%
DocsExample::public_dispatch 5,288 (+285) +5.70%
TokenBridge::claim_public 10,462 (+563) +5.69%
StaticChild::public_dispatch 2,920 (+157) +5.68%
DocsExample::get_shared_immutable_constrained_public_multiple 727 (+39) +5.67%
AvmTest::nested_call_to_add_with_gas 839 (+45) +5.67%
PriceFeed::public_dispatch 3,727 (+199) +5.64%
Crowdfunding::_publish_donation_receipts 1,275 (+68) +5.63%
Child::pub_inc_value 686 (+36) +5.54%
StaticChild::pub_inc_value 686 (+36) +5.54%
StaticParent::public_dispatch 5,922 (+310) +5.52%
StaticChild::pub_illegal_inc_value 707 (+37) +5.52%
AuthWitTest::consume_public 883 (+46) +5.50%
AvmTest::nested_call_to_add 864 (+45) +5.49%
AvmTest::nested_call_to_assert_same 864 (+45) +5.49%
AvmTest::nested_static_call_to_add 864 (+45) +5.49%
Child::pub_inc_value_internal 712 (+37) +5.48%
Child::public_dispatch 5,677 (+294) +5.46%
Child::set_value_twice_with_nested_first 917 (+47) +5.40%
Child::set_value_twice_with_nested_last 917 (+47) +5.40%
Parent::public_static_call 979 (+50) +5.38%
StaticParent::public_static_call 979 (+50) +5.38%
Benchmarking::public_dispatch 4,977 (+254) +5.38%
Uniswap::_approve_bridge_and_exit_input_asset_to_L1 5,978 (+304) +5.36%
TokenBridge::_call_mint_on_token 1,242 (+63) +5.34%
StatefulTest::increment_public_value_no_init_check 1,865 (+94) +5.31%
AvmTest::variable_base_msm 457 (+23) +5.30%
DocsExample::initialize_shared_immutable 795 (+40) +5.30%
DocsExample::initialize_public_immutable 795 (+40) +5.30%
Spam::public_dispatch 3,524 (+177) +5.29%
Lending::get_asset 1,993 (+100) +5.28%
Benchmarking::broadcast 1,796 (+90) +5.28%
Router::public_dispatch 2,862 (+143) +5.26%
StatefulTest::increment_public_value 1,893 (+94) +5.23%
PriceFeed::get_price 1,781 (+88) +5.20%
AvmTest::read_storage_list 632 (+31) +5.16%
Benchmarking::increment_balance 3,062 (+150) +5.15%
FeeJuice::balance_of_public 1,842 (+90) +5.14%
NFT::set_minter 1,925 (+94) +5.13%
Token::set_minter 1,925 (+94) +5.13%
NFT::transfer_in_public 3,712 (+181) +5.13%
Lending::_deposit 2,322 (+113) +5.12%
Child::pub_set_value 329 (+16) +5.11%
StaticChild::pub_set_value 329 (+16) +5.11%
StatefulTest::get_public_value 1,750 (+85) +5.11%
StaticParent::public_nested_static_call 1,321 (+64) +5.09%
AuthRegistry::set_reject_all 1,653 (+80) +5.09%
PriceFeed::set_price 1,737 (+84) +5.08%
AvmTest::add_storage_map 2,070 (+100) +5.08%
AvmTest::read_storage_map 1,739 (+84) +5.08%
AuthRegistry::is_reject_all 1,760 (+85) +5.07%
AvmTest::set_read_storage_single 602 (+29) +5.06%
Router::_check_block_number 1,290 (+62) +5.05%
TokenBlacklist::balance_of_public 1,875 (+90) +5.04%
Token::balance_of_public 1,875 (+90) +5.04%
AuthRegistry::set_authorized 1,813 (+87) +5.04%
StaticParent::public_get_value_from_child 710 (+34) +5.03%
AuthRegistry::_set_authorized 1,840 (+88) +5.02%
AuthRegistry::is_consumable 1,925 (+92) +5.02%
NFT::_finish_transfer_to_public 1,696 (+81) +5.02%
Token::assert_minter_and_mint 2,326 (+111) +5.01%
FeeJuice::_increase_public_balance 2,353 (+112) +5.00%
AvmTest::assert_calldata_copy 442 (+21) +4.99%
Router::_check_timestamp 1,307 (+62) +4.98%
NFT::is_minter 1,793 (+85) +4.98%
Token::is_minter 1,793 (+85) +4.98%
Lending::init 2,912 (+138) +4.97%
Parent::pub_entry_point 719 (+34) +4.96%
StaticParent::public_call 719 (+34) +4.96%
AvmTest::create_different_nullifier_in_nested_call 698 (+33) +4.96%
Child::set_value_with_two_nested_calls 1,313 (+62) +4.96%
Auth::set_authorized_delay 5,614 (+265) +4.95%
Lending::repay_public 1,658 (+78) +4.94%
AvmTest::nested_static_call_to_set_storage 681 (+32) +4.93%
FPC::pay_refund 1,705 (+80) +4.92%
ImportTest::pub_call_public_fn 682 (+32) +4.92%
Lending::deposit_public 1,769 (+83) +4.92%
Auth::set_authorized 5,696 (+267) +4.92%
AuthRegistry::consume 2,989 (+140) +4.91%
Spam::public_spam 2,541 (+119) +4.91%
FeeJuice::public_dispatch 5,879 (+275) +4.91%
TokenBlacklist::_reduce_total_supply 920 (+43) +4.90%
Token::_reduce_total_supply 920 (+43) +4.90%
AvmTest::set_storage_map 1,760 (+82) +4.89%
AvmTest::create_same_nullifier_in_nested_call 687 (+32) +4.89%
AuthRegistry::public_dispatch 8,031 (+374) +4.88%
Auth::get_authorized 1,827 (+85) +4.88%
AvmTest::elliptic_curve_add 516 (+24) +4.88%
EasyPrivateVoting::end_vote 645 (+30) +4.88%
NFT::set_admin 646 (+30) +4.87%
Token::set_admin 646 (+30) +4.87%
FPC::pay_refund_with_shielded_rebate 1,637 (+76) +4.87%
DocsExample::get_shared_immutable_constrained_public_indirect 432 (+20) +4.85%
Auth::get_scheduled_authorized 1,730 (+80) +4.85%
Parent::pub_entry_point_twice 1,190 (+55) +4.85%
Lending::borrow_public 1,170 (+54) +4.84%
Lending::withdraw_public 1,170 (+54) +4.84%
TokenBlacklist::_increase_public_balance 2,579 (+119) +4.84%
Token::_increase_public_balance 2,579 (+119) +4.84%
AvmTest::poseidon2_hash 1,564 (+72) +4.83%
EasyPrivateVoting::add_to_tally_public 2,289 (+105) +4.81%
Token::transfer_public 4,799 (+217) +4.74%
TokenBlacklist::get_roles 2,545 (+115) +4.73%
FPC::prepare_fee 1,241 (+56) +4.73%
AvmTest::get_args_hash 1,640 (+74) +4.73%
TokenBlacklist::burn_public 5,815 (+262) +4.72%
Token::burn_public 4,686 (+211) +4.72%
FeeJuice::check_balance 2,165 (+97) +4.69%
DocsExample::get_shared_immutable_constrained_public 539 (+24) +4.66%
TokenBlacklist::transfer_public 6,230 (+275) +4.62%
AvmTest::public_dispatch 56,974 (+2,513) +4.61%
NFT::public_dispatch 22,292 (+983) +4.61%
AvmTest::read_storage_single 457 (+20) +4.58%
TokenBlacklist::update_roles 7,268 (+318) +4.58%
AvmTest::return_oracle 345 (+15) +4.55%
TokenBlacklist::total_supply 613 (+26) +4.43%
Token::total_supply 613 (+26) +4.43%
Lending::public_dispatch 26,507 (+1,123) +4.42%
NFT::mint 3,049 (+129) +4.42%
Lending::get_assets 804 (+34) +4.42%
NFT::owner_of 2,507 (+106) +4.41%
FeeJuice::set_portal 949 (+40) +4.40%
AvmTest::set_storage_list 262 (+11) +4.38%
Uniswap::public_dispatch 24,534 (+1,028) +4.37%
Auth::get_authorized_delay 2,135 (+89) +4.35%
TokenBridge::_assert_token_is_same 528 (+22) +4.35%
AvmTest::set_storage_single 241 (+10) +4.33%
Token::mint_public 3,232 (+134) +4.33%
NFT::public_get_symbol 511 (+21) +4.29%
TokenBridge::get_token 511 (+21) +4.29%
Token::get_admin 511 (+21) +4.29%
TokenBlacklist::mint_public 4,290 (+176) +4.28%
AvmInitializerTest::read_storage_immutable 490 (+20) +4.26%
NFT::get_admin 516 (+21) +4.24%
NFT::public_get_name 516 (+21) +4.24%
Token::public_get_name 516 (+21) +4.24%
Token::public_get_symbol 516 (+21) +4.24%
Uniswap::_assert_token_is_same 467 (+19) +4.24%
Lending::_withdraw 7,689 (+312) +4.23%
DelegatedOn::public_set_value 247 (+10) +4.22%
Lending::_borrow 7,629 (+308) +4.21%
AvmTest::elliptic_curve_add_and_double 645 (+26) +4.20%
Lending::update_accumulator 5,329 (+214) +4.18%
Lending::get_position 4,831 (+194) +4.18%
Token::public_dispatch 28,394 (+1,140) +4.18%
DocsExample::update_leader 300 (+12) +4.17%
Token::public_get_decimals 531 (+21) +4.12%
Lending::_repay 5,314 (+206) +4.03%
TokenBlacklist::public_dispatch 25,359 (+976) +4.00%
TokenBridge::get_portal_address_public 548 (+21) +3.98%
TokenBridge::public_dispatch 20,450 (+764) +3.88%
NFT::_store_point_in_transient_storage 350 (+13) +3.86%
StaticChild::pub_get_value 168 (+6) +3.70%
Child::pub_get_value 142 (+5) +3.65%
AvmTest::add_u128 208 (+7) +3.48%
AvmTest::check_selector 181 (+6) +3.43%
AvmTest::to_radix_le 243 (+8) +3.40%
AvmTest::get_function_selector 166 (+5) +3.11%
Test::assert_public_global_vars 250 (+7) +2.88%
AvmTest::assertion_failure 159 (+4) +2.58%
AvmTest::keccak_f1600 478 (+11) +2.36%
Test::dummy_public_call 107 (+2) +1.90%
AvmTest::add_args_return 109 (+2) +1.87%
AvmTest::assert_timestamp 115 (+2) +1.77%
AvmTest::assert_same 120 (+2) +1.69%
Test::is_time_equal 121 (+2) +1.68%
Auth::public_dispatch 10,763 (+150) +1.41%
FPC::public_dispatch 9,225 (+104) +1.14%
AvmTest::new_note_hash 91 (+1) +1.11%
AvmTest::new_nullifier 91 (+1) +1.11%
Test::emit_nullifier_public 91 (+1) +1.11%
AvmTest::get_fee_per_da_gas 92 (+1) +1.10%
AvmTest::get_sender 92 (+1) +1.10%
AvmTest::get_fee_per_l2_gas 92 (+1) +1.10%
AvmTest::get_version 92 (+1) +1.10%
AvmTest::get_transaction_fee 92 (+1) +1.10%
AvmTest::get_address 92 (+1) +1.10%
AvmTest::get_block_number 92 (+1) +1.10%
AvmTest::get_chain_id 92 (+1) +1.10%
AvmTest::get_timestamp 92 (+1) +1.10%
AvmTest::get_storage_address 92 (+1) +1.10%
AvmTest::set_opcode_u8 92 (+1) +1.10%
AvmTest::get_l2_gas_left 92 (+1) +1.10%
AvmTest::get_da_gas_left 92 (+1) +1.10%
AvmTest::nullifier_collision 95 (+1) +1.06%
AvmTest::set_opcode_u32 96 (+1) +1.05%
AvmTest::send_l2_to_l1_msg 99 (+1) +1.02%
Test::create_l2_to_l1_message_arbitrary_recipient_public 99 (+1) +1.02%
AvmTest::set_opcode_u64 100 (+1) +1.01%
DocsExample::spend_public_authwit 102 (+1) +0.99%
AvmTest::set_opcode_small_field 108 (+1) +0.93%
AvmTest::nullifier_exists 110 (+1) +0.92%
AvmTest::l1_to_l2_msg_exists 111 (+1) +0.91%
AvmTest::note_hash_exists 111 (+1) +0.91%
AvmTest::modulo2 114 (+1) +0.88%
AvmTest::assert_nullifier_exists 115 (+1) +0.88%
AvmTest::emit_nullifier_and_check 119 (+1) +0.85%
AvmTest::set_opcode_big_field 124 (+1) +0.81%
AvmTest::set_opcode_really_big_field 124 (+1) +0.81%
InclusionProofs::push_nullifier_public 124 (+1) +0.81%
InclusionProofs::test_nullifier_inclusion_from_public 143 (+1) +0.70%
TokenBlacklist::constructor 8,020 (+36) +0.45%
StatefulTest::public_dispatch 7,752 (+31) +0.40%
Crowdfunding::public_dispatch 6,356 (-28) -0.44%
EasyPrivateVoting::public_dispatch 6,193 (-58) -0.93%
AppSubscription::public_dispatch 5,260 (-103) -1.92%
Token::constructor 5,332 (-124) -2.27%
Claim::public_dispatch 4,473 (-112) -2.44%
PrivateFPC::public_dispatch 4,305 (-112) -2.54%
NFT::constructor 4,980 (-133) -2.60%
InclusionProofs::public_dispatch 4,345 (-131) -2.93%
AppSubscription::constructor 3,939 (-174) -4.23%
Crowdfunding::init 3,788 (-178) -4.49%
StatefulTest::public_constructor 3,660 (-172) -4.49%
AvmInitializerTest::public_dispatch 3,587 (-169) -4.50%
Claim::constructor 3,501 (-170) -4.63%
TokenBridge::constructor 3,480 (-170) -4.66%
PrivateFPC::constructor 3,338 (-170) -4.85%
EasyPrivateVoting::constructor 3,488 (-186) -5.06%
Auth::constructor 3,343 (-188) -5.32%
FPC::constructor 3,321 (-190) -5.41%
Uniswap::constructor 3,321 (-190) -5.41%
AvmInitializerTest::constructor 2,823 (-207) -6.83%
InclusionProofs::constructor 2,869 (-211) -6.85%
AvmTest::test_get_contract_instance 862 (-306) -26.20%
AvmTest::test_get_contract_instance_raw 388 (-421) -52.04%
AvmTest::pedersen_commit 373 (-729) -66.15%

@github-actions GitHub Actions
Copy link
Contributor

Changes to circuit sizes

Generated at commit: f721b3a9ac24fcfb0234f61de7ecb58011dd11f8, compared to commit: ab0c80d7493e6bdbc58dcd517b248de6ddd6fd67

🧾 Summary (100% most significant diffs)

Program ACIR opcodes (+/-) % Circuit size (+/-) %
empty_nested 0 ➖ +∞% +5 ❌ +31.25%
empty_nested_simulated 0 ➖ 0.00% +5 ❌ +31.25%
private_kernel_empty_simulated 0 ➖ 0.00% +5 ❌ +31.25%
rollup_base -58 ✅ -0.01% -518 ✅ -0.02%
public_kernel_tail -9 ✅ -0.00% -461 ✅ -0.02%
public_kernel_merge -62 ✅ -0.12% -520 ✅ -0.05%
parity_root -28 ✅ -0.52% -1,847 ✅ -0.05%
rollup_block_root -128 ✅ -3.01% -1,513 ✅ -0.05%
rollup_block_merge -120 ✅ -1.65% -1,048 ✅ -0.06%
rollup_root -120 ✅ -1.65% -1,050 ✅ -0.06%
rollup_merge -122 ✅ -3.21% -1,051 ✅ -0.06%
private_kernel_reset -59 ✅ -0.06% -520 ✅ -0.11%
private_kernel_reset_4_4_4_4_4_4_4_4_1 -59 ✅ -0.17% -519 ✅ -0.69%
private_kernel_inner -61 ✅ -0.14% -523 ✅ -0.99%
private_kernel_tail_to_public -62 ✅ -0.21% -523 ✅ -1.31%
private_kernel_tail -62 ✅ -1.32% -524 ✅ -5.49%
private_kernel_empty 0 ➖ 0.00% -936,476 ✅ -99.63%
Full diff report 👇
Program ACIR opcodes (+/-) % Circuit size (+/-) %
empty_nested 0 (0) +∞% 21 (+5) +31.25%
empty_nested_simulated 1 (0) 0.00% 21 (+5) +31.25%
private_kernel_empty_simulated 1 (0) 0.00% 21 (+5) +31.25%
rollup_base 483,173 (-58) -0.01% 3,355,737 (-518) -0.02%
public_kernel_tail 258,424 (-9) -0.00% 2,270,260 (-461) -0.02%
public_kernel_merge 53,532 (-62) -0.12% 1,103,602 (-520) -0.05%
parity_root 5,399 (-28) -0.52% 3,775,125 (-1,847) -0.05%
rollup_block_root 4,124 (-128) -3.01% 2,837,267 (-1,513) -0.05%
rollup_block_merge 7,169 (-120) -1.65% 1,899,418 (-1,048) -0.06%
rollup_root 7,153 (-120) -1.65% 1,899,402 (-1,050) -0.06%
rollup_merge 3,673 (-122) -3.21% 1,896,246 (-1,051) -0.06%
private_kernel_reset 91,933 (-59) -0.06% 467,730 (-520) -0.11%
private_kernel_reset_4_4_4_4_4_4_4_4_1 34,897 (-59) -0.17% 74,479 (-519) -0.69%
private_kernel_inner 44,042 (-61) -0.14% 52,354 (-523) -0.99%
private_kernel_tail_to_public 29,828 (-62) -0.21% 39,510 (-523) -1.31%
private_kernel_tail 4,648 (-62) -1.32% 9,021 (-524) -5.49%
private_kernel_empty 671 (0) 0.00% 3,486 (-936,476) -99.63%

@critesjosh critesjosh merged commit 2a4188c into master Oct 20, 2024
56 checks passed
@critesjosh critesjosh deleted the jc/docs-debug branch October 20, 2024 08:30
@AztecBot AztecBot mentioned this pull request Oct 20, 2024
Merged
spypsy pushed a commit that referenced this pull request Oct 21, 2024
@AztecBot
chore(master): Release 0.59.0 (#9281)
6ce20e9 
🤖 I have created a release *beep* *boop*
---


<details><summary>aztec-package: 0.59.0</summary>

##
[0.59.0](aztec-package-v0.58.0...aztec-package-v0.59.0)
(2024-10-21)


### ⚠ BREAKING CHANGES

* **seq:** disable sequencer and disable validator as one env var,
update p2p listen port names
([#9266](#9266))

### Miscellaneous

* **seq:** Disable sequencer and disable validator as one env var,
update p2p listen port names
([#9266](#9266))
([367c38c](367c38c))
</details>

<details><summary>barretenberg.js: 0.59.0</summary>

##
[0.59.0](barretenberg.js-v0.58.0...barretenberg.js-v0.59.0)
(2024-10-21)


### Bug Fixes

* Publish readme on bb.js NPM package
([#9303](#9303))
([1d860a8](1d860a8))
</details>

<details><summary>aztec-packages: 0.59.0</summary>

##
[0.59.0](aztec-packages-v0.58.0...aztec-packages-v0.59.0)
(2024-10-21)


### ⚠ BREAKING CHANGES

* **seq:** disable sequencer and disable validator as one env var,
update p2p listen port names
([#9266](#9266))

### Bug Fixes

* **docs:** Dapp tutorial edits
([#8695](#8695))
([f95bcff](f95bcff))
* **docs:** Update debugging docs
([#9200](#9200))
([2a4188c](2a4188c))
* Publish readme on bb.js NPM package
([#9303](#9303))
([1d860a8](1d860a8))


### Miscellaneous

* Re-enable test fixed by Native World State
([#9289](#9289))
([3fd1802](3fd1802)),
closes
[#8306](#8306)
* Replace relative paths to noir-protocol-circuits
([ceeab4e](ceeab4e))
* **seq:** Disable sequencer and disable validator as one env var,
update p2p listen port names
([#9266](#9266))
([367c38c](367c38c))
</details>

<details><summary>barretenberg: 0.59.0</summary>

##
[0.59.0](barretenberg-v0.58.0...barretenberg-v0.59.0)
(2024-10-21)


### Miscellaneous

* **barretenberg:** Synchronize aztec-packages versions
</details>

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
AztecBot added a commit to AztecProtocol/barretenberg that referenced this pull request Oct 22, 2024
@AztecBot
chore(master): Release 0.59.0 (#9281)
5098f9a 
🤖 I have created a release *beep* *boop*
---


<details><summary>aztec-package: 0.59.0</summary>

##
[0.59.0](AztecProtocol/aztec-packages@aztec-package-v0.58.0...aztec-package-v0.59.0)
(2024-10-21)


### ⚠ BREAKING CHANGES

* **seq:** disable sequencer and disable validator as one env var,
update p2p listen port names
([#9266](AztecProtocol/aztec-packages#9266))

### Miscellaneous

* **seq:** Disable sequencer and disable validator as one env var,
update p2p listen port names
([#9266](AztecProtocol/aztec-packages#9266))
([367c38c](AztecProtocol/aztec-packages@367c38c))
</details>

<details><summary>barretenberg.js: 0.59.0</summary>

##
[0.59.0](AztecProtocol/aztec-packages@barretenberg.js-v0.58.0...barretenberg.js-v0.59.0)
(2024-10-21)


### Bug Fixes

* Publish readme on bb.js NPM package
([#9303](AztecProtocol/aztec-packages#9303))
([1d860a8](AztecProtocol/aztec-packages@1d860a8))
</details>

<details><summary>aztec-packages: 0.59.0</summary>

##
[0.59.0](AztecProtocol/aztec-packages@aztec-packages-v0.58.0...aztec-packages-v0.59.0)
(2024-10-21)


### ⚠ BREAKING CHANGES

* **seq:** disable sequencer and disable validator as one env var,
update p2p listen port names
([#9266](AztecProtocol/aztec-packages#9266))

### Bug Fixes

* **docs:** Dapp tutorial edits
([#8695](AztecProtocol/aztec-packages#8695))
([f95bcff](AztecProtocol/aztec-packages@f95bcff))
* **docs:** Update debugging docs
([#9200](AztecProtocol/aztec-packages#9200))
([2a4188c](AztecProtocol/aztec-packages@2a4188c))
* Publish readme on bb.js NPM package
([#9303](AztecProtocol/aztec-packages#9303))
([1d860a8](AztecProtocol/aztec-packages@1d860a8))


### Miscellaneous

* Re-enable test fixed by Native World State
([#9289](AztecProtocol/aztec-packages#9289))
([3fd1802](AztecProtocol/aztec-packages@3fd1802)),
closes
[#8306](AztecProtocol/aztec-packages#8306)
* Replace relative paths to noir-protocol-circuits
([ceeab4e](AztecProtocol/aztec-packages@ceeab4e))
* **seq:** Disable sequencer and disable validator as one env var,
update p2p listen port names
([#9266](AztecProtocol/aztec-packages#9266))
([367c38c](AztecProtocol/aztec-packages@367c38c))
</details>

<details><summary>barretenberg: 0.59.0</summary>

##
[0.59.0](AztecProtocol/aztec-packages@barretenberg-v0.58.0...barretenberg-v0.59.0)
(2024-10-21)


### Miscellaneous

* **barretenberg:** Synchronize aztec-packages versions
</details>

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ludamad ludamad ludamad approved these changes

@catmcgee catmcgee Awaiting requested review from catmcgee

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

Successfully merging this pull request may close these issues.

[docs]: Update Debugging page
3 participants
@critesjosh @AztecBot @ludamad