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

log_sklearn_plot: Support custom name. #339

Merged

Conversation

daavoo
Copy link
Contributor

@daavoo daavoo commented Oct 24, 2022

Rename the existing name argument to kind. This argument is used to determine the type of plot. Add a new optional name argument to define the output file.

Closes #323

@daavoo daavoo self-assigned this Oct 24, 2022
@daavoo daavoo requested a review from dberenbaum October 24, 2022 16:07
@daavoo daavoo added the A: log_sklearn_plot Area: `live.log_sklearn_plot` label Oct 24, 2022
@daavoo daavoo force-pushed the 323-log_plot-custom-filename-andor-title-for-sklearn-plots branch from c552d15 to 1054b0a Compare October 24, 2022 16:09
@codecov-commenter
Copy link

codecov-commenter commented Oct 24, 2022

Codecov Report

❗ No coverage uploaded for pull request base (1.0@5a120c2). Click here to learn what that means.
Patch has no changes to coverable lines.

Additional details and impacted files
@@          Coverage Diff           @@
##             1.0     #339   +/-   ##
======================================
  Coverage       ?   94.96%           
======================================
  Files          ?       36           
  Lines          ?     1786           
  Branches       ?      165           
======================================
  Hits           ?     1696           
  Misses         ?       65           
  Partials       ?       25           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@dberenbaum
Copy link
Collaborator

One minor request: look for a .json extension and strip it so users don't end up with roc.json.json.

Rename existing `name` argument to `kind`. This argument is used to determine the type of plot.
Add new optional `name` argument to define output file.

Closes #323
@daavoo daavoo force-pushed the 323-log_plot-custom-filename-andor-title-for-sklearn-plots branch from 1054b0a to 1cfe551 Compare October 25, 2022 16:25
@daavoo
Copy link
Contributor Author

daavoo commented Oct 25, 2022

One minor request: look for a .json extension and strip it so users don't end up with roc.json.json.

should be done now

@daavoo daavoo merged commit 955b10a into 1.0 Oct 25, 2022
@daavoo daavoo deleted the 323-log_plot-custom-filename-andor-title-for-sklearn-plots branch October 25, 2022 17:17
@dberenbaum
Copy link
Collaborator

Nice, thanks! Is there a docs PR?

@daavoo
Copy link
Contributor Author

daavoo commented Oct 25, 2022

Nice, thanks! Is there a docs PR?

I am grouping all 1.0 in a single branch

@dberenbaum
Copy link
Collaborator

Can we have separate PRs and merge into a similar dvclive-1.0 branch in dvc.org so we don't have one massive review?

daavoo added a commit to iterative/dvc.org that referenced this pull request Oct 26, 2022
daavoo added a commit to iterative/dvc.org that referenced this pull request Oct 26, 2022
daavoo added a commit to iterative/dvc.org that referenced this pull request Oct 27, 2022
daavoo added a commit to iterative/dvc.org that referenced this pull request Oct 28, 2022
daavoo added a commit that referenced this pull request Oct 31, 2022
Rename existing `name` argument to `kind`. This argument is used to determine the type of plot.
Add new optional `name` argument to define output file.

Closes #323
daavoo added a commit that referenced this pull request Nov 4, 2022
Rename existing `name` argument to `kind`. This argument is used to determine the type of plot.
Add new optional `name` argument to define output file.

Closes #323
daavoo added a commit that referenced this pull request Nov 4, 2022
Rename existing `name` argument to `kind`. This argument is used to determine the type of plot.
Add new optional `name` argument to define output file.

Closes #323
daavoo added a commit to iterative/dvc.org that referenced this pull request Nov 5, 2022
* initial refactor

* Mention API Reference

* Fix link

* Add redirect

* Update content/docs/dvclive/how-to/checkpoints.md

Co-authored-by: Dave Berenbaum <[email protected]>

* Remove how-to folder

* Add outputs content

* Reorder sidebar

* Add tabs with popular frameworks

* Remove outdated `dvclive with DVC`

* clean

* Add params

* Clarify value without  DVC

* Update outputs to current status

* Update content/docs/dvclive/get-started.md

Co-authored-by: Dave Berenbaum <[email protected]>

* Reduce wording

* Add Python API tab

* More details

* Updates from review

* Capitalize pages

* Mention studio for sharing experiments

* New section for Studio

* Add sample output

* Move ml-frameworks to api-refeference

* Drop resume page

* Apply suggestions from code review

Co-authored-by: Dave Berenbaum <[email protected]>

* Restyled by prettier (#4050)

Co-authored-by: Restyled.io <[email protected]>

* Suggestions from code review

* caps

* Add sections

* Add example output

* Add admon

* Share experiment -> Share Results

* Update output structure.

Per iterative/dvclive#322

* Revisit output names

* path -> dir

* Add make_report call

* Rename log -> log_metric

* Update log_sklearn_plot. (#4074)

Per iterative/dvclive#339

* Apply suggestions from code review

Co-authored-by: Jorge Orpinel <[email protected]>

* Replace dvc with cli

* Remove tree

* Update output dir in frameworks

* grammarly

* Revisit frameworks naming.

* Remplaze `path` with `dir`

* Rename `outputs` to `how-it-works`

* Rename `Output Folder Structure` to `How it Works`

* Rename Scalars to Metrics

* Simplify `Live` Attributes description

* Renamed folder to directory

* Revisit `step` definition

* Fix tree

* Remove cat

* Replace vscode plots with data series

* Updates with new summary behavior and step property

* Fix links

* Consistent framework tabs

* Update content/docs/dvclive/api-reference/live/log_metric.md

* start: dvclive 1.0 updates to exp viz page

* Update content/docs/start/experiment-management/visualization.md

Co-authored-by: David de la Iglesia Castro <[email protected]>

* Revisit dvclive usage outside doc/dvclive

* Focus on context manager usage.

* hint about next_step

* Update content/docs/dvclive/api-reference/live/next_step.md

Co-authored-by: Dave Berenbaum <[email protected]>

* Link for `Live.metrics_file`

* Restyled by prettier (#4097)

Co-authored-by: Restyled.io <[email protected]>

* Updates per iterative/example-repos-dev#143

Co-authored-by: Dave Berenbaum <[email protected]>
Co-authored-by: restyled-io[bot] <32688539+restyled-io[bot]@users.noreply.github.com>
Co-authored-by: Restyled.io <[email protected]>
Co-authored-by: Jorge Orpinel <[email protected]>
Co-authored-by: rogermparent <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A: log_sklearn_plot Area: `live.log_sklearn_plot`
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants