Skip to content

Use unicast instead of echo broadcast for 1 party exchange - #282

Merged
APshenkin merged 1 commit into
masterfrom
tast/optimise-broadcast
Aug 10, 2026
Merged

APshenkin merged 1 commit into
masterfrom
tast/optimise-broadcast

Conversation

@APshenkin

Copy link
Copy Markdown
Contributor

Description

In case when we run echoBroadcast, but there is only two parties, we can use unicast instead, as equivocation between receivers is impossible with one receiver

Pull Request Checklist

Scope

  • Summary and description are provided.
  • Changes are focused and scoped to one purpose.

Testing (select all that apply)

  • Unit tests
  • Property tests
  • Test vectors tests
  • Benchmarks
  • Not run (explain why)

Documentation / Spec (if relevant)

  • Updated/added docs or comments
  • Spec updated or linked

Notes

  • Additional context is provided (if needed)

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Claude Code Review

Claude Code Review is paused for this repository. To reconnect it, an admin of this repository's GitHub organization (or the account owner, for personal repositories) who can also manage your Claude organization's Code Review settings needs to re-link GitHub in Code Review settings. This is a one-time step.

Tip: disable this comment in your organization's Code Review settings.

@mkramarczyk mkramarczyk left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@APshenkin
APshenkin merged commit 0b614dd into master Aug 10, 2026
1 of 4 checks passed
@APshenkin
APshenkin deleted the tast/optimise-broadcast branch August 10, 2026 10:02
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