Fix "Get Started" walkthrough typos and outdated mentions #4118
Conversation
| @@ -30,7 +30,7 @@ | |||
| [python api]: https://dvc.org/doc/api-reference | |||
| [free course]: https://learn.iterative.ai/ | |||
| [contribute]: https://dvc.org/doc/contributing/core | |||
| [example project]: https://github.com/iterative/example-get-started | |||
| [example project]: https://github.com/iterative/example-dvc-experiments | |||
There was a problem hiding this comment.
I believe we want to direct users to example-dvc-experiments instead of example-get-started
There was a problem hiding this comment.
I am not sure if this is correct or if it should now be https://github.com/iterative/example-get-started-experiments. Probably best to ask @shcheklein / @dberenbaum.
There was a problem hiding this comment.
Fine to make this change and follow up
There was a problem hiding this comment.
and there are multiple instances of this repo being linked to.
There was a problem hiding this comment.
yes, it should be https://github.com/iterative/example-get-started-experiments
There was a problem hiding this comment.
Links have been updated!
|
|
||
| <p align="center"> | ||
| <img src="images/plots-view-icon.png" | ||
| alt="Plots View Icon" /> | ||
| </p> | ||
|
|
||
| The **Plots Dashboard** can be configured and accessed from the _Plots_ and | ||
| _Experiments_ side panels in the [**DVC View**](command:views.dvc-views). |
There was a problem hiding this comment.
Plots is no longer a link in the Experiments side panel.
| @@ -36,6 +37,6 @@ If DVC and the extension are initialized, side bar will expose multiple views: | |||
| columns. | |||
| - `PLOTS`. Used to fine-tune the plots dashboard, e.g. by toggling plots | |||
| visibility. | |||
| - `FILTER BY`. Used to view, add, or remove metrics and parameters filters. | |||
| - `FILTER BY`. Used to view, add, or remove metrics/parameters filters. These | |||
| will hide specific experiments in the table. | |||
There was a problem hiding this comment.
Could be wrong but I think the mention about hiding experiments should be in the filter by section. I don't think sorting hides experiments 🤔
|
Code Climate has analyzed commit ba87f37 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 95.2% (0.1% change). View more on Code Climate. |
Was reviewing the walkthrough and spotted some typos plus a couple outdated links/sentences