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

Jump to bottom

guide: copy edit Comparing Exps #3008

Merged
merged 1 commit into from
Nov 8, 2021
Merged

guide: copy edit Comparing Exps #3008

merged 1 commit into from
Nov 8, 2021

Conversation

jorgeorpinel
Copy link
Contributor

@jorgeorpinel jorgeorpinel commented Nov 7, 2021 •
edited
Loading

Rel. #2995

  • Needs detailed copy editing. E.g. missing "the", disjointed paragraphs, unnecessary scrolling blocks, etc.
  • --sort/order-by/no-timestamp sections may be too detailed. Hide or just mention + link to ref.
  • Merge JSON and CSV sections somehow
  • Shorten JSON/CSV output sample blocks and remove corresponding schema descriptions (comprehensive examples are enough).
  • Clarify exp diff sections to separate cases: workspace vs. HEAD, rev A vs rev B, etc.

@shcheklein shcheklein temporarily deployed to dvc-org-guide-comp-exps-stmipn November 7, 2021 01:58 Inactive
jorgeorpinel
jorgeorpinel commented Nov 7, 2021
View reviewed changes
content/docs/user-guide/experiment-management/comparing-experiments.md
Comment on lines -32 to +35
## List experiments in another Git remote
## List experiments saved remotely

As we discuss in [Sharing Experiments], you can use `dvc exp push` to upload
experiments to Git remotes. `dvc exp list` can be used to list the experiments
in a Git remote.
Experiments can be [shared] (with `dvc exp push`) from another location. To
review experiments uploaded to a remote <abbr>repository</abbr> (which you may
not have locally), provide a Git remote name to `dvc exp list`.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Note: These changes are to match #2908

@jorgeorpinel jorgeorpinel self-assigned this Nov 7, 2021
iesahin
iesahin approved these changes Nov 8, 2021
View reviewed changes
Copy link
Contributor

@iesahin iesahin left a comment

Choose a reason for hiding this comment

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

Thank you Jorge. 🙏🏼

@iesahin iesahin merged commit 7c729ee into master Nov 8, 2021
@iesahin iesahin deleted the guide/comp-exps branch November 8, 2021 16:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@shcheklein shcheklein shcheklein approved these changes

@iesahin iesahin iesahin approved these changes

Assignees

@jorgeorpinel jorgeorpinel

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jorgeorpinel @iesahin @shcheklein