Skip to content

Add RunnerConfig typed configuration infrastructure (#5131)#5131

Closed
Cesar-Cardoso wants to merge 2 commits intofacebook:mainfrom
Cesar-Cardoso:export-D99150292
Closed

Add RunnerConfig typed configuration infrastructure (#5131)#5131
Cesar-Cardoso wants to merge 2 commits intofacebook:mainfrom
Cesar-Cardoso:export-D99150292

Conversation

@Cesar-Cardoso
Copy link
Copy Markdown
Contributor

@Cesar-Cardoso Cesar-Cardoso commented Apr 2, 2026

Summary:

Introduce RunnerConfig base class in runner.py with nested SearchSpaceUpdateArguments and RunnerUpdateArguments frozen dataclasses, and a config_type ClassVar on Runner so each runner subclass can declare its config type.

Reviewed By: andycylmeta

Differential Revision: D99150292

@meta-cla meta-cla bot added the CLA Signed Do not delete this pull request or issue due to inactivity. label Apr 2, 2026
@meta-codesync
Copy link
Copy Markdown

meta-codesync bot commented Apr 2, 2026

@Cesar-Cardoso has exported this pull request. If you are a Meta employee, you can view the originating Diff in D99150292.

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Apr 2, 2026

Codecov Report

❌ Patch coverage is 71.42857% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 96.38%. Comparing base (2993339) to head (14384e4).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
ax/utils/common/sentinel.py 0.00% 4 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5131      +/-   ##
==========================================
- Coverage   96.39%   96.38%   -0.01%     
==========================================
  Files         613      614       +1     
  Lines       68378    68391      +13     
==========================================
+ Hits        65913    65922       +9     
- Misses       2465     2469       +4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Cesar-Cardoso added a commit to Cesar-Cardoso/Ax that referenced this pull request Apr 3, 2026
Summary:

Introduce `RunnerConfig` base class in `runner.py` with nested `SearchSpaceUpdateArguments` and `RunnerUpdateArguments` frozen dataclasses, and a `config_type` ClassVar on `Runner` so each runner subclass can declare its config type.

Reviewed By: andycylmeta

Differential Revision: D99150292
Cesar-Cardoso added a commit to Cesar-Cardoso/Ax that referenced this pull request Apr 7, 2026
Summary:

Introduce `RunnerConfig` base class in `runner.py` with nested `SearchSpaceUpdateArguments` and `RunnerUpdateArguments` frozen dataclasses, and a `config_type` ClassVar on `Runner` so each runner subclass can declare its config type.

Reviewed By: andycylmeta

Differential Revision: D99150292
Summary:
Introduce `RunnerConfig` base class in `runner.py` with nested
`SearchSpaceUpdateArguments` and `RunnerUpdateArguments` frozen dataclasses, and a `config_type`
ClassVar on `Runner` so each runner subclass can declare its config type.

Differential Revision: D97988143
@meta-codesync meta-codesync bot changed the title Add RunnerConfig typed configuration infrastructure Add RunnerConfig typed configuration infrastructure (#5131) Apr 8, 2026
Cesar-Cardoso added a commit to Cesar-Cardoso/Ax that referenced this pull request Apr 8, 2026
Summary:

Introduce `RunnerConfig` base class in `runner.py` with nested `SearchSpaceUpdateArguments` and `RunnerUpdateArguments` frozen dataclasses, and a `config_type` ClassVar on `Runner` so each runner subclass can declare its config type.

Reviewed By: andycylmeta

Differential Revision: D99150292
Cesar-Cardoso added a commit to Cesar-Cardoso/Ax that referenced this pull request Apr 8, 2026
Summary:

Introduce `RunnerConfig` base class in `runner.py` with nested `SearchSpaceUpdateArguments` and `RunnerUpdateArguments` frozen dataclasses, and a `config_type` ClassVar on `Runner` so each runner subclass can declare its config type.

Reviewed By: andycylmeta

Differential Revision: D99150292
Cesar-Cardoso added a commit to Cesar-Cardoso/Ax that referenced this pull request Apr 8, 2026
Summary:
Pull Request resolved: facebook#5131

Introduce `RunnerConfig` base class in `runner.py` with nested `SearchSpaceUpdateArguments` and `RunnerUpdateArguments` frozen dataclasses, and a `config_type` ClassVar on `Runner` so each runner subclass can declare its config type.

Reviewed By: andycylmeta

Differential Revision: D99150292
Cesar-Cardoso added a commit to Cesar-Cardoso/Ax that referenced this pull request Apr 8, 2026
Summary:

Introduce `RunnerConfig` base class in `runner.py` with nested `SearchSpaceUpdateArguments` and `RunnerUpdateArguments` frozen dataclasses, and a `config_type` ClassVar on `Runner` so each runner subclass can declare its config type.

Reviewed By: andycylmeta

Differential Revision: D99150292
Cesar-Cardoso added a commit to Cesar-Cardoso/Ax that referenced this pull request Apr 8, 2026
Summary:

Introduce `RunnerConfig` base class in `runner.py` with nested `SearchSpaceUpdateArguments` and `RunnerUpdateArguments` frozen dataclasses, and a `config_type` ClassVar on `Runner` so each runner subclass can declare its config type.

Reviewed By: andycylmeta

Differential Revision: D99150292
Cesar-Cardoso added a commit to Cesar-Cardoso/Ax that referenced this pull request Apr 8, 2026
Summary:

Introduce `RunnerConfig` base class in `runner.py` with nested `SearchSpaceUpdateArguments` and `RunnerUpdateArguments` frozen dataclasses, and a `config_type` ClassVar on `Runner` so each runner subclass can declare its config type.

Reviewed By: andycylmeta

Differential Revision: D99150292
Summary:
Pull Request resolved: facebook#5131

Introduce `RunnerConfig` base class in `runner.py` with nested `SearchSpaceUpdateArguments` and `RunnerUpdateArguments` frozen dataclasses, and a `config_type` ClassVar on `Runner` so each runner subclass can declare its config type.

Reviewed By: andycylmeta

Differential Revision: D99150292
@meta-codesync
Copy link
Copy Markdown

meta-codesync bot commented Apr 8, 2026

This pull request has been merged in f8a813b.

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

Labels

CLA Signed Do not delete this pull request or issue due to inactivity. fb-exported Merged meta-exported

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants