Skip to content

fix: clarify full-sweep vs test-config usage and add exponential backoff#524

Merged
Oseltamivir merged 2 commits intomainfrom
claude/issue-520-20260122-0541
Jan 22, 2026
Merged

fix: clarify full-sweep vs test-config usage and add exponential backoff#524
Oseltamivir merged 2 commits intomainfrom
claude/issue-520-20260122-0541

Conversation

@Oseltamivir
Copy link
Copy Markdown
Collaborator

This PR fixes the CLI command misunderstanding where Claude was incorrectly using --min-conc, --max-conc, and --seq-len flags with test-config instead of full-sweep.

Changes:

  • Add explicit subcommand reference explaining flag support
  • Add example showing concurrency/seq-len flags with full-sweep
  • Clarify that full-sweep with filters is not an "unfiltered sweep"
  • Replace workflow monitoring instructions with exponential backoff

Fixes #520

Generated with Claude Code

…off for workflow monitoring

- Add explicit subcommand reference explaining that full-sweep accepts filter flags
  while test-config only accepts --config-files and --config-keys
- Add example showing --min-conc, --max-conc, and --seq-len usage with full-sweep
- Clarify that using full-sweep with filters is NOT the same as an unfiltered sweep
- Replace ambiguous workflow monitoring instructions with exponential backoff behavior
  (starting at 2 min sleep, doubling each iteration)

Fixes #520

Co-authored-by: Bryan Shan <Oseltamivir@users.noreply.github.com>
@Oseltamivir Oseltamivir merged commit 56206fb into main Jan 22, 2026
@Oseltamivir Oseltamivir deleted the claude/issue-520-20260122-0541 branch January 22, 2026 05:54
@Oseltamivir
Copy link
Copy Markdown
Collaborator Author

Test test test say hi @claude

@Klaud-Cold
Copy link
Copy Markdown
Collaborator

Klaud-Cold commented Jan 22, 2026

Claude finished @Oseltamivir's task in 18s —— View job


Responding to your message

  • Analyze context and request
  • Respond appropriately

Hi there, Bryan! 👋

I see you're testing me out. I'm here and ready to help! This PR has already been merged, but if you need any assistance with follow-up tasks, code reviews, or questions about the changes, feel free to ask.


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

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

[bug] claude keeps messing up generate_sweep_config.py command

2 participants