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.

Sign up for GitHub

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

ref: Add plots.auto_open in dvc.config #3101

Merged
merged 6 commits into from
Jan 13, 2022
Merged

ref: Add plots.auto_open in dvc.config #3101

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
6601925
ref: Add `plots.auto_open` in `dvc.config`
daavoo Dec 16, 2021
aecaf8b
ref: copy edits around plots --open
jorgeorpinel Dec 17, 2021
143a8df
Restyled by prettier (#3109)
restyled-io[bot] Dec 17, 2021
91c251b
Update content/docs/command-reference/config.md
jorgeorpinel Dec 17, 2021
687559a
Merge branch 'master' into plots-auto-open
daavoo Jan 13, 2022
736a9a1
Mention auto_open in pcp
daavoo Jan 13, 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
3 changes: 3 additions & 0 deletions content/docs/command-reference/config.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -281,6 +281,9 @@ be overidden explicitly through CLI arguments or through responses in prompts
[custom HTML template](/doc/command-reference/plots#html-templates) for
`dvc plots`. Accepts a path relative to the `.dvc/` folder.

- `plots.auto_open` - if `true`, DVC will automatically open the HTML
generated by `dvc plots` commands in a browser. `false` by default
Copy link
Contributor

Choose a reason for hiding this comment

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

Q: Will this apply to dvc exp show --html (or whatever the final command for PCPs is) ? If so maybe add a check box to update this in #3105 at some point).

This comment was marked as resolved.

Sign in to view

Copy link
Contributor

@jorgeorpinel jorgeorpinel Jan 4, 2022 •
edited
Loading

Choose a reason for hiding this comment

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

So now that #3098 was merged first maybe we should address #3098 (review) here?


## Example: Add an S3 remote, and set it as default

> 💡 Before adding an S3 remote, be sure to
Expand Down
3 changes: 2 additions & 1 deletion content/docs/command-reference/plots/diff.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -90,7 +90,8 @@ all the current plots, without comparisons.
- `--show-vega` - produce a [Vega-Lite](https://vega.github.io/vega-lite/) spec
file instead of HTML. See `dvc plots` for more info.

- `--open` - opens the generated plot in the browser automatically.
- `--open` - open the HTML generated in a browser automatically. You can
enable `dvc config plots.auto_open` to make this the default behavior.

- `--no-header` - lets DVC know that CSV or TSV `--targets` do not have a
header. A 0-based numeric index can be used to identify each column instead of
Expand Down
3 changes: 2 additions & 1 deletion content/docs/command-reference/plots/show.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,8 @@ please see `dvc plots`.
- `--show-vega` - produce a [Vega-Lite](https://vega.github.io/vega-lite/) spec
file instead of HTML. See `dvc plots` for more info.

- `--open` - opens the generated plot in the browser automatically.
- `--open` - open the HTML generated in a browser automatically. You can
enable `dvc config plots.auto_open` to make this the default behavior.

- `--no-header` - lets DVC know that CSV or TSV `targets` do not have a header.
A 0-based numeric index can be used to identify each column instead of names.
Expand Down