PR feedback

This commit is contained in:
Danny Ryan 2019-10-24 21:49:07 +09:00
parent 69730cc267
commit 70c2063cfa
No known key found for this signature in database
GPG Key ID: 2765A792E42CE07A
5 changed files with 2 additions and 14 deletions

View File

@ -116,8 +116,6 @@ MAX_ATTESTATIONS: 128
MAX_DEPOSITS: 16 MAX_DEPOSITS: 16
# 2**4 (= 16) # 2**4 (= 16)
MAX_VOLUNTARY_EXITS: 16 MAX_VOLUNTARY_EXITS: 16
# Originally 2**4 (= 16), disabled for now.
MAX_TRANSFERS: 0
# Signature domains # Signature domains
@ -127,7 +125,6 @@ DOMAIN_BEACON_ATTESTER: 0x01000000
DOMAIN_RANDAO: 0x02000000 DOMAIN_RANDAO: 0x02000000
DOMAIN_DEPOSIT: 0x03000000 DOMAIN_DEPOSIT: 0x03000000
DOMAIN_VOLUNTARY_EXIT: 0x04000000 DOMAIN_VOLUNTARY_EXIT: 0x04000000
DOMAIN_TRANSFER: 0x05000000
DOMAIN_CUSTODY_BIT_CHALLENGE: 0x06000000 DOMAIN_CUSTODY_BIT_CHALLENGE: 0x06000000
DOMAIN_SHARD_PROPOSER: 0x80000000 DOMAIN_SHARD_PROPOSER: 0x80000000
DOMAIN_SHARD_ATTESTER: 0x81000000 DOMAIN_SHARD_ATTESTER: 0x81000000

View File

@ -118,8 +118,6 @@ MAX_ATTESTATIONS: 128
MAX_DEPOSITS: 16 MAX_DEPOSITS: 16
# 2**4 (= 16) # 2**4 (= 16)
MAX_VOLUNTARY_EXITS: 16 MAX_VOLUNTARY_EXITS: 16
# Originally 2**4 (= 16), disabled for now.
MAX_TRANSFERS: 0
# Signature domains # Signature domains
@ -129,7 +127,6 @@ DOMAIN_BEACON_ATTESTER: 0x01000000
DOMAIN_RANDAO: 0x02000000 DOMAIN_RANDAO: 0x02000000
DOMAIN_DEPOSIT: 0x03000000 DOMAIN_DEPOSIT: 0x03000000
DOMAIN_VOLUNTARY_EXIT: 0x04000000 DOMAIN_VOLUNTARY_EXIT: 0x04000000
DOMAIN_TRANSFER: 0x05000000
DOMAIN_CUSTODY_BIT_CHALLENGE: 0x06000000 DOMAIN_CUSTODY_BIT_CHALLENGE: 0x06000000
DOMAIN_SHARD_PROPOSER: 0x80000000 DOMAIN_SHARD_PROPOSER: 0x80000000
DOMAIN_SHARD_ATTESTER: 0x81000000 DOMAIN_SHARD_ATTESTER: 0x81000000

View File

@ -285,14 +285,14 @@ class Checkpoint(Container):
```python ```python
class Validator(Container): class Validator(Container):
pubkey: BLSPubkey pubkey: BLSPubkey
withdrawal_credentials: Hash # Commitment to pubkey for withdrawals and transfers withdrawal_credentials: Hash # Commitment to pubkey for withdrawals
effective_balance: Gwei # Balance at stake effective_balance: Gwei # Balance at stake
slashed: boolean slashed: boolean
# Status epochs # Status epochs
activation_eligibility_epoch: Epoch # When criteria for activation were met activation_eligibility_epoch: Epoch # When criteria for activation were met
activation_epoch: Epoch activation_epoch: Epoch
exit_epoch: Epoch exit_epoch: Epoch
withdrawable_epoch: Epoch # When validator can withdraw or transfer funds withdrawable_epoch: Epoch # When validator can withdraw funds
``` ```
#### `AttestationData` #### `AttestationData`

View File

@ -46,7 +46,6 @@ Operations:
| `block_header` | `Block` | **`block`** | `process_block_header(state, block)` | | `block_header` | `Block` | **`block`** | `process_block_header(state, block)` |
| `deposit` | `Deposit` | `deposit` | `process_deposit(state, deposit)` | | `deposit` | `Deposit` | `deposit` | `process_deposit(state, deposit)` |
| `proposer_slashing` | `ProposerSlashing` | `proposer_slashing` | `process_proposer_slashing(state, proposer_slashing)` | | `proposer_slashing` | `ProposerSlashing` | `proposer_slashing` | `process_proposer_slashing(state, proposer_slashing)` |
| `transfer` | `Transfer` | `transfer` | `process_transfer(state, transfer)` |
| `voluntary_exit` | `VoluntaryExit` | `voluntary_exit` | `process_voluntary_exit(state, voluntary_exit)` | | `voluntary_exit` | `VoluntaryExit` | `voluntary_exit` | `process_voluntary_exit(state, voluntary_exit)` |
Note that `block_header` is not strictly an operation (and is a full `Block`), but processed in the same manner, and hence included here. Note that `block_header` is not strictly an operation (and is a full `Block`), but processed in the same manner, and hence included here.

View File

@ -6,7 +6,6 @@ from eth2spec.test.phase_0.block_processing import (
test_process_block_header, test_process_block_header,
test_process_deposit, test_process_deposit,
test_process_proposer_slashing, test_process_proposer_slashing,
test_process_transfer,
test_process_voluntary_exit, test_process_voluntary_exit,
) )
@ -48,10 +47,6 @@ if __name__ == "__main__":
create_provider('deposit', test_process_deposit, 'mainnet'), create_provider('deposit', test_process_deposit, 'mainnet'),
create_provider('proposer_slashing', test_process_proposer_slashing, 'minimal'), create_provider('proposer_slashing', test_process_proposer_slashing, 'minimal'),
create_provider('proposer_slashing', test_process_proposer_slashing, 'mainnet'), create_provider('proposer_slashing', test_process_proposer_slashing, 'mainnet'),
create_provider('transfer', test_process_transfer, 'minimal'),
# Disabled, due to the high amount of different transfer tests, this produces a shocking size of tests.
# Unnecessarily, as transfer are disabled currently, so not a priority.
# create_provider('transfer', test_process_transfer, 'mainnet'),
create_provider('voluntary_exit', test_process_voluntary_exit, 'minimal'), create_provider('voluntary_exit', test_process_voluntary_exit, 'minimal'),
create_provider('voluntary_exit', test_process_voluntary_exit, 'mainnet'), create_provider('voluntary_exit', test_process_voluntary_exit, 'mainnet'),
]) ])