Make epoch period flexible #118
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ChainDev931105
force-pushed
the
feat/period_flex
branch
from
3ccb103 to
06ab37f
Compare
There was a problem hiding this comment.
@ChainDev931105 we did upgrade the mainnet with extra epoch param storage, do you think this approach is compatible with the deployed one?
That makes sense. I think I need to make some simulation test forking from current network, @cool-develope |
cool-develope
approved these changes
Jun 12, 2024
3 tasks
ChainDev931105
added a commit
that referenced
this pull request
Jul 30, 2024
* fix: clean up and add LICENSE (#78) * clean up code * clean build * update license * add audit reports * fix committee.test.js (#79) * revert epoch (#85) * adjust operator count (#84) * add workflow (#89) * feat: update the BLS key and signer address (#88) * allow to update the bls keys and signer address * add tests --------- Co-authored-by: Master Engineer <[email protected]> * FEAT: Allow to update epoch period (#93) * epoch_modifi draft * update epoch formula * fix reviewed issues * fix reviewed issues * fix naming of getEpochNumber * epoch period modify unit test (#97) * CHORE: Epoch period modify simulation (#98) * simulation test for epoch period modify * fix issue in redeloy script * format * fix: avoid to remove all bls pub keys (#102) * fix the remove bls keys * more tests * Workflow with solhint (#101) * solhint cli * update solhint checkpoints * Update contracts/protocol/VoteWeigher.sol Co-authored-by: cool-developer <[email protected]> --------- Co-authored-by: cool-developer <[email protected]> * FEAT: eject operator (#105) * feat eject operator * fix reviewed errors * strengthen test * unsubscribe multiple operators * FEAT: Deploy to Sepolia (#106) * deploy sepolia * testnet deploy script Co-authored-by: cool-developer <[email protected]> * add eigen sepolia addresses --------- Co-authored-by: cool-developer <[email protected]> * add fork test for rotate and revert (#107) * Docker workflow (#109) * push-docker yml * add permission makefile * push-docker-test * install forge build * remove test yml * CHORE: Script for register chain (#108) * add register chain script * update epoch period * update min/max weight * update genersis block * add basescript * script eject op * [FIX] Register operator sleep (#110) * fix register-operator sleep * remove test yml * archive mode geth (#112) * fix: geth docker cmd (#113) * archive mode geth * fix * [Fix] audited issues (#115) * fix audit issues * add audit report * Make epoch period flexible (#118) * epoch period flexible * test for epoch period flex * add simulation test for flex period * fix epoch period for prior blocks * flow auto release (#127) * add issue templates (#130) * Script - Transfer ownership (#121) * forge install: safe-smart-account 499b17ad0191b575fcadc5cb5b8e3faeae5391ae * script transfer ownership * Override for testnet committee - L1 Block Number (#123) * testnet l1bias override * fix annoying function naming * feat: bls key ownership check (#138) * bls key ownership check * unit test & inherit checker * optimize epoch number search * fork test bls own check * update storage hash --------- Co-authored-by: ChainDev931105 <[email protected]> * epic sequencer bls key (#140) * Revert "epic sequencer bls key (#140)" (#141) This reverts commit 1e57449. * fix deploy-register (#142) * fix * [FIX] Audit Issues - Main (#149) * Update README.md (#136) * [FIX] script redeploy testnet (#144) * fix script redeploy testnet * fix get_updated_l1_block * solc version 0.8.20, blsKeyChecker issue * digest hash add version field * hardhat solidity version update --------- Co-authored-by: cool-developer <[email protected]> --------- Co-authored-by: Master Engineer <[email protected]> Co-authored-by: ChainDev <[email protected]> Co-authored-by: Kashish Shah <[email protected]> Co-authored-by: ChainDev931105 <[email protected]>
ChainDev931105
added a commit
that referenced
this pull request
Aug 26, 2024
* fix: clean up and add LICENSE (#78) * clean up code * clean build * update license * add audit reports * fix committee.test.js (#79) * revert epoch (#85) * adjust operator count (#84) * add workflow (#89) * feat: update the BLS key and signer address (#88) * allow to update the bls keys and signer address * add tests --------- Co-authored-by: Master Engineer <[email protected]> * FEAT: Allow to update epoch period (#93) * epoch_modifi draft * update epoch formula * fix reviewed issues * fix reviewed issues * fix naming of getEpochNumber * epoch period modify unit test (#97) * CHORE: Epoch period modify simulation (#98) * simulation test for epoch period modify * fix issue in redeloy script * format * fix: avoid to remove all bls pub keys (#102) * fix the remove bls keys * more tests * Workflow with solhint (#101) * solhint cli * update solhint checkpoints * Update contracts/protocol/VoteWeigher.sol Co-authored-by: cool-developer <[email protected]> --------- Co-authored-by: cool-developer <[email protected]> * FEAT: eject operator (#105) * feat eject operator * fix reviewed errors * strengthen test * unsubscribe multiple operators * FEAT: Deploy to Sepolia (#106) * deploy sepolia * testnet deploy script Co-authored-by: cool-developer <[email protected]> * add eigen sepolia addresses --------- Co-authored-by: cool-developer <[email protected]> * add fork test for rotate and revert (#107) * Docker workflow (#109) * push-docker yml * add permission makefile * push-docker-test * install forge build * remove test yml * CHORE: Script for register chain (#108) * add register chain script * update epoch period * update min/max weight * update genersis block * add basescript * script eject op * [FIX] Register operator sleep (#110) * fix register-operator sleep * remove test yml * archive mode geth (#112) * fix: geth docker cmd (#113) * archive mode geth * fix * [Fix] audited issues (#115) * fix audit issues * add audit report * Make epoch period flexible (#118) * epoch period flexible * test for epoch period flex * add simulation test for flex period * fix epoch period for prior blocks * flow auto release (#127) * add issue templates (#130) * Script - Transfer ownership (#121) * forge install: safe-smart-account 499b17ad0191b575fcadc5cb5b8e3faeae5391ae * script transfer ownership * Override for testnet committee - L1 Block Number (#123) * testnet l1bias override * fix annoying function naming * feat: bls key ownership check (#138) * bls key ownership check * unit test & inherit checker * optimize epoch number search * fork test bls own check * update storage hash --------- Co-authored-by: ChainDev931105 <[email protected]> * epic sequencer bls key (#140) * Revert "epic sequencer bls key (#140)" (#141) This reverts commit 1e57449. * fix deploy-register (#142) * update readme --------- Co-authored-by: cool-developer <[email protected]> Co-authored-by: Master Engineer <[email protected]> Co-authored-by: Kashish Shah <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Context
Closes: #119
Reviewers