Refactor Scheduler schema definitions to make it easier to add new ones.

[bpkroth]

Pull Request

Title

Refactor Scheduler schema definitions to make it easier to add new ones.

---

Description

Simple refactor of the Scheduler schemas to allow making new ones as a copy/edit of the synchscheduler-subschema.json.

---

Type of Change

• 🔄 Refactor

---

Testing

• Exisiting CI checks for good and bad scheduler config files still pass.
• Adding additional tests to actually load those configs with mlos_bench next.

---

Additional Notes (optional)

• Splitting some new testing infra out from WIP: Prepare Experiment.load to handle async out of order trial completion #973.
• Adding MockScheduler next, then ParallelScheduler in WIP: Parallel Trial Scheduler #971.

---

[Copilot]

Pull Request Overview

This PR refactors scheduler schema definitions by extracting common base config properties, introducing a dedicated subschema for the SyncScheduler, and updating the top-level scheduler schema to compose these subschemas.

• Extracted shared scheduler config properties into base-scheduler-subschema.json
• Added sync-scheduler-subschema.json for SyncScheduler-specific config
• Refactored scheduler-schema.json to use allOf/oneOf composition instead of inline if/then

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File	Description
mlos_bench/.../sync-scheduler-subschema.json	New subschema defining SyncScheduler class & config
mlos_bench/.../scheduler-schema.json	Top-level schema now composes base and sync subschemas
mlos_bench/.../base-scheduler-subschema.json	New base subschema with common scheduler config defs

Comments suppressed due to low confidence (1)

mlos_bench/mlos_bench/config/schemas/schedulers/base-scheduler-subschema.json:1

• The new base scheduler subschema isn’t yet covered by any validation tests. Consider adding unit tests that load and validate this schema to catch regressions early.

{