Skip to content

Conversation

@algolucky
Copy link
Contributor

Summary

  • this changes the LastParKeyRound of the mmnet recipe to 50,000 instead of 3,000,000. this should speed up network bootstrapping.

@algolucky algolucky requested a review from a team October 4, 2022 12:59
@algolucky algolucky self-assigned this Oct 4, 2022
@codecov
Copy link

codecov bot commented Oct 4, 2022

Codecov Report

Merging #4625 (5bc6503) into master (fdef01f) will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #4625      +/-   ##
==========================================
+ Coverage   54.18%   54.20%   +0.01%     
==========================================
  Files         402      402              
  Lines       51828    51828              
==========================================
+ Hits        28084    28094      +10     
+ Misses      21379    21371       -8     
+ Partials     2365     2363       -2     
Impacted Files Coverage Δ
agreement/cryptoVerifier.go 67.60% <0.00%> (-2.12%) ⬇️
agreement/proposalManager.go 96.07% <0.00%> (-1.97%) ⬇️
data/transactions/verify/txn.go 46.19% <0.00%> (-0.96%) ⬇️
network/wsNetwork.go 64.57% <0.00%> (-0.20%) ⬇️
cmd/tealdbg/debugger.go 73.49% <0.00%> (+0.80%) ⬆️
ledger/tracker.go 74.89% <0.00%> (+0.85%) ⬆️
crypto/merkletrie/node.go 93.48% <0.00%> (+1.86%) ⬆️
network/wsPeer.go 67.64% <0.00%> (+2.13%) ⬆️
crypto/merkletrie/trie.go 68.61% <0.00%> (+2.18%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Copy link
Contributor

@excalq excalq left a comment

Choose a reason for hiding this comment

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

Looks good.

@onetechnical onetechnical merged commit a5a4b10 into algorand:master Oct 4, 2022
@algolucky algolucky deleted the mmnet/50000-lastpartkeyround branch October 4, 2022 15: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.

4 participants