Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

plots diff: Show previous revisions for new plots #7913

Closed
dberenbaum opened this issue Jun 17, 2022 · 5 comments
Closed

plots diff: Show previous revisions for new plots #7913

dberenbaum opened this issue Jun 17, 2022 · 5 comments
Labels
A: plots Related to the plots p2-medium Medium priority, should be done, but less important

Comments

@dberenbaum
Copy link
Collaborator

If I add a new plot, dvc plots diff should be able to try to render previous revisions even if the plots were not configured in that revision.

For example, let's say I have plots data in train.csv for multiple revisions and then add a plot in dvc.yaml:

stages:
  train:
    cmd: train
    plots:
    - train.csv:
        x: epoch
        y: loss
        title: stage_plots

dvc plots diff shows only the workspace, but it should also show the HEAD revision.

@dberenbaum dberenbaum added p2-medium Medium priority, should be done, but less important A: plots Related to the plots labels Jun 17, 2022
@dberenbaum
Copy link
Collaborator Author

@pared Is a feature request or a bug? Any thoughts on priority?

@pared
Copy link
Contributor

pared commented Jun 20, 2022

It's a feature request, I wanted to include it in the current state of #7477 but it's already hanging for too long, let's do it right after it gets merged.

@dberenbaum
Copy link
Collaborator Author

Let's figure out priority later. If it hasn't been supported until now with few complaints, no need to rush when we have other priorities.

@pared
Copy link
Contributor

pared commented Jun 29, 2022

For anyone having a problem with this, a workaround, for now, would be to create new commit that contains desired plot definitions, before starting to experiment.

@mattseddon
Copy link
Member

Should work if the revision with the required plot is the first rev in the diff call.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A: plots Related to the plots p2-medium Medium priority, should be done, but less important
Projects
None yet
Development

No branches or pull requests

3 participants