Skip to content

Standardize Nevergrad sweeper search space config#3311

Merged
omry merged 1 commit into
mainfrom
pr3311
Jul 24, 2026
Merged

Standardize Nevergrad sweeper search space config#3311
omry merged 1 commit into
mainfrom
pr3311

Conversation

@omry

@omry omry commented Jul 24, 2026

Copy link
Copy Markdown
Collaborator

Add hydra.sweeper.params support using Hydra's override grammar while preserving Nevergrad scalar dictionaries and command-line precedence.

Deprecate hydra.sweeper.parametrization for Hydra 1.4, reject simultaneous use of both fields, and document the Hydra 1.5 removal.

Update examples, migration docs, release news, and regression coverage.

Refs #1890

@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jul 24, 2026
@omry
omry marked this pull request as ready for review July 24, 2026 17:23
Copilot AI review requested due to automatic review settings July 24, 2026 17:23

Copilot AI 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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 1ed7c80fc2

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Add hydra.sweeper.params support using Hydra's override grammar while preserving Nevergrad scalar dictionaries and command-line precedence.

Deprecate hydra.sweeper.parametrization for Hydra 1.4, reject simultaneous use of both fields, and document the Hydra 1.5 removal.

Update examples, migration docs, release news, and regression coverage.

Refs #1890

omry commented Jul 24, 2026

Copy link
Copy Markdown
Collaborator Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Bravo.

Reviewed commit: 633fe54cd8

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@omry
omry merged commit 8ab6601 into main Jul 24, 2026
25 checks passed
@omry
omry deleted the pr3311 branch July 24, 2026 18:59
@omry omry mentioned this pull request Jul 24, 2026
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants