vscode-dvc icon indicating copy to clipboard operation
vscode-dvc copied to clipboard

Published 20 hours ago • verified publisher icon iterative

Update the DVC: Get Started / Plots Dashboard

Open maxagin opened this issue 3 years ago • 4 comments

Select (click the circle beside the experiment name) one or more up to 7 experiments in the table view to visualize in the Plots Dashboard. screenshot Use DVC: Show Plots from the command palette to open it or open it using the table's row context menu.

@shcheklein : We need to wait for the https://github.com/iterative/vscode-dvc/pull/2553 to land

maxagin avatar Oct 12 '22 14:10 maxagin

Thanks @maxagin . The image should be also part of this PR please.

Also, please see in the file and wrap text at 80 symbols.

shcheklein avatar Oct 12 '22 22:10 shcheklein

The screenshot was updated. Thanks

maxagin avatar Oct 13 '22 18:10 maxagin

Sorry forgot to wrap the text. Will create an update

maxagin avatar Oct 13 '22 18:10 maxagin

The problem with this approach is that people don't see now the plots dashboard itself when they open the page (they would need to scroll). Should we change the order of things?

shcheklein avatar Oct 20 '22 00:10 shcheklein

The problem with this approach is that people don't see now the plots dashboard itself when they open the page (they would need to scroll). Should we change the order of things?

Maybe we can rename the section titles e.g.

  • Experiments View ~~Experiments Table~~
  • Plots View ~~Plots Dashboard~~
Screen Shot 2022-10-20 at 6 21 50 PM

WDYT?

Otherwise, we need to change the textual content I think.

that people don't see now the plots dashboard

Before it also was not at the top!

Screen Shot 2022-10-20 at 6 27 10 PM

maxagin avatar Oct 20 '22 22:10 maxagin

@maxagin re renaming - let's do one thing at at a time please ;)

Was it at least somewhat visible? My point is that even if it was bad we are making it worse. Should we reorder things a bit?

shcheklein avatar Oct 21 '22 00:10 shcheklein

A possible solution is to add overview images for some sections e.g. sidebar, EXPs, Plots. I like this solution also because we educate users on what views we actually have and how possibly you should organize the panels WDYT @shcheklein ?

Screen Shot 2022-10-20 at 9 45 19 PM Screen Shot 2022-10-20 at 9 45 28 PM Screen Shot 2022-10-20 at 9 45 38 PM

maxagin avatar Oct 21 '22 01:10 maxagin

@maxagin feels a bit too much and more images to maintain.

shcheklein avatar Oct 21 '22 03:10 shcheklein

@maxagin so, how about changing the order and finally merging this and the second PR (we can release everything today / tomorrow then)?

shcheklein avatar Oct 26 '22 00:10 shcheklein

more images to maintain.

  1. Three more images won't make any difference if all the images will have to be changed.
  2. Personally I think this is a very clever idea - to show right away the main extension views, thus we can educate the user about what he can expect and hopefully reduce the number of leaving users.

changing the order

I am not sure this is good, as the content of this section is clearly organized to explain what you need to do step by step:

  1. What to do to show EXPs in Plots view
  2. Types of plots
  3. Explaining the types

@shcheklein, what is your take on this?

maxagin avatar Oct 26 '22 16:10 maxagin

@maxagin any updates on this? :)

shcheklein avatar Nov 02 '22 05:11 shcheklein

Completely went out of my radar! I will deal with it ASAP @shcheklein

maxagin avatar Nov 02 '22 14:11 maxagin

@maxagin could you update / or upload the video please?

shcheklein avatar Nov 03 '22 17:11 shcheklein

@shcheklein the description video is updated.

maxagin avatar Nov 03 '22 19:11 maxagin

Code Climate has analyzed commit 2b2cc5cf and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (85% is the threshold).

This pull request will bring the total coverage in the repository to 96.8% (0.0% change).

View more on Code Climate.

codeclimate[bot] avatar Nov 03 '22 22:11 codeclimate[bot]