-
Notifications
You must be signed in to change notification settings - Fork 565
Removed outdated recommendation #4293
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
base: develop
Are you sure you want to change the base?
Conversation
🔍 Broken Links ReportSummary
Details
📂 Full file paths
|
Documentation Link Check Results❌ Absolute links check failed |
| - Hyperparameter tuning | ||
|
|
||
| Note that when implementing the fan-in step, you'll need to use the ZenML Client to query the results from previous parallel steps, as shown in the example above, and you can't pass in the result directly. | ||
| {% hint style="info" %} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Dynamic pipelines is still experimental, I wouldn't recommend them to anyone yet. But even once they're ready, don't you think it makes sense to keep the code sample on how to do it with static pipelines as well, just for the users that want to/must use static pipelines for some reason or another?
We can make the fan-in much nicer now even, as in static pipelines we also allow list of input artifacts, which gets rid of this whole code in the combine step. The only real limitation of static pipelines is the fan-out amount is fixed, but in some cases this might not even be a problem for users.
|
|
Describe changes
I implemented/fixed _ to achieve _.
Pre-requisites
Please ensure you have done the following:
developand the open PR is targetingdevelop. If your branch wasn't based on develop read Contribution guide on rebasing branch to develop.Types of changes