Skip to content

Commit

Permalink
Improve doc on using unbounded integers
Browse files Browse the repository at this point in the history
Co-authored-by: Aditya Asgaonkar <adiasg@users.noreply.github.com>
  • Loading branch information
saltiniroberto and adiasg authored Jul 25, 2023
1 parent 0b8eb0b commit 499e4f6
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion fork_choice/confirmation-rule.md
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ There are two algorithms in the document:
- [**Confirmation Rule**](#confirmation-rule): Given a block and confirmation safety parameters, outputs whether the block is confirmed.
- [**Confirmation Score**](#confirmation-score): Given a block, outputs the confirmation score for the block, i.e., the maximum possible confirmation safety parameters to deem the block confirmed.

*Note*: These algorithms use unbounded integer arithmetic in some places. The rest of `consensus-specs` uses `uint64` arithmetic exclusively to ensure that results fit into length-limited fields - a property crucial for consensus objects (such as the `BeaconBlockBody`). This document describes a local confirmation rule that does not require storing anything in length-limited fields. Using unbounded integer arithmetic here prevents possible overflowing issues.
*Note*: These algorithms use unbounded integer arithmetic in some places. The rest of `consensus-specs` uses `uint64` arithmetic exclusively to ensure that results fit into length-limited fields - a property crucial for consensus objects (such as the `BeaconBlockBody`). This document describes a local confirmation rule that does not require storing anything in length-limited fields. Using unbounded integer arithmetic here prevents possible overflowing issues for the spec tests generated using this Python specification (or when executing these specifications directly).

## Confirmation Rule

Expand Down

0 comments on commit 499e4f6

Please sign in to comment.