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

Update starter and create new production guide #2143

Merged
merged 272 commits into from
Jan 4, 2024

Conversation

htahir1
Copy link
Contributor

@htahir1 htahir1 commented Dec 13, 2023

Describe changes

This PR is a revamp of the doc structure. Primary things effected are:

The gitbook space is available here

Things to do:

  • Run through GPT4 suggestions for all guides
  • Connect to a deployed zenml service account stuff needs an advanced guide section
  • Link MCP and GCP advanced guides to starter guide versions
  • Fix advanced guide sections about MCP and DCP to be clearer and linked to the guide versions
  • Add a docs page about tags in the advanced guide
  • Take a look at the projects section and misc other docs pages that are outdated like FAQs

Pre-requisites

Please ensure you have done the following:

  • I have read the CONTRIBUTING.md document.
  • If my change requires a change to docs, I have updated the documentation accordingly.
  • If I have added an integration, I have updated the integrations table and the corresponding website section.
  • I have added tests to cover my changes.
  • I have based my new branch on develop and the open PR is targeting develop. If your branch wasn't based on develop read Contribution guide on rebasing branch to develop.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Other (add details above)

Copy link
Contributor

github-actions bot commented Jan 3, 2024

Images automagically compressed by Calibre's image-actions

Compression reduced images by 40.4%, saving 13.68 KB.

Filename Before After Improvement Visual comparison
docs/site/_assets/favicon.png 33.88 KB 20.19 KB -40.4% View diff

243 images did not require optimisation.

Update required: Update image-actions configuration to the latest version before 1/1/21. See README for instructions.

Copy link
Contributor

@strickvl strickvl left a comment

Choose a reason for hiding this comment

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

Nice work on catching all the remaining broken references :)

This looks good to me following a skim through all the changes. You can maybe ignore the scarf comments and we can fix in a separate PR. I'll also maybe set something up to detect / ensure that we have those within the CI process since it often seems to get forgotten.

- [Metadata](logging-metadata.md) which is associated with various components of ZenML.

This guide dives deeper into the management of this data.
Copy link
Contributor

Choose a reason for hiding this comment

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

needs scarf?

Copy link
Contributor

Choose a reason for hiding this comment

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

file needs scarf at end?

first_pipeline()
```

</details>
Copy link
Contributor

Choose a reason for hiding this comment

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

also needs scarf here?

@htahir1 htahir1 merged commit 109a8b5 into develop Jan 4, 2024
6 checks passed
@htahir1 htahir1 deleted the doc/update-starter-guide branch January 4, 2024 11:39
htahir1 added a commit that referenced this pull request Jan 4, 2024
* latest

* latest

* latest

* latest

* latest

* latest

* latest

* Concepts

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* Lean starter

* new TOC

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* GITBOOK-5: No subject

* Revert "latest"

This reverts commit c898778.

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* Optimised images with calibre/image-actions

* Revereted some errorneous things

* Revereted some errorneous things

* Latest

* Latest

* Latest

* Latest

* Latest

* Latest

* Latest

* Latest

* Latest

* Apply suggestions from code review

Co-authored-by: Alex Strick van Linschoten <strickvl@users.noreply.github.com>

* s3 access

* Ok updated

* Review changes

* Review changes

* Latest

* artifact metadata

* artifact metadata

* Optimised images with calibre/image-actions

* Latest

* Update

* Update

* New logging metadata

* New logging metadata

* More

* latest

* more

* Metadata added

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* GitBook: No commit message

* latest

* latest

* latest

* Optimised images with calibre/image-actions

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* latest

* Update docs/book/user-guide/starter-guide/track-ml-models.md

* Update docs/book/user-guide/starter-guide/track-ml-models.md

* Update docs/book/user-guide/starter-guide/track-ml-models.md

* Update docs/book/user-guide/production-guide/connect-code-repository.md

* Update docs/book/user-guide/production-guide/production-guide.md

* Update docs/book/user-guide/production-guide/configure-pipeline.md

* Update docs/book/user-guide/production-guide/scale-compute.md

* Update docs/book/user-guide/production-guide/cloud-stack.md

* Update docs/book/stacks-and-components/component-guide/data-validators/great-expectations.md

* Update docs/book/user-guide/starter-guide/fetching-pipelines.md

* Update docs/book/user-guide/starter-guide/fetching-pipelines.md

* Update docs/book/user-guide/starter-guide/manage-artifacts.md

* Update docs/book/user-guide/starter-guide/manage-artifacts.md

* latest

* latest

* latest

* latest

* latest

* Fixed validation

* Fixed expectation suite

* Fixed artifact vis

* deleted redundnat asset

* deleted redundnat asset

* Fixed broken links

* Took some input from baris into account

* Small changes

* Fixed materialization

* Fixed materialization

* Fixed materialization

* Update docs/book/getting-started/core-concepts.md

* Update docs/book/user-guide/production-guide/end-to-end.md

* Update docs/book/user-guide/production-guide/end-to-end.md

* Update docs/book/user-guide/production-guide/end-to-end.md

* Update docs/book/user-guide/production-guide/end-to-end.md

* Update docs/book/user-guide/production-guide/end-to-end.md

* Update docs/book/getting-started/core-concepts.md

* Update docs/book/user-guide/advanced-guide/data-management/model-management.md

* Update docs/book/user-guide/starter-guide/create-an-ml-pipeline.md

* Update docs/book/user-guide/production-guide/production-guide.md

* Update docs/book/user-guide/starter-guide/create-an-ml-pipeline.md

* Update docs/book/user-guide/starter-guide/create-an-ml-pipeline.md

* Update docs/book/user-guide/starter-guide/create-an-ml-pipeline.md

* Update docs/book/user-guide/starter-guide/create-an-ml-pipeline.md

* Update docs/book/user-guide/starter-guide/create-an-ml-pipeline.md

* Update docs/book/user-guide/starter-guide/cache-previous-executions.md

* Update docs/book/user-guide/starter-guide/fetching-pipelines.md

* Update docs/book/user-guide/starter-guide/fetching-pipelines.md

* Update docs/book/user-guide/starter-guide/fetching-pipelines.md

* Update docs/book/user-guide/starter-guide/fetching-pipelines.md

* Update docs/book/user-guide/starter-guide/fetching-pipelines.md

* projects reworked

* Update docs/book/user-guide/starter-guide/fetching-pipelines.md

* Update docs/book/user-guide/starter-guide/manage-artifacts.md

* Update docs/book/user-guide/starter-guide/fetching-pipelines.md

* Apply suggestions from code review

* Update docs/book/user-guide/starter-guide/fetching-pipelines.md

* Apply suggestions from code review

* learning section

* learning section

* adding slight more info

* Apply suggestions from code review

* Apply suggestions from code review

* Best of times

* Update docs/book/getting-started/core-concepts.md

Co-authored-by: Barış Can Durak <36421093+bcdurak@users.noreply.github.com>

* Update docs/book/user-guide/starter-guide/starter-guide.md

Co-authored-by: Barış Can Durak <36421093+bcdurak@users.noreply.github.com>

* Latest review comments

* Optimised images with calibre/image-actions

* Latest review comments

* Removed

* Renamed to log_model_metadata

* More stuff for materializers

* tags

* small data

* Slimmed down

* Added scarf

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Alex Strick van Linschoten <strickvl@users.noreply.github.com>
Co-authored-by: Barış Can Durak <36421093+bcdurak@users.noreply.github.com>
(cherry picked from commit 109a8b5)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation internal To filter out internal PRs and issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants