search

kubeflow / training-operator

Distributed ML Training and Fine-Tuning on Kubernetes
https://www.kubeflow.org/docs/components/training
Apache License 2.0
1.51k stars 657 forks source link

Adapt TFJob examples to TensorFlow v2 #2015

Open tenzen-y opened 3 months ago

tenzen-y commented 3 months ago

Our TFJob examples are using TensorFlow v1, but that version is too old.

So, all TFJob examples should be adapted to TensorFlow v1.

Contributors should address the following items:

/good-first-issue

google-oss-prow[bot] commented 3 months ago

@tenzen-y: This request has been marked as suitable for new contributors.

Please ensure the request meets the requirements listed here.

If this request no longer meets these requirements, the label can be removed by commenting with the /remove-good-first-issue command.

In response to [this](https://github.com/kubeflow/training-operator/issues/2015): >Our TFJob examples are using TensorFlow v1, but that version is too old. > >- https://github.com/kubeflow/training-operator/blob/e2d6ba41d4d11eb333a80cb95a1c22a29e3da156/examples/tensorflow/tf_sample/Dockerfile#L1 >- https://github.com/kubeflow/training-operator/blob/e2d6ba41d4d11eb333a80cb95a1c22a29e3da156/examples/tensorflow/dist-mnist/Dockerfile#L15 >- https://github.com/kubeflow/training-operator/blob/e2d6ba41d4d11eb333a80cb95a1c22a29e3da156/examples/tensorflow/distribution_strategy/estimator-API/Dockerfile#L1 >- https://github.com/kubeflow/training-operator/blob/e2d6ba41d4d11eb333a80cb95a1c22a29e3da156/examples/tensorflow/mnist_with_summaries/Dockerfile#L15 > >So, all TFJob examples should be adapted to TensorFlow v1. > >Contributors should address the following items: > - [ ] Upgrade TensorFlow version in all Dockerfiles to v2 > - [ ] Adapt training codes to TensorFlow v2 API > >/good-first-issue > Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository.
sandipantech commented 3 months ago

I am interested in working on this.

/assign

github-actions[bot] commented 3 weeks ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

tenzen-y commented 3 weeks ago

/remove-lifecycle stale