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

README rewrite (+ CONTRIB updates) #1816

Merged
merged 67 commits into from
Jun 14, 2022
Merged
Changes from 6 commits
Commits
Show all changes
67 commits
Select commit Hold shift + click to select a range
9a3b54e
readme: make intro = value prop +
jorgeorpinel Jun 2, 2022
9258493
readme: spave between VS and Code
jorgeorpinel Jun 2, 2022
7a089a1
readme: reorg existing content per
jorgeorpinel Jun 2, 2022
9d21977
readme: add features list
jorgeorpinel Jun 2, 2022
249d88e
readme: extend/fix/link other parts of the structure
jorgeorpinel Jun 2, 2022
35aa202
readme: fix DVC icon
jorgeorpinel Jun 2, 2022
6dc30f6
readme: value prop edited from
jorgeorpinel Jun 7, 2022
2f9ccd1
Merge branch 'main' into readme
jorgeorpinel Jun 7, 2022
9da66a8
Update README.md
jorgeorpinel Jun 8, 2022
d9b7130
Update README.md
jorgeorpinel Jun 8, 2022
7635990
Merge branch 'readme' of github.com:iterative/vscode-dvc into readme
jorgeorpinel Jun 8, 2022
aabb42f
readme: rewrap
jorgeorpinel Jun 8, 2022
af21897
readme: add list of high-level features
jorgeorpinel Jun 8, 2022
ed3ccd3
readme: update features and add competitive advantages
jorgeorpinel Jun 8, 2022
abbb7c0
readme: mention that it's beta
jorgeorpinel Jun 8, 2022
96e3365
readme: small improvements to lists
jorgeorpinel Jun 8, 2022
a035ea5
readme: better Quick start
jorgeorpinel Jun 8, 2022
632d8e6
readme: populate Config section
jorgeorpinel Jun 8, 2022
447245c
readme: move Useful commands before Config
jorgeorpinel Jun 8, 2022
92a37b1
config: update settings descriptions (readme and package)
jorgeorpinel Jun 8, 2022
7ce9fdf
contrib: copy edits
jorgeorpinel Jun 8, 2022
e4cde08
readme: fix vs-code commands lnk
jorgeorpinel Jun 8, 2022
6ee4d02
contrib: kill link to wiki (outdated)
jorgeorpinel Jun 8, 2022
49b7991
Merge branch 'main' into readme
jorgeorpinel Jun 8, 2022
9e6218a
readme: lint (BROKEN)
jorgeorpinel Jun 8, 2022
392c60e
Update README.md
jorgeorpinel Jun 8, 2022
14e80ec
Update README.md
jorgeorpinel Jun 8, 2022
69fe8ad
Merge branch 'main' into readme
jorgeorpinel Jun 13, 2022
3aaea33
fix linter-broken line
jorgeorpinel Jun 13, 2022
3aca21d
contrib: simpler dev env steps
jorgeorpinel Jun 13, 2022
2fdf8f5
readme: better intro wording
jorgeorpinel Jun 13, 2022
820983e
readme: plots view -> dashboard :/
jorgeorpinel Jun 13, 2022
2375289
readme: support should go to Discord first
jorgeorpinel Jun 13, 2022
9804ca6
readme: exp bookkeeping -> tracking
jorgeorpinel Jun 13, 2022
1f3165d
readme: clarify reproducibility feature
jorgeorpinel Jun 13, 2022
736a01b
readme: more specific data mgmt use
jorgeorpinel Jun 14, 2022
0b6a2e8
readme: improve UI components list
jorgeorpinel Jun 14, 2022
f7363fb
readme: fix img hack
jorgeorpinel Jun 14, 2022
b486ac6
add badges to README
mattseddon Jun 10, 2022
de63e81
drop some badges
mattseddon Jun 11, 2022
c15964d
preview banner
mattseddon Jun 11, 2022
ed3f4ff
update link to point to main
mattseddon Jun 13, 2022
45876a6
preview gif
mattseddon Jun 14, 2022
935b36b
Merge pull request #1873 from iterative/add-badges
jorgeorpinel Jun 14, 2022
4ca6219
Merge pull request #1875 from iterative/preview-banner
jorgeorpinel Jun 14, 2022
b3d63ff
readme: impro feats
jorgeorpinel Jun 14, 2022
52acd1d
Merge branch 'main' into readme
jorgeorpinel Jun 14, 2022
b3c21a1
contrib: move note about DVC projects
jorgeorpinel Jun 14, 2022
3e96de3
revert to changelog from main and add prettier rule
mattseddon Jun 14, 2022
adb1330
Merge pull request #1883 from iterative/fix-changelog
jorgeorpinel Jun 14, 2022
91f37db
Update README.md
jorgeorpinel Jun 14, 2022
2f3d458
readme: move animation after value prop paragraph
jorgeorpinel Jun 14, 2022
f55dba4
Apply suggestions from code review
jorgeorpinel Jun 14, 2022
91138fb
Update README.md
jorgeorpinel Jun 14, 2022
0cd8a3d
Restyled by whitespace
restyled-commits Jun 14, 2022
005e5fc
Merge pull request #1887 from iterative/restyled/readme
jorgeorpinel Jun 14, 2022
f227d87
Restyled by prettier-markdown
restyled-commits Jun 14, 2022
1aaee80
Update README.md
jorgeorpinel Jun 14, 2022
fb9619a
Merge pull request #1888 from iterative/restyled/readme
jorgeorpinel Jun 14, 2022
864a362
contrib: valid md new line >:(
jorgeorpinel Jun 14, 2022
16d6dda
readme: move images to extension/images and
jorgeorpinel Jun 14, 2022
e8c681e
Merge branch 'main' into readme
jorgeorpinel Jun 14, 2022
d099f31
readme: move images back to extension/docs/ foler
jorgeorpinel Jun 14, 2022
75e883f
swap banner to png
mattseddon Jun 14, 2022
4aadbd3
replace outdated walkthrough screenshot
mattseddon Jun 14, 2022
38bb119
update style of inline dvc png so it does not look bonkers
mattseddon Jun 14, 2022
b0fa7ab
(fixup) use absolute links for the marketplace and in product
mattseddon Jun 14, 2022
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
91 changes: 68 additions & 23 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,20 +4,57 @@
<!--- [![Continuous Integration](https://github.com/iterative/vscode-dvc/actions/workflows/continuous-integration.yml/badge.svg)](https://github.com/iterative/vscode-dvc/actions/workflows/continuous-integration.yml) --->
<!--- [![Cross-Platform Test](https://github.com/iterative/vscode-dvc/actions/workflows/cross-platform-test.yml/badge.svg)](https://github.com/iterative/vscode-dvc/actions/workflows/cross-platform-test.yml) --->
<!-- [![DVC CLI Output Test](https://github.com/iterative/vscode-dvc/actions/workflows/dvc-cli-output-test.yml/badge.svg)](https://github.com/iterative/vscode-dvc/actions/workflows/dvc-cli-output-test.yml) -->

[![Maintainability](https://api.codeclimate.com/v1/badges/fb243c31ea059c0038b2/maintainability)](https://codeclimate.com/repos/608b5886f52398018b00264c/maintainability)
[![Test Coverage](https://api.codeclimate.com/v1/badges/fb243c31ea059c0038b2/test_coverage)](https://codeclimate.com/repos/608b5886f52398018b00264c/test_coverage)

A [Visual Studio Code](https://code.visualstudio.com/)
[extension](https://marketplace.visualstudio.com/VSCode) that aims to allow
users of all technical backgrounds to effectively use [DVC](https://dvc.org/).

## Current State

We want early adopter feedback! Please let us know what you like and don't like
about the extension. Feel free to reach out in this repository's
[issues](https://github.com/iterative/vscode-dvc/issues) or via any of the other
existing [support channels](https://dvc.org/support).
This [extension] integrates [DVC features] right into the popular [VS Code IDE],
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ideally, the readme should start with value-proposition (not too DVC specific) with following competitive advantages (can be specific). Examples (Why TPI/MLEM? is competitive advantages):

Copy link
Member

@dmpetrov dmpetrov Jun 5, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ivan just added value-proposition & competitive advantages in the beginning of this doc:
https://docs.google.com/document/d/10THN0WwvPaDOAnmWd37xMOSWftJrUSIPlA2m0iHGDNI/edit#

PS: It is the 1st iteration (please suggest changes) but it gives an idea how to describe the product.

Copy link
Contributor Author

@jorgeorpinel jorgeorpinel Jun 7, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ideally, the readme should start with value-proposition (not too DVC specific)

I thought the value prop was that it brings DVC into VSC 😅

But I think you're saying that the value is that of the parts of DVC included in the extension + being able to use them visually in the IDE... ⏳

Copy link
Contributor Author

@jorgeorpinel jorgeorpinel Jun 8, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

with following competitive advantages (can be specific). Examples (Why TPI/MLEM? ...

We discussed that format and I thought we agreed that those 2 lists (features vs. advantages) tend to be redundant (probably OK for TPI but I'm still not sure about MLEM).

Anyway, I'll go with that for now so we can move on.

allowing users of all backgrounds to use DVC.

> **Note**: We want early adopter feedback!
> Please let us know what you like and don't like about the extension. Feel free
> to open [issues in this repo] or reach out via [DVC support] channels.

[extension]: https://marketplace.visualstudio.com/VSCode
[dvc features]: https://dvc.org/doc/start
[vs code ide]: https://code.visualstudio.com/
[issues in this repo]: https://github.com/iterative/vscode-dvc/issues
[dvc support]: https://dvc.org/support

## What it does (extension features)

The DVC Extension augments and adds several [Views], adds special [Editors].
Furthermore it exposes these and other DVC functions in the [Command Palette].
Specifically:

- It provides _Editors_ to manage [Experiments][exp-view] and display
[Plots][plots-view] as _Tabs_ in the IDE.
- Adds a [DVC Tracked] panel to the _Explorer_ view. This shows a tree of the
data that DVC is tracking, with real-time file/dir [states] and options to
[synchronize] them (from/to remote storage).
- Adds a [DVC panel] to the _Source Control_ view to list the [workspace
status]. You can [restore] or reset project versions (based on the current Git
`HEAD` commit) as well as [manage new and existing data] from here.
jorgeorpinel marked this conversation as resolved.
Show resolved Hide resolved
- Adds a brand-new [DVC View] (<img
src="https://user-images.githubusercontent.com/1477535/171570901-9012413c-f0bb-41d9-9a45-2653b4e3f1fe.png"
alt="DVC logo" style="height: 1em;"/> icon in the Activity Bar) with panels to
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image looks like this styling doesn't work for the in-app content

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed this by changing from style to height.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks

visualize and manage [DVC Experiments].
- Registers several [commands] in the _Command Palette_.
- Includes a [DVC channel] for the _Output_ panel (useful for
[debugging](#debugging)).

[views]: https://code.visualstudio.com/docs/getstarted/userinterface#_views
[editors]:
https://code.visualstudio.com/docs/getstarted/userinterface#_open-editors
[command palette]:
https://code.visualstudio.com/docs/getstarted/userinterface#_command-palette

[exp-view]: extension/resources/walkthrough/experiments-table.md
[plots-view]: extension/resources/walkthrough/plots.md
[dvc tracked]: extension/resources/walkthrough/tracked-explorer.md
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All of these relative links need to get changed to the github hosted ones or they won't work in product either

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Used absolute paths for all of these links because viewers could be:

  1. In Github.
  2. In the marketplace.
  3. In VS Code.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch

[dvc panel]: extension/resources/walkthrough/source-control-management.md
[dvc view]: extension/resources/walkthrough/view-container.md
[command palette]: extension/resources/walkthrough/command-palette.md
[dvc output]: #
jorgeorpinel marked this conversation as resolved.
Show resolved Hide resolved

## Quick Start

Expand All @@ -27,6 +64,16 @@ existing [support channels](https://dvc.org/support).
[Install the DVC extension for Visual Studio Code](https://code.visualstudio.com/docs/editor/extension-gallery).
- **Step 3.** See Walkthrough.

### Learn more about DVC (Data Version Control)

View [more resources](extension/resources/walkthrough/dvc-learn-more.md).

## Configuration

<!-- TODO -->

See the .vscode/ dir.
jorgeorpinel marked this conversation as resolved.
Show resolved Hide resolved

## Useful commands

Open the Command Palette (`F1` or ⇧⌃P on Windows/Linux or ⇧⌘P on macOS) and type
Expand All @@ -40,16 +87,18 @@ in one of the following commands:
| `DVC: Show Experiments` | Show an interactive version of DVC's `exp show` [command](https://dvc.org/doc/command-reference/exp/show). |
| `DVC: Show Plots` | Show an interactive version of DVC's `plots diff` [command](https://dvc.org/doc/command-reference/plots/diff). |

## Features
## Debugging
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It might be helpful to mention https://github.com/iterative/vscode-dvc/issues again here and try to describe which issues should go where. Or should we just give one channel for issues and leave it to our internal teams to move issues as needed?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm using Discord first and then link to GH issues in both places now. I think people will mostly know what each channel is for, probably preferring the chat in most cases, esp. for beginner questions.


### The DVC Extension

Please see the DVC [channel] in the IDE's _Output_ panel to see the underlying
DVC commands being run, as wel as their error output. Feel free to share this
jorgeorpinel marked this conversation as resolved.
Show resolved Hide resolved
with us via [DVC support].

- [Command Palette](extension/resources/walkthrough/command-palette.md)
- [Source Control Management](extension/resources/walkthrough/source-control-management.md)
- [Tracked Resources](extension/resources/walkthrough/tracked-explorer.md)
- [DVC View Container](extension/resources/walkthrough/view-container.md)
- [Experiments Table](extension/resources/walkthrough/experiments-table.md)
- [Plots](extension/resources/walkthrough/plots.md)
[channel]:
https://code.visualstudio.com/api/extension-capabilities/common-capabilities#output-channel

### Debugging
### Your DVC project

Due to the way DVC pipelines run scripts of any language from the command line,
users must debug pipeline scripts (e.g. `train.py`) standalone in whatever way
Expand All @@ -61,10 +110,6 @@ pipelines, and most scripts are capable of running outside of DVC.
See development docs and contributing guidelines in
[CONTRIBUTING.md](CONTRIBUTING.md)

## Explore

View [more resources](extension/resources/walkthrough/dvc-learn-more.md).
mattseddon marked this conversation as resolved.
Show resolved Hide resolved

## Data and telemetry

The DVC Extension for Visual Studio Code collects usage data and sends it to
Expand Down