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
RCHAIN-4103: Set fault tolerance as 1 for VALID old blocks [v0.9.23LTS] #2932
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@nzpr please create ticket to create complete solution and also the same PR for dev branch.
1f.pure[F], | ||
SafetyOracle[F] | ||
.normalizedFaultTolerance(dag, block.blockHash) | ||
) // TODO: Warn about parent block finalization |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I cannot imagine what this might mean
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't know, either.
normalizedFaultTolerance <- SafetyOracle[F] | ||
.normalizedFaultTolerance(dag, block.blockHash) // TODO: Warn about parent block finalization | ||
// TODO this is temporary solution to not calculate fault tolerance for old blocks which is costly | ||
oldBlock = dag.latestBlockNumber.map(_ - block.body.state.blockNumber).map(_ > 100) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually I don't like the hardcoded value here. Butnvm, let's fix it first
bors r+ |
2932: Set fault tolearnce as 1 for old blocks r=nzpr a=nzpr Don't calculate fault tolerance for old blocks, temporary fix. ### Please make sure that this PR: - [x] is at most 200 lines of code (excluding tests), - [x] meets [RChain development coding standards](https://rchain.atlassian.net/wiki/spaces/DOC/pages/28082177/Coding+Standards), - [x] includes tests for all added features, - [x] has a reviewer assigned, - [x] has [all commits signed](https://rchain.atlassian.net/wiki/spaces/DOC/pages/498630673/How+to+sign+commits+to+rchain+rchain). ### [Bors](https://bors.tech/) cheat-sheet: - `bors r+` runs integration tests and merges the PR (if it's approved), - `bors try` runs integration tests for the PR, - `bors delegate+` enables non-maintainer PR authors to run the above. Co-authored-by: nutzipper <1746367+nzpr@users.noreply.github.com>
Build failed: |
bors r+ |
2932: Set fault tolearnce as 1 for old blocks r=nzpr a=nzpr Don't calculate fault tolerance for old blocks, temporary fix. ### Please make sure that this PR: - [x] is at most 200 lines of code (excluding tests), - [x] meets [RChain development coding standards](https://rchain.atlassian.net/wiki/spaces/DOC/pages/28082177/Coding+Standards), - [x] includes tests for all added features, - [x] has a reviewer assigned, - [x] has [all commits signed](https://rchain.atlassian.net/wiki/spaces/DOC/pages/498630673/How+to+sign+commits+to+rchain+rchain). ### [Bors](https://bors.tech/) cheat-sheet: - `bors r+` runs integration tests and merges the PR (if it's approved), - `bors try` runs integration tests for the PR, - `bors delegate+` enables non-maintainer PR authors to run the above. Co-authored-by: nutzipper <1746367+nzpr@users.noreply.github.com>
Timed out. |
Closed as superseded by #2947 |
Overview
Don't calculate fault tolerance for old blocks, temporary fix.
JIRA ticket:
https://rchain.atlassian.net/browse/RCHAIN-4103
Please make sure that this PR:
Bors cheat-sheet:
bors r+
runs integration tests and merges the PR (if it's approved),bors try
runs integration tests for the PR,bors delegate+
enables non-maintainer PR authors to run the above.