How to Extend Flyte doc with a flowchart #763
Merged
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
Contributor
kumare3
commented
Feb 22, 2021
kumare3
commented
- Update the Available Plugins section to actually have a doc on extending Flyte, the background and how to make a decision
- Also added a new static resource
katrogan
previously approved these changes
Feb 22, 2021
| and you want this to be available like a template for all other users - open source or within your organization. This can be done by creating a task plugin. | ||
| A Task-plugin makes it possible for you or other users to use your idea natively within Flyte as it this capability was built into the flyte platform. | ||
|
|
||
| Thus for example, if you want users to write code simply using the ``@task`` decorator, but you want to provide a capability of running the function as a spark job or a sagemaker training job - then you can extend Flyte's task system - we will refer to this as the plugin and it could be possible to do the following |
There was a problem hiding this comment.
So one point of feedback we've received is that task_config is confusing as a name. Can we call out explicitly that task plugins are implemented with custom task configs?
There was a problem hiding this comment.
@katrogan I dont know if the ship has sailed, but what would you call it rather. this is configuration right?
There was a problem hiding this comment.
sure, i guess configuration just has multiple interpretations and perhaps this doesn't really belong in how to create plugins anyways. i can add a PR for going over that after this one
Co-authored-by: Katrina Rogan <[email protected]>
Co-authored-by: Katrina Rogan <[email protected]>
Co-authored-by: Katrina Rogan <[email protected]>
Co-authored-by: Katrina Rogan <[email protected]>
Co-authored-by: Katrina Rogan <[email protected]>
Co-authored-by: Katrina Rogan <[email protected]>
Co-authored-by: Katrina Rogan <[email protected]>
Co-authored-by: Katrina Rogan <[email protected]>
katrogan
approved these changes
Feb 22, 2021
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.