github-actions[bot]
commented
2 years ago | Gas costs | 3c722ad7720f2e1706d7901d6d8889d46ad92ab1 | 329f3f6f6414e32733323606b5dfc940f6c5bb49 | Diff |
|---|---|---|---|
| checker%touch | 46985 | None | -46985 |
| checker%create_burrow | 14788 | None | -14788 |
| checker%deactivate_burrow | 14495 | None | -14495 |
| checker%withdraw_collateral | 14161 | None | -14161 |
| checker%deposit_collateral | 11483 | None | -11483 |
| checker%activate_burrow | 11324 | None | -11324 |
| checker%set_burrow_delegate | 9679 | None | -9679 |
| checker%remove_liquidity | 8161 | None | -8161 |
| checker%add_liquidity | 7673 | None | -7673 |
| wtez%transfer | 7384 | None | -7384 |
| checker%buy_kit | 7350 | None | -7350 |
| checker%sell_kit | 7349 | None | -7349 |
| wtez%withdraw | 5628 | None | -5628 |
| wctez%mint | 5448 | None | -5448 |
| wctez%redeem | 5000 | None | -5000 |
| checker%burn_kit | 4591 | None | -4591 |
| wtez%set_delegate | 4457 | None | -4457 |
| checker%mint_kit | 4324 | None | -4324 |
| wtez%deposit | 3933 | None | -3933 |
| checker%transfer | 3098 | None | -3098 |
| wctez%transfer | 3036 | None | -3036 |
| mockFA2%transfer | 3025 | None | -3025 |
| checker%touch_burrow | 2982 | None | -2982 |
| checker%update_operators | 2586 | None | -2586 |
| mockFA2%redeem | 1995 | None | -1995 |
| mockFA2%mint | 1785 | None | -1785 |
| wctez%update_operators | 1703 | None | -1703 |
| wtez%update_operators | 1694 | None | -1694 |
| mockFA2%update_operators | 1692 | None | -1692 |
| Entrypoint sizes | 3c722ad7720f2e1706d7901d6d8889d46ad92ab1 | 329f3f6f6414e32733323606b5dfc940f6c5bb49 | Diff |
|---|---|---|---|
| touch | 56880 | None | -56880 |
| mark_for_liquidation | 17046 | None | -17046 |
| touch_liquidation_slices | 14350 | None | -14350 |
| cancel_liquidation_slice | 12122 | None | -12122 |
| liquidation_auction_place_bid | 2157 | None | -2157 |
| remove_liquidity | 1998 | None | -1998 |
| add_liquidity | 1878 | None | -1878 |
| mint_kit | 1531 | None | -1531 |
| burn_kit | 1514 | None | -1514 |
| deactivate_burrow | 1459 | None | -1459 |
| buy_kit | 1395 | None | -1395 |
| sell_kit | 1391 | None | -1391 |
| withdraw_collateral | 1292 | None | -1292 |
| activate_burrow | 1272 | None | -1272 |
| create_burrow | 1192 | None | -1192 |
| deposit_collateral | 1145 | None | -1145 |
| liquidation_auction_claim_win | 1096 | None | -1096 |
| touch_burrow | 611 | None | -611 |
| set_burrow_delegate | 457 | None | -457 |
| update_operators | 435 | None | -435 |
| receive_price | 289 | None | -289 |
| receive_ctez_marginal_price | 140 | None | -140 |
| Test coverage | 3c722ad7720f2e1706d7901d6d8889d46ad92ab1 | 329f3f6f6414e32733323606b5dfc940f6c5bb49 | Diff |
|---|---|---|---|
| fa2Implementation.ml | 100 | None | -100 |
| error.ml | 100 | None | -100 |
| cfmm.ml | 100 | None | -100 |
| liquidationAuctionTypes.ml | 100 | None | -100 |
| fa2Ledger.ml | 100 | None | -100 |
| ptr.ml | 100 | None | -100 |
| driftDerivative.ml | 100 | None | -100 |
| parameters.ml | 100 | None | -100 |
| mem.ml | 100 | None | -100 |
| fixedPoint.ml | 100 | None | -100 |
| kit.ml | 100 | None | -100 |
| tokenMetadata.ml | 100 | None | -100 |
| constants.ml | 100 | None | -100 |
| cfmmTypes.ml | 100 | None | -100 |
| targetCalculation.ml | 100 | None | -100 |
| checkerEntrypoints.ml | 97.73 | None | -97.73 |
| burrow.ml | 96.21 | None | -96.21 |
| sliceList.ml | 95.38 | None | -95.38 |
| checker.ml | 94.98 | None | -94.98 |
| checkerMain.ml | 93.33 | None | -93.33 |
| avl.ml | 91.05 | None | -91.05 |
| common.ml | 90.36 | None | -90.36 |
| checkerTypes.ml | 78.57 | None | -78.57 |
| tok.ml | 78.38 | None | -78.38 |
| liquidationAuction.ml | 77.83 | None | -77.83 |
| price.ml | 76.92 | None | -76.92 |
| getOracleEntrypoint.ml | 75 | None | -75 |
| lqt.ml | 72.97 | None | -72.97 |
| ctok.ml | 72.97 | None | -72.97 |
| mockFA2.ml | 18.52 | None | -18.52 |
| wctez.ml | 14.29 | None | -14.29 |
| burrowOrigination.ml | 8 | None | -8 |
| wtez.ml | 4.78 | None | -4.78 |
| TOTAL | 81.88 | None | -81.88 |
This PR combines our two main python packages,
checker-clientandchecker-builderinto a single package and removes the (largely unused) packaging for the e2e tests. Normally it makes sense to create smaller, self-contained packages like this, but in our case they have become more tightly coupled through time and having them as separate dependencies in the top-levelpyproject.tomlimpedes effective build caching since any changes to our code forces a re-build of the python virtual environment.These changes mean that making changes to things like the end to end tests will no longer force earthly to reinstall all of the python dependencies, significantly speeding up build times.
Also updated our python formatting script since I discovered a small bug in it while updating our packaging.