Hydra fix passed all tests but failed on a nested YAML override case
A developer fixed a Hydra bug where the wrong config override won when multiple overrides targeted the same group. The fix passed all existing tests, but maintainer omry showed with a short YAML example that it produced the wrong result. The problem was that overrides were registered before Hydra walked into child configs, which ignored the depth-first ordering rule. The existing tests did not cover this nested ordering case, so nothing flagged it. The maintainer suggested registering overrides during Hydra's reverse traversal, which keeps the first-registered override as the correct winner.
This is an AI-generated summary. ShortSingh links to the original source for the complete article.

Discussion (0)
Log in to join the discussion and vote.
Log in