Merge pull request #2382 from ethereum/vbuterin-patch-12

Reorganization of config params to put sync committee in one section

configs/mainnet/altair.yaml

@@ -10,20 +10,20 @@ MIN_SLASHING_PENALTY_QUOTIENT_ALTAIR: 64
 PROPORTIONAL_SLASHING_MULTIPLIER_ALTAIR: 2
 
 
-# Misc
+# Sync committee
 # ---------------------------------------------------------------
 # 2**9 (= 512)
 SYNC_COMMITTEE_SIZE: 512
-# 2**2 (= 4)
-INACTIVITY_SCORE_BIAS: 4
-
-
-# Time parameters
-# ---------------------------------------------------------------
 # 2**9 (= 512)
 EPOCHS_PER_SYNC_COMMITTEE_PERIOD: 512
 
 
+# Misc
+# ---------------------------------------------------------------
+# 2**2 (= 4)
+INACTIVITY_SCORE_BIAS: 4
+
+
 # Signature domains
 # ---------------------------------------------------------------
 DOMAIN_SYNC_COMMITTEE: 0x07000000

configs/minimal/altair.yaml

@@ -10,20 +10,20 @@ MIN_SLASHING_PENALTY_QUOTIENT_ALTAIR: 64
 PROPORTIONAL_SLASHING_MULTIPLIER_ALTAIR: 2
 
 
-# Misc
+# Sync committee
 # ---------------------------------------------------------------
 # [customized]
 SYNC_COMMITTEE_SIZE: 32
-# 2**2 (= 4)
-INACTIVITY_SCORE_BIAS: 4
-
-
-# Time parameters
-# ---------------------------------------------------------------
 # [customized]
 EPOCHS_PER_SYNC_COMMITTEE_PERIOD: 8
 
 
+# Misc
+# ---------------------------------------------------------------
+# 2**2 (= 4)
+INACTIVITY_SCORE_BIAS: 4
+
+
 # Signature domains
 # ---------------------------------------------------------------
 DOMAIN_SYNC_COMMITTEE: 0x07000000
@@ -45,6 +45,7 @@ ALTAIR_FORK_EPOCH: 18446744073709551615
 MIN_SYNC_COMMITTEE_PARTICIPANTS: 1
 
 
+
 # Validator
 # ---------------------------------------------------------------
 # 2**2 (= 4)

specs/altair/beacon-chain.md

@@ -14,8 +14,8 @@
   - [Misc](#misc)
 - [Configuration](#configuration)
   - [Updated penalty values](#updated-penalty-values)
+  - [Sync committee](#sync-committee)
   - [Misc](#misc-1)
-  - [Time parameters](#time-parameters)
   - [Domain types](#domain-types)
 - [Containers](#containers)
   - [Modified containers](#modified-containers)
@@ -114,19 +114,19 @@ This patch updates a few configuration values to move penalty parameters closer
 | `MIN_SLASHING_PENALTY_QUOTIENT_ALTAIR` | `uint64(2**6)` (= 64) |
 | `PROPORTIONAL_SLASHING_MULTIPLIER_ALTAIR` | `uint64(2)` |
 
+### Sync committee
+
+| Name | Value | Unit | Duration |
+| - | - | - | - |
+| `SYNC_COMMITTEE_SIZE` | `uint64(2**9)` (= 512) | Validators | |
+| `EPOCHS_PER_SYNC_COMMITTEE_PERIOD` | `uint64(2**9)` (= 512) | epochs | ~54 hours |
+
 ### Misc
 
 | Name | Value |
 | - | - |
-| `SYNC_COMMITTEE_SIZE` | `uint64(2**9)` (= 512) |
 | `INACTIVITY_SCORE_BIAS` | `uint64(4)` |
 
-### Time parameters
-
-| Name | Value | Unit | Duration |
-| - | - | :-: | :-: |
-| `EPOCHS_PER_SYNC_COMMITTEE_PERIOD` | `uint64(2**9)` (= 512) | epochs | ~54 hours |
-
 ### Domain types
 
 | Name | Value |

tests/core/pyspec/eth2spec/test/altair/block_processing/test_process_sync_committee.py

@@ -369,7 +369,7 @@ def test_valid_signature_future_committee(spec, state):
     yield from run_sync_committee_processing(spec, state, block)
 
 
-@with_all_phases_except([PHASE0])
+@with_altair_and_later
 @spec_state_test
 def test_sync_committee_is_only_computed_at_epoch_boundary(spec, state):
     """