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

Parameters: separate info among docs sections #3899

Merged
merged 105 commits into from
Sep 7, 2022
Merged

Parameters: separate info among docs sections #3899

merged 105 commits into from
Sep 7, 2022

Conversation

jorgeorpinel
Copy link
Contributor

@jorgeorpinel jorgeorpinel commented Aug 26, 2022 •
edited
Loading

On top of #3414

Exercise in guide vs. ref separation of concerns.

iterative and others added 30 commits April 5, 2022 18:29
initial plan and some content
f91f015
added content about stages
fd20226
@iesahin
title and restyle fixes
b279db7
@iesahin
added dag section
881f3af
@iesahin
depend to -> depend on
0e95212
@iesahin
added dependencies section
78f342d
@jorgeorpinel
Update content/docs/user-guide/pipelines/index.md
b2a949d
@jorgeorpinel
Update content/docs/user-guide/pipelines/index.md
9d5ee22
@restyled-io @restyled-commits
Restyled by prettier (#3532)
678753d 
Co-authored-by: Restyled.io <[email protected]>
@iesahin @jorgeorpinel
Update content/docs/user-guide/pipelines/index.md
7fe19e7 
Co-authored-by: Jorge Orpinel <[email protected]>
@iesahin
added pipelines to sidebar
e932c8d
@iesahin
updated the title
baf6ce1
@iesahin
fixed formatting
caf3291
@iesahin
updating for dvc.yaml first
2e98221
@iesahin
fed -> used
15ba67f
@iesahin
dvc.yaml-first
d749c2a
@iesahin
editing to tell dvc.yaml first
f01e750
@iesahin
minor fix
78cfebd
@iesahin
url dependency
9989902
@iesahin
dvc lock example
f51bfd5
@iesahin
section titles for deps
5edc29b
@iesahin
section titles for outputs
9649c0f
@iesahin
reproduction -> running
bb7c0a1
@iesahin
adding hyperparameters section
e26734e
@iesahin
added experiments section
424a196
@iesahin
adding url dependencies
e43bc98
@iesahin
added outputs section content
ee7703c
@iesahin
minor
602e327
@iesahin
added running pipelines content
1b08b42
@iesahin
moved outputs below running
1ff0479
@jorgeorpinel jorgeorpinel mentioned this pull request Aug 30, 2022
Merged
@shcheklein shcheklein temporarily deployed to dvc-org-params-refactor-my8p0m August 31, 2022 20:37 Inactive
@shcheklein shcheklein temporarily deployed to dvc-org-params-refactor-my8p0m September 2, 2022 05:56 Inactive
@shcheklein shcheklein temporarily deployed to dvc-org-params-refactor-my8p0m September 2, 2022 06:51 Inactive
@shcheklein shcheklein temporarily deployed to dvc-org-params-refactor-my8p0m September 2, 2022 06:58 Inactive
dberenbaum
dberenbaum approved these changes Sep 2, 2022
View reviewed changes
Copy link
Contributor

@dberenbaum dberenbaum left a comment

Choose a reason for hiding this comment

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

LGTM, thanks @jorgeorpinel!

@jorgeorpinel
Copy link
Contributor Author

OK! I just don't want to merge this before #3414 is approved... ⏳

@jorgeorpinel jorgeorpinel added the ⌛ status: wait-core-merge Waiting for related product PR merge/release label Sep 7, 2022
Base automatically changed from iesahin/ug-pipelines to main September 7, 2022 22:25
@jorgeorpinel
Copy link
Contributor Author

Oops forgot to merge this before #3414 (review) 😅 Fixing... ⏳

@shcheklein shcheklein temporarily deployed to dvc-org-params-refactor-my8p0m September 7, 2022 22:36 Inactive
@jorgeorpinel jorgeorpinel merged commit 90254db into main Sep 7, 2022
@jorgeorpinel jorgeorpinel deleted the params/refactor branch September 7, 2022 22:37
@jorgeorpinel jorgeorpinel removed the ⌛ status: wait-core-merge Waiting for related product PR merge/release label Nov 3, 2022
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

@dberenbaum dberenbaum dberenbaum approved these changes

Assignees
No one assigned
Labels
A: docs Area: user documentation (gatsby-theme-iterative) C: guide Content of /doc/user-guide C: ref Content of /doc/*-reference
Projects
None yet
Milestone
No milestone
Development

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