-
Notifications
You must be signed in to change notification settings - Fork 300
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
tensorflow plugin implementation #141
Conversation
@pingsutw can I look at it tomorrow morning? |
@kumare3 Sure, I'm not in a hurry. |
the PR is failing because of some un-related error. I see that happening for all PR's |
This actually looks pretty good to me. We should probably add an example in flytesnacks. |
@wild-endeavor do you have any comments? |
@kumare3 Thanks for the review. |
@pingsutw I don't think I have access to your fork. Could you help us clean up the lint errors? If not, feel free to change this to a PR to go in something other than the master branch, and I can pull it and modify it. Thanks! |
Hey I want to make a few more changes to this PR, so merged it into a different branch, but on the main fork. I'll open up a new PR into master and tag you on it. Thanks @pingsutw ! |
TL;DR
Tensorflow plugin for Flyte that leverages https://github.com/kubeflow/tf-operator to run distributed tensorflow jobs on k8s.
Type
Are all requirements met?
Complete description
Tensorflow plugin for Flyte that leverages https://github.com/kubeflow/tf-operator to run distributed TensorFlow jobs on k8s.
https://pypi.org/project/flyteidl/0.17.36/
Related PRs:
TFOperator support in FlytePlugins
flyteorg/flyte#115
Subtask:
flyteorg/flyteidl#71
flyteorg/flyteplugins#103
flyteorg/flyte#424
Tracking Issue
NA
Follow-up issue
NA