-
Notifications
You must be signed in to change notification settings - Fork 435
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
Added success message #603
Merged
AlexejPenner
merged 2 commits into
develop
from
feature/added-success-message-to-flavor-register
May 18, 2022
Merged
Added success message #603
AlexejPenner
merged 2 commits into
develop
from
feature/added-success-message-to-flavor-register
May 18, 2022
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
github-actions
bot
added
internal
To filter out internal PRs and issues
enhancement
New feature or request
labels
May 17, 2022
htahir1
approved these changes
May 18, 2022
strickvl
approved these changes
May 18, 2022
htahir1
added a commit
that referenced
this pull request
May 18, 2022
* Getting there * midway best practices * Added more explain stuff * Edited best practices * checkpoint * Checkpoint for quickstart * Added notebook * Added dash visualizer jupyter * Refactored imports * Added runner script * Quickstart now follows best practices * Quickstart formatted * Updated zenml go * Latest quickstart * Adapted zenml go to new quickstart * Added event for extending ZenML * Linting fixes * zenml go is now optout too * Made an exception for pipeline yamls * Added config * Apply suggestions from code review Co-authored-by: Alex Strick van Linschoten <strickvl@users.noreply.github.com> * Deleting * preserving user metadata * Fromatted latest * Formatting and linting * Formatting and linting * Typo * One function down * Update stakc comp done * Deregister stack * Create user * Delete user * Get user * Added all events * Added project * Added facade for all base methods * Deleted flavor event delted * Wrapped track event to make sure deduplication doesnt happen * Wrapped track event to make sure deduplication doesnt happen * Registered stack refactored * Flavor event is now properly done * Update stack done * Added import and export stack * Added import and export stack * Analytics events done * Copy paste error * Fixed mypy errors * Toc updated * Restructurung * Restructure * GitBook: [#2] No subject * Fix/update `zenml go` (#606) * Fix broken feature store example (#602) * allow yaml files * include yaml config * Added success message (#603) * Added success message * Reformatted * remove extra colon * add extra spacing * update zenml go behaviour Co-authored-by: Alexej Penner <thealexejpenner@gmail.com> * Analytics turned off in copy_active_configuration * Docstring nits * Formatting * Added todo * Revamp Quickstart (#604) * WIP: rework/reformat quickstart. * Finished quickstart.ipynb (untested). * Improved docs/comments of quickstart.ipynb. * Fix bugs in quickstart.ipynb. Everything except dash integration works. * Update quickstart README and setup.py. * Bug fix: allow Facets visualizer magic in Colab. * Added screenshots for all visualizations to quickstart.ipynb. * Added pipeline visualization to quickstart.ipynb. * Added stack visualization to quickstart.ipynb. * Wrote new quickstart run.py script / code modules. * Fix grammar / spelling of quickstart. * Update quickstart README: declare notebook execution as recommended and move run.py-specific set up instructions out of prerequisites. * Adjust quickstart to reviewer feedback. * Added magic flag Co-authored-by: Hamza Tahir <htahir111@gmail.com> * Changed best practices to misc and added resources section * Finished landing page of resoruces * Half finished best practices * Added eventbrite * Getting there * FAQs added basically * Fixed link; * GitBook: [#3] No subject * Apply suggestions from code review Co-authored-by: Alex Strick van Linschoten <strickvl@users.noreply.github.com> Co-authored-by: Alex Strick van Linschoten <strickvl@users.noreply.github.com> Co-authored-by: Alexej Penner <thealexejpenner@gmail.com> Co-authored-by: Felix Altenberger <felix@zenml.io>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Describe changes
Registering a flavor used to print this:
Now it prints this:
Pre-requisites
Please ensure you have done the following:
Types of changes