Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix UpdateCommitteeCache slot #4074

Merged
merged 2 commits into from
Nov 21, 2019
Merged

Fix UpdateCommitteeCache slot #4074

merged 2 commits into from
Nov 21, 2019

Conversation

terencechain
Copy link
Member

The slot used to calculate CommitteeCountAtSlot was incorrect.

@codecov
Copy link

codecov bot commented Nov 21, 2019

Codecov Report

Merging #4074 into master will increase coverage by 3.38%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #4074      +/-   ##
==========================================
+ Coverage   21.47%   24.85%   +3.38%     
==========================================
  Files         179      164      -15     
  Lines       11612    10785     -827     
==========================================
+ Hits         2494     2681     +187     
+ Misses       8786     7639    -1147     
- Partials      332      465     +133

@terencechain terencechain merged commit 835418d into master Nov 21, 2019
@delete-merged-branch delete-merged-branch bot deleted the cache-committee-count branch November 21, 2019 05:29
cryptomental pushed a commit to cryptomental/prysm that referenced this pull request Feb 24, 2020
cryptomental pushed a commit to cryptomental/prysm that referenced this pull request Feb 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants