Skip to content

Commit

Permalink
Merge pull request #4060 from jimmygchen/fix-custody-sampling-size
Browse files Browse the repository at this point in the history
Align PeerDAS `sampling_size` comparison units
  • Loading branch information
jtraglia authored Dec 18, 2024
2 parents 5567434 + 8e0d0d4 commit e6bddd9
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion specs/fulu/das-core.md
Original file line number Diff line number Diff line change
Expand Up @@ -237,7 +237,7 @@ The particular columns/groups that a node custodies are selected pseudo-randomly

## Custody sampling

At each slot, a node advertising `custody_group_count` downloads a minimum of `sampling_size = max(SAMPLES_PER_SLOT, custody_group_count)` total custody groups. The corresponding set of columns is selected by `groups = get_custody_groups(node_id, sampling_size)` and `compute_columns_for_custody_group(group) for group in groups`, so that in particular the subset of columns to custody is consistent with the output of `get_custody_groups(node_id, custody_group_count)`. Sampling is considered successful if the node manages to retrieve all selected columns.
At each slot, a node advertising `custody_group_count` downloads a minimum of `sampling_size = max(SAMPLES_PER_SLOT, custody_group_count * columns_per_group)` total columns, where `columns_per_group = NUMBER_OF_COLUMNS // NUMBER_OF_CUSTODY_GROUPS`. The corresponding set of columns is selected by `groups = get_custody_groups(node_id, sampling_size)` and `compute_columns_for_custody_group(group) for group in groups`, so that in particular the subset of columns to custody is consistent with the output of `get_custody_groups(node_id, custody_group_count)`. Sampling is considered successful if the node manages to retrieve all selected columns.

## Extended data

Expand Down

0 comments on commit e6bddd9

Please sign in to comment.