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

guide: Managing Experiments #2752

Closed
wants to merge 53 commits into from
Closed

guide: Managing Experiments #2752

wants to merge 53 commits into from

Conversation

jorgeorpinel
Copy link
Contributor

@jorgeorpinel jorgeorpinel commented Aug 25, 2021

Extracted from #2690 (review)

  • Un-nest from guide: Running Experiments #2690 when that PR is merged
  • Combine and summarize the motivation text in the section index
  • Finish Managing Exps guide (mainly about exp show/compare)
  • Remove deep details from the command reference that can be moved here instead

Maybe also include (if natural)

@jorgeorpinel jorgeorpinel mentioned this pull request Aug 25, 2021
4 tasks
@shcheklein shcheklein temporarily deployed to dvc-org-guide-exps-mgmt-muaser August 25, 2021 00:24 Inactive
Comment on lines +17 to +18
When you create an experiment, DVC generates a Git-like SHA-1 hash from its
contents. This is shown when you [queue experiments] with `--queue`:
Copy link
Contributor Author

@jorgeorpinel jorgeorpinel Aug 25, 2021

Choose a reason for hiding this comment

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

From #2690 (comment)

This is actually different from the hash generated after the experiments is committed, so I'm not sure it makes sense to mention it here.

@jorgeorpinel jorgeorpinel mentioned this pull request Aug 25, 2021
1 task
Queued experiment '6518f17' for future execution.
```

After running queued experiment, DVC uses the regular name mentioned earlier.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

From #2690 (comment)

The experiment name should show in dvc exp show even for queued experiments now.

@@ -1,5 +1,18 @@
# Experiment Management
Copy link
Contributor Author

Choose a reason for hiding this comment

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

From #2690 (comment)

I guess I'm the reason for this new section? I'm okay with leaving this in Running experiments depending on what others think. It does seem a bit disjointed as is. I think we need to at least link here from Running experiments so it's clear how to see and compare experiments.

Base automatically changed from iesahin/ug-exp-run-2675 to master August 25, 2021 16:13
@jorgeorpinel jorgeorpinel temporarily deployed to dvc-org-guide-exps-mgmt-muaser August 31, 2021 07:39 Inactive
@iesahin
Copy link
Contributor

iesahin commented Aug 31, 2021

For

Finish Managing Exps guide (mainly about exp show/compare)

I have #2737.

I think I can also create issues for other items as well.

@jorgeorpinel
Copy link
Contributor Author

jorgeorpinel commented Sep 1, 2021

I have #2737.

Yeah @iesahin I created this one based on #2690 (review) before noticing your Exp Compare ticket.

I think it could be included in this guide for now, as comparing them is probably a basic need to manage them. May not even need an explicit section but rather implied throughout this guide.

@jorgeorpinel
Copy link
Contributor Author

@iesahin should we close this and turn it into an issue?

@iesahin iesahin mentioned this pull request Sep 21, 2021
5 tasks
@iesahin
Copy link
Contributor

iesahin commented Sep 21, 2021

Done @jorgeorpinel

@iesahin iesahin closed this Sep 21, 2021
@jorgeorpinel
Copy link
Contributor Author

Thanks @iesahin !

@jorgeorpinel jorgeorpinel deleted the guide/exps-mgmt branch October 5, 2021 05:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants