Skip to content

[Bug] Wire RoundRobin index into scheduling for optional cross-run ro… - #1960

Open
ShashwatXD wants to merge 1 commit into
kyegomez:masterfrom
ShashwatXD:fix/round-robin-rotation-1864
Open

[Bug] Wire RoundRobin index into scheduling for optional cross-run ro…#1960
ShashwatXD wants to merge 1 commit into
kyegomez:masterfrom
ShashwatXD:fix/round-robin-rotation-1864

Conversation

@ShashwatXD

Copy link
Copy Markdown

Changes

  • add persist_rotation (default False) and drive scheduling from (index + turn) % N
  • advance index by one after each run() when the flag is set
  • document the flag in the class docstring

Tests

  • default off: two consecutive run() calls produce the same visit order
  • persist_rotation=True: run_batch over N tasks on N agents gives each agent the opening turn once
  • within-run turn counts unchanged

Closes #1864

@ShashwatXD
ShashwatXD requested a review from kyegomez as a code owner August 22, 2026 13:52
@github-actions

Copy link
Copy Markdown

Hello there, thank you for opening an PR ! 🙏🏻 The team was notified and they will get back to you asap.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug][RoundRobinSwarm] self.index is written but never read — run_batch gives agents[0] the opening turn on every task

1 participant