status-im / nimbus-eth2

Nim implementation of the Ethereum Beacon Chain
https://nimbus.guide
Other
543 stars 233 forks source link

fix DB size estimates for light client data #6373

Closed etan-status closed 4 months ago

etan-status commented 4 months ago

Addresses two inaccuracies in light client data size documentation:

  1. SyncCommittee pubkeys serialize are 48 bytes not 64 bytes
  2. Some of the estimates used 1000 vs 1024 bytes/KB, aligned to 1024
github-actions[bot] commented 4 months ago

Unit Test Results

         9 files  ±0    1 328 suites  ±0   32m 58s :stopwatch: - 2m 39s   4 998 tests ±0    4 650 :heavy_check_mark: ±0  348 :zzz: ±0  0 :x: ±0  20 850 runs  ±0  20 446 :heavy_check_mark: ±0  404 :zzz: ±0  0 :x: ±0 

Results for commit 2bc0f024. ± Comparison against base commit 22b5bc62.