Handle DVC errors gracefully in plots

[shcheklein]

This depends on treeverse/dvc#7692

Figma specs

Motivation

1. Will be missing data for a specific experiment within one of the plots. This will be most apparent in the comparison table. We need a placeholder and some way to refresh the missing experiment's data.
2. Currently, if an experiment completely fails to load plots data there will be nothing to let the user know the experiment is missing. Add an indicator into the webview's top experiment selection section in order to let the user know that an experiment has something wrong.

TODO

Will be handled in relation to the: DVC errors that may occur during or after running experiment #1636

• Placeholder and refresh Handle DVC errors gracefully in plots #1649 (comment)
• Missing experiment indicator Handle DVC errors gracefully in plots #1649 (comment)

[maxagin]

Just so we could start the discussion

[mattseddon]

@maxagin it's more likely that we'll have all of the plots but we will be missing data for a specific experiment within one of the plots. This will be most apparent in the comparison table. This is what currently happens when we have no data for an experiment:

IMO we need a placeholder and some way to refresh the missing experiment's data.

Can I use the above placeholder in this situation?

What is the significance of the rhombus?

[mattseddon]

Something like this:

[mattseddon]

Worth noting that the extension UX guidelines state we should try and stick to the codicons in views wherever possible.

[maxagin]

the extension UX guidelines state

Yeah. I would say, we should be able to find solutions based on the VS Code fundamentals.

What is the significance of the rhombus?

Using it as a placeholder for the error mark (currently exclamation mark). WIP

[maxagin]

@mattseddon I think we may want to use buttons where it is possible. take a look and let me know.

[mattseddon]

Currently, if an experiment completely fails to load plots data there will be nothing to let the user know the experiment is missing.

IMO we will also need to add an indicator into the webview's top experiment selection section in order to let the user know that an experiment has something wrong. I.e the top section of the proposed design:

WDYT?