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

start: reader instructions (in index pages) #4000

Merged
merged 14 commits into from
Dec 21, 2022
Merged

Conversation

jorgeorpinel
Copy link
Contributor

@jorgeorpinel jorgeorpinel commented Sep 27, 2022

Per #1943 (comment):

Change language at the very beginning, may be at the very top of every chapter. Make it super clear "how to use the get started"...
Also, don't forget that we have the whole project prepared - iterative/example-get-started that can be taken, any tag can be git checkout-ed and it's an ideal starting point.

In review app: ⏳


Jump to #4000 (comment)

@jorgeorpinel jorgeorpinel added type: enhancement Something is not clear, small updates, improvement suggestions A: docs Area: user documentation (gatsby-theme-iterative) C: start Content of /doc/start labels Sep 27, 2022
@shcheklein shcheklein temporarily deployed to dvc-org-start-instructi-e03fn9 September 27, 2022 03:20 Inactive
@jorgeorpinel jorgeorpinel changed the title start: add index for Exp Mgmt start: add reader instructions in index pages Sep 27, 2022
@shcheklein shcheklein temporarily deployed to dvc-org-start-instructi-e03fn9 September 27, 2022 03:23 Inactive
@github-actions
Copy link
Contributor

github-actions bot commented Sep 27, 2022

e6bfa08

Link Check Report

All 5 links passed!

CML watermark

@jorgeorpinel jorgeorpinel changed the title start: add reader instructions in index pages start: reader instructions (in index pages) Sep 27, 2022
@shcheklein shcheklein temporarily deployed to dvc-org-start-instructi-e03fn9 September 28, 2022 07:39 Inactive
Comment on lines 61 to 62
or import a model to another project.
- **[Data and model access]** goes over using data artifacts from outside of the
project and importing them from another DVC project. This can help to download
a specific version of an ML model to a deployment server or import a model to
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated some of these descriptions to match the cards at the bottom of the trail index pages.

@jorgeorpinel jorgeorpinel marked this pull request as ready for review September 28, 2022 07:41
@shcheklein shcheklein temporarily deployed to dvc-org-start-instructi-e03fn9 September 28, 2022 08:00 Inactive
Comment on lines 14 to 30
### ⚙️ Expand to prepare a project.

We'll be building an NLP project from scratch together. The end result is
published on [GitHub](https://github.com/iterative/example-get-started) -- feel
free to clone the repo.

Let's start with `git init`:
Imagine we want to building an ML project from scratch. Let's start by creating
a Git repository:

```dvc
$ mkdir example-get-started
$ cd example-get-started
$ git init
```

<admon type="info">

This name is actually used in our
[example-get-started](https://github.com/iterative/example-get-started) repo.

</admon>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I also realized this is only applicable to the Data Mgmt trail now, so I updated the note here (made it more general) and added hidden tips about each example repo into the trail index pages (context: #2496 (comment)).

Comment on lines 38 to 43
The steps and results of some of these chapters are captured in our
[example-get-started] repo on GitHub. Feel free to clone it and `git checkout`
any of it's [tags] to explore further.

[example-get-started]: https://github.com/iterative/example-get-started
[tags]: https://github.com/iterative/example-get-started/tags
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Should we maybe turn the tags that correspond to certain chapters and sections into Releases in the GH repo? I.e. from some https://github.com/iterative/example-get-started/tags to https://github.com/iterative/example-get-started/releases in this case. May be easier to mention releases than all the tags (not so easy to explore and match to the GS).

@shcheklein shcheklein temporarily deployed to dvc-org-start-instructi-e03fn9 September 28, 2022 08:11 Inactive
Copy link
Member

@shcheklein shcheklein left a comment

Choose a reason for hiding this comment

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

Sorry for stressing it one more time, but I think we are optimizing things that are not very important.

Re the PR - can we also simplify it? do an a common one paragraph description that is common for all trails? It feels sad that we need to explain in such length. Get started should be self-descriptive, simple, etc.

@jorgeorpinel jorgeorpinel self-assigned this Sep 29, 2022
@jorgeorpinel jorgeorpinel added the p2-nice-to-have Less of a priority at the moment. We don't usually deal with this immediately. label Sep 29, 2022
@shcheklein shcheklein temporarily deployed to dvc-org-start-instructi-ewmumm December 20, 2022 05:49 Inactive
@shcheklein shcheklein temporarily deployed to dvc-org-start-instructi-frkwss December 20, 2022 06:02 Inactive
@shcheklein shcheklein temporarily deployed to dvc-org-start-instructi-q84u7k December 20, 2022 06:15 Inactive
@shcheklein shcheklein temporarily deployed to dvc-org-start-instructi-gyqonb December 20, 2022 06:28 Inactive
@shcheklein shcheklein temporarily deployed to dvc-org-start-instructi-ouup2m December 20, 2022 06:52 Inactive
@shcheklein shcheklein temporarily deployed to dvc-org-start-instructi-rmskzb December 20, 2022 07:10 Inactive
@shcheklein shcheklein temporarily deployed to dvc-org-start-instructi-nsog0g December 20, 2022 07:24 Inactive
@shcheklein shcheklein temporarily deployed to dvc-org-start-instructi-miaaca December 20, 2022 07:38 Inactive
@shcheklein shcheklein temporarily deployed to dvc-org-start-instructi-kjfwmh December 20, 2022 07:52 Inactive
@shcheklein shcheklein temporarily deployed to dvc-org-start-instructi-vmoqdf December 20, 2022 08:05 Inactive
@shcheklein shcheklein temporarily deployed to dvc-org-start-instructi-zp1zwl December 20, 2022 08:19 Inactive
@shcheklein shcheklein temporarily deployed to dvc-org-start-instructi-wger1j December 20, 2022 08:33 Inactive
@shcheklein shcheklein temporarily deployed to dvc-org-start-instructi-dsqv19 December 20, 2022 08:46 Inactive
@shcheklein shcheklein had a problem deploying to dvc-org-start-instructi-ju4fni December 20, 2022 08:59 Failure
@yathomasi yathomasi temporarily deployed to dvc-org-start-instructi-ju4fni December 20, 2022 09:01 Inactive
@gatsby-cloud
Copy link

gatsby-cloud bot commented Dec 20, 2022

✅ dvc.org-2022-test-main deploy preview ready

@shcheklein shcheklein temporarily deployed to dvc-org-start-instructi-fdhlc3 December 20, 2022 09:14 Inactive
@yathomasi yathomasi closed this Dec 20, 2022
@yathomasi

This comment was marked as off-topic.

@yathomasi yathomasi reopened this Dec 20, 2022
@shcheklein shcheklein temporarily deployed to dvc-org-start-instructi-thuwox December 20, 2022 09:44 Inactive
@shcheklein
Copy link
Member

Btw, I think Pick a trail below to see an overview of all features from that perspective: is what confuses people. Most likely they think that each link below on the page is an independent trail. I would suggest a minor change - Data Management -> Data Management Trail on the page (same for experiments) + we can make it clear in the links - in the part 1 of the trail .... ...

@shcheklein shcheklein merged commit 32b56d9 into main Dec 21, 2022
@shcheklein shcheklein deleted the start/instructions branch December 21, 2022 17:48
jorgeorpinel added a commit that referenced this pull request Dec 22, 2022
@jorgeorpinel
Copy link
Contributor Author

Good observation @shcheklein . PTAL at #4197.

jorgeorpinel added a commit that referenced this pull request Dec 22, 2022
* start: reader instructions (2)

Small improvement per #4000 (comment)

* Update content/docs/start/index.md

* Restyled by prettier (#4198)

Co-authored-by: Restyled.io <commits@restyled.io>

Co-authored-by: restyled-io[bot] <32688539+restyled-io[bot]@users.noreply.github.com>
Co-authored-by: Restyled.io <commits@restyled.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A: docs Area: user documentation (gatsby-theme-iterative) C: start Content of /doc/start p2-nice-to-have Less of a priority at the moment. We don't usually deal with this immediately. type: enhancement Something is not clear, small updates, improvement suggestions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants