-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Open
Labels
A: plotsRelated to the plotsRelated to the plotsp3-nice-to-haveIt should be done this or next sprintIt should be done this or next sprint
Description
When dvc.yaml is either corrupted or fails validation, dvc plots show/diff output shows DVC failed to load some plots for following revisions: workspace. There are a couple issues with this:
- It makes it seem like the issue is with one or more of the plots, when it's really about the structure of
dvc.yamland may be completely unrelated to plots. - It makes it seem like some plots from that revision might still be shown, which isn't true.
- It doesn't give any indication of how to fix it. Other commands like
dvc reprogive output about what failed like:
'./dvc.yaml' is invalid.
While scanning a simple key, in line 24, column 1
24 dfdf
Could not find expected ':', in line 25, column 1
25
For dvc plots show, can we show something similar?
For dvc plots diff, this might not make sense by default since there could be many revisions. However, it still seems like we could improve the message. For example, it could say something like Failed to read dvc.yaml for the following revisions (add -v to see errors): workspace.
Metadata
Metadata
Assignees
Labels
A: plotsRelated to the plotsRelated to the plotsp3-nice-to-haveIt should be done this or next sprintIt should be done this or next sprint