Skip to content

Conversation

@algochoi
Copy link
Contributor

@algochoi algochoi commented Sep 13, 2021

Summary

This commit makes consensus parameter changes for V30.

v30 introduces AVM 1.0 and TEAL 5, increases the app opt in limit to 50, and allows costs to be pooled in grouped stateful transactions.

@algorandskiy algorandskiy changed the title [WIP] Consensus V30 updates Consensus V30 updates Sep 14, 2021
@codecov-commenter
Copy link

codecov-commenter commented Sep 14, 2021

Codecov Report

Merging #2877 (65b05cf) into master (bbe8865) will decrease coverage by 4.04%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2877      +/-   ##
==========================================
- Coverage   47.28%   43.24%   -4.05%     
==========================================
  Files         355      355              
  Lines       57195    57199       +4     
==========================================
- Hits        27044    24734    -2310     
- Misses      27093    29477    +2384     
+ Partials     3058     2988      -70     
Impacted Files Coverage Δ
protocol/consensus.go 0.00% <ø> (ø)
config/consensus.go 84.48% <100.00%> (+0.20%) ⬆️
crypto/merklearray/worker.go 0.00% <0.00%> (-100.00%) ⬇️
crypto/compactcert/structs.go 0.00% <0.00%> (-100.00%) ⬇️
agreement/agreementtest/keyManager.go 0.00% <0.00%> (-100.00%) ⬇️
daemon/algod/api/server/lib/middlewares/auth.go 0.00% <0.00%> (-100.00%) ⬇️
agreement/agreementtest/simulate.go 0.00% <0.00%> (-87.68%) ⬇️
cmd/algofix/deadlock.go 3.12% <0.00%> (-84.38%) ⬇️
crypto/merklearray/partial.go 0.00% <0.00%> (-84.10%) ⬇️
ledger/onlineacct.go 17.64% <0.00%> (-82.36%) ⬇️
... and 90 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bbe8865...65b05cf. Read the comment docs.

onetechnical
onetechnical previously approved these changes Sep 14, 2021
Copy link
Contributor

@onetechnical onetechnical left a comment

Choose a reason for hiding this comment

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

Didn't verify tests, but the rest of it looks good to me.

@algojohnlee algojohnlee merged commit b7a5b82 into algorand:master Sep 15, 2021
@algochoi algochoi deleted the algochoi/consensus-v30 branch February 17, 2022 22:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants