params/metrics/exp diff: consistent output

[jorgeorpinel]

Extracted from treeverse/dvc.org#2200 (comment)

• metrics diff currently shows old, new, change
• params diff currently shows old, new
• exp diff currently shows new, change

Should they all be Old, New, Change? (Possibly with some flag to skip Old or Change)

[jorgeorpinel]

cc @dberenbaum

[dberenbaum]

Thanks, @jorgeorpinel! Let's refer this back to #5392.

Should they all be Old, New, Change?

👍

[jorgeorpinel]

What do you mean @dberenbaum? 🙂

[jorgeorpinel]

BTW @pmrowla do you know if we've considered merging dvc params/metrics/exp diff into a single command (e.g. dvc compare)? Or maybe as flags of dvc diff.

[pmrowla]

@jorgeorpinel the idea has come up before as something that we could consider doing in the future, but not seriously considered yet

[dberenbaum]

What do you mean @dberenbaum? 🙂

I just wanted to mention it so we have all of these UI issues linked in that one issue. No other action needed!

[daavoo]

Was about to comment this in #5392 (haven't seen any other related issues). I think that the terms Old / New used in diff sometimes could be confusing, given that users can pass refs in any order.

For example they could run dvc metrics diff {newer commit hash} {older commit hash} and the table will show the values of the newer commit under the column Old.

[dberenbaum]

cc @skshetry

I'd prefer having the ref names instead of Old and New.