Skip to content

fix: restore valid executor.source in custom.yaml#1

Merged
jack-sherman01 merged 1 commit into
mainfrom
copilot/fix-problem
Apr 23, 2026
Merged

fix: restore valid executor.source in custom.yaml#1
jack-sherman01 merged 1 commit into
mainfrom
copilot/fix-problem

Conversation

Copilot AI commented Apr 23, 2026

Copy link
Copy Markdown

A recent commit introduced invalid YAML for executor.source, breaking Hydra's ability to recognize the key — causing 24 test errors across the entire test suite.

Root cause

# Before (invalid — no space after colon, not a list)
executor:
  source:arxiv

# After
executor:
  source: [arxiv]

source:arxiv is parsed as the scalar string "source:arxiv" rather than a mapping entry, so executor.source never existed in the composed config. Any test override of executor.source=[arxiv] then failed with:

hydra.errors.ConfigCompositionException: Could not override 'executor.source'.
To append to your config use +executor.source=[arxiv]

@jack-sherman01 jack-sherman01 marked this pull request as ready for review April 23, 2026 23:33
@jack-sherman01 jack-sherman01 merged commit 5ffc33d into main Apr 23, 2026
1 check passed
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