Skip to content

Remove hyperband#26

Merged
Wenyueh merged 2 commits intomainfrom
remove-hyperband
Mar 20, 2026
Merged

Remove hyperband#26
Wenyueh merged 2 commits intomainfrom
remove-hyperband

Conversation

@QianJaneXie
Copy link
Copy Markdown
Collaborator

Hyperband (successive halving) is typically designed for training curves, where partial evaluations provide biased but informative signals about final performance. In our setting, however, partial evaluations are better viewed as unbiased but noisy estimates, so a confidence-aware approach like successive elimination may be more appropriate.

Therefore I decide to remove it.

@QianJaneXie QianJaneXie requested a review from Wenyueh March 20, 2026 01:30
@Wenyueh Wenyueh merged commit d9f6c9e into main Mar 20, 2026
Wenyueh pushed a commit that referenced this pull request Mar 20, 2026
Resolve merge conflicts:
- README.md: keep refactor branch version (single-package structure)
- docs/model-selection-design.md: accept deletion (replaced by mkdocs)
- src/agentopt/model_selection/hyperband.py: accept deletion (PR #26)

Clean up remaining hyperband references in README and docs that
were introduced by the refactor branch after PR #26 landed on main.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@QianJaneXie QianJaneXie deleted the remove-hyperband branch March 20, 2026 21:10
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