algorand / graviton

🧑‍🔬 verify your TEAL program by experiment and observation
MIT License
17 stars 8 forks source link

Stop using dry-run's cost #20

Closed tzaffi closed 1 year ago

tzaffi commented 2 years ago

Cost is Deprecated in go-algorand's Dry Run

As of go-algorand PR #3957 the dry run response transaction top-level-field cost is being deprecated. There is one usage of this field in non-deprecated graviton code.

The good news is that new better fields BudgetConsumed and BudgetAdded are being introduced in the same PR and that the cost can be calculated as

net cost = BudgetConsumed - BudgetAdded

Action items

michaeldiamant commented 1 year ago

Closing - https://github.com/algorand/graviton/releases/tag/v0.5.0 exists and https://github.com/algorand/pyteal/pull/607 uses the functionality to implement OpUp tests.