Skip to content

Commit

Permalink
[Hydra] Improve error message in parse_overrides
Browse files Browse the repository at this point in the history
Summary:

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:
  • Loading branch information
wu0607 committed Feb 5, 2025
1 parent e824695 commit b2a96f1
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions hydra/core/override_parser/overrides_parser.py
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,7 @@ def parse_override(self, s: str) -> Override:

def parse_overrides(self, overrides: List[str]) -> List[Override]:
ret: List[Override] = []
for override in overrides:
for idx, override in enumerate(overrides):
try:
parsed = self.parse_rule(override, "override")
except HydraException as e:
Expand All @@ -98,7 +98,7 @@ def parse_overrides(self, overrides: List[str]) -> List[Override]:
msg = f"Error parsing override '{override}'" f"\n{e}"
raise OverrideParseException(
override=override,
message=f"{msg}"
message=f"Error when parsing index: {idx}, string: {override} out of {overrides}. {msg}"
f"\nSee https://hydra.cc/docs/1.2/advanced/override_grammar/basic for details",
) from e.__cause__
assert isinstance(parsed, Override)
Expand Down

0 comments on commit b2a96f1

Please sign in to comment.