Skip to content

feat[venom]: gas bench#4945

Open
harkal wants to merge 5 commits intovyperlang:masterfrom
harkal:feat/ci/gas-bench
Open

feat[venom]: gas bench#4945
harkal wants to merge 5 commits intovyperlang:masterfrom
harkal:feat/ci/gas-bench

Conversation

@harkal
Copy link
Copy Markdown
Collaborator

@harkal harkal commented Apr 30, 2026

What I did

How I did it

How to verify it

Commit message

Commit message for the final, squashed PR. (Optional, but reviewers will appreciate it! Please see our commit message style guide for what we would ideally like to see in a commit message.)

Description for the changelog

Cute Animal Picture

Put a link to a cute animal picture inside the parenthesis-->

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 30, 2026

📊 Bytecode Size Changes (venom)

No changes detected.

Full bytecode sizes

Contract legacy-O2 legacy-Os -O2 -O3 -Os
curvefi/amm/tricryptoswap/implementation/implementation_v_200.vy
curvefi/amm/tricryptoswap/math/math_v_200.vy
curvefi/amm/twocryptoswap/implementation/implementation_v_210.vy
curvefi/amm/twocryptoswap/math/math_v_210.vy
curvefi/legacy/CurveCryptoMathOptimized3.vy
curvefi/amm/stableswap/meta_implementation/meta_implementation_v_700.vy 23610 22805 20884 19821 19383
curvefi/legacy/CurveStableSwapMetaNG.vy 24952 23578 20721 20072 19376
curvefi/amm/stableswap/implementation/implementation_v_700.vy 24962 23769 20546 19645 19097
curvefi/legacy/CurveStableSwapNG.vy 24473 23298 19939 19065 18546
yearnfi/VaultV3.vy 19972 19063 17139 15221 14569
curvefi/legacy/CurveCryptoSwap2.vy 18947 18382 15626 15068 14703
yearnfi/VaultV2.vy 16676 15763 13934 13446 12615
curvefi/amm/stableswap/factory/factory_v_100.vy 14558 13978 13352 11920 11901
curvefi/amm/stableswap/views/views_v_120.vy 12784 12368 10390 9748 9892
curvefi/gauge/child_gauge/implementation/implementation_v_110.vy 12338 11561 10345 9811 9227
curvefi/gauge/child_gauge/implementation/implementation_v_100.vy 12017 11249 10073 9538 8965
curvefi/gauge/child_gauge/implementation/implementation_v_020.vy 10665 9947 9141 8608 8096
curvefi/helpers/router/router_v_110.vy 6717 6717 7068 6407 6642
curvefi/amm/tricryptoswap/views/views_v_200.vy 7821 7776 6780 6498 6567
curvefi/registries/metaregistry/metaregistry_v_110.vy 7590 6732 6716 5923 5762
curvefi/helpers/stable_swap_meta_zap/stable_swap_meta_zap_v_100.vy 7302 7067 6595 6198 6300
curvefi/amm/twocryptoswap/views/views_v_200.vy 6991 6946 6337 6061 6124
curvefi/registries/metaregistry/registry_handlers/stableswap/handler_v_110.vy 6633 6259 5982 5157 5625
curvefi/amm/twocryptoswap/factory/factory_v_200.vy 5540 5252 5931 4654 4825
curvefi/amm/tricryptoswap/factory/factory_v_200.vy 5246 5021 5921 4916 5028
curvefi/gauge/child_gauge/factory/factory_v_201.vy 4844 4547 4342 4060 3826
yearnfi/VaultFactory.vy 3765 3617 4017 2520 2999
curvefi/registries/metaregistry/registry_handlers/tricryptoswap/handler_v_110.vy 4241 3939 3895 3574 3574
curvefi/registries/metaregistry/registry_handlers/twocryptoswap/handler_v_110.vy 4186 3884 3834 3444 3460
curvefi/gauge/child_gauge/factory/factory_v_100.vy 4183 3914 3768 3472 3257
curvefi/helpers/rate_provider/rate_provider_v_101.vy 3260 3260 2862 2490 2503
curvefi/registries/address_provider/address_provider_v_201.vy 2973 2782 2790 2616 2471
curvefi/amm/stableswap/math/math_v_100.vy 3067 3046 2655 2430 2450
curvefi/helpers/rate_provider/rate_provider_v_100.vy 2847 2841 2428 2091 2099
curvefi/helpers/deposit_and_stake_zap/deposit_and_stake_zap_v_100.vy 2322 2316 2184 1920 1952
curvefi/governance/relayer/taiko/relayer_v_001.vy 2068 2064 1712 1567 1600
curvefi/governance/relayer/polygon_cdk/relayer_v_101.vy 1556 1523 1519 1397 1398
curvefi/governance/relayer/arb_orbit/relayer_v_101.vy 1266 1262 1207 1111 1142
curvefi/governance/relayer/op_stack/relayer_v_101.vy 1186 1182 1146 1053 1081
curvefi/governance/relayer/not_rollup/relayer_v_100.vy 1168 1153 1135 1048 1060
curvefi/governance/vault/vault_v_100.vy 964 941 940 891 889
curvefi/governance/agent/agent_v_100.vy 541 541 498 440 444
curvefi/governance/agent/agent_v_101.vy 541 541 498 440 444
curvefi/governance/relayer/relayer_v_100.vy 496 496 467 462 467

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 30, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.62%. Comparing base (e630d29) to head (4167322).

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #4945   +/-   ##
=======================================
  Coverage   92.62%   92.62%           
=======================================
  Files         186      186           
  Lines       27299    27299           
  Branches     4695     4695           
=======================================
  Hits        25285    25285           
  Misses       1376     1376           
  Partials      638      638           

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@harkal harkal marked this pull request as ready for review April 30, 2026 16:08
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: c5f166f82b

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread .github/workflows/gas-bench.yml
Comment thread .github/scripts/compare_gas.py
@harkal harkal requested a review from charles-cooper April 30, 2026 18:55
uses: actions/checkout@v4
with:
repository: pcaversaccio/snekmate
ref: 37e1513dcb993dcc7e58f8c56aaeaaa14f8d8be8
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

should we use master? or

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.

2 participants