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: support for flexible x-axis values #7754

Closed
dberenbaum opened this issue May 16, 2022 · 0 comments · Fixed by #8421
Closed

plots: support for flexible x-axis values #7754

dberenbaum opened this issue May 16, 2022 · 0 comments · Fixed by #8421
Labels
A: plots Related to the plots p2-medium Medium priority, should be done, but less important

Comments

@dberenbaum
Copy link
Contributor

In #7086 (comment), there are plots where it can be useful to specify the x source file separately from y. For example, if I keep the actual independent variable saved with my original data, but save the predictions for each model in a separate file, I might need something like:

plots:
  confusion:
    x: 
      dir/actual.csv: actual
    y:
      dir/preds.csv: predicted
    template: confusion

We might also need to change x between files. For example, maybe I want to plot ROC and precision-recall on the same plot to see how they correlate:

plots:
  roc_vs_prc:
    x:
      precision_recall.json: recall
      roc.json: fpr
    y:
      precision_recall.json: precision
      roc.json: tpr

Another use case for this would be if I named x differently between files, like:

plots:
  train_vs_val:
    x:
      train_loss.csv: epoch
      val_loss.csv: step
    y: 
      train_loss.csv: loss
      val_loss.csv: loss
@dberenbaum dberenbaum added p2-medium Medium priority, should be done, but less important A: plots Related to the plots labels May 16, 2022
pared added a commit to pared/dvc that referenced this issue Oct 17, 2022
pared added a commit to pared/dvc that referenced this issue Oct 18, 2022
pared added a commit to pared/dvc that referenced this issue Oct 19, 2022
pared added a commit to pared/dvc that referenced this issue Oct 20, 2022
pared added a commit to pared/dvc that referenced this issue Oct 20, 2022
pared added a commit to pared/dvc that referenced this issue Oct 20, 2022
pared added a commit to pared/dvc that referenced this issue Oct 24, 2022
pared added a commit to pared/dvc that referenced this issue Oct 24, 2022
pared added a commit that referenced this issue Nov 15, 2022
daavoo pushed a commit to pared/dvc that referenced this issue Nov 18, 2022
daavoo pushed a commit that referenced this issue Nov 18, 2022
* plots: vega: converter: support multi series data split and props inference

Fixes: #7754, #8399
Related: iterative/studio#4369

* plots: converter: make converter infer x and y labels if not provided

* plots: vega: converter: roll back to datapoints format

* plots: vega: converter: refactor

* plots: infer labels on

* plots: get_plot_id: fix naming

* plots: vega: converter: notes regarding converting
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

Successfully merging a pull request may close this issue.

1 participant