-
Notifications
You must be signed in to change notification settings - Fork 706
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
Enable tests for all frameworks #1311
Comments
/help |
@Jeffwan: Please ensure the request meets the requirements listed here. If this request no longer meets these requirements, the label can be removed In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/assign |
As we discussed today on the AutoML and Training meeting, we want to use Python scripts instead of deprecated To transfer this workflow to Python we should do the similar steps as for the Kubeflow Notebook Controller:
I think we can simplify workflow utils for the Training Operator to leave only necessary functions. Does it sound good @Jeffwan @thunderboltsid @johnugeorge ? cc @kimwnasptd for reference. |
I created PR in Katib: kubeflow/katib#1674 to switch from ksonnet to python. |
Sounds really good |
Using python, we can still leverage kubeflow/testing common code base, right? |
@Jeffwan Yes, we can. I am using shared infra utils. |
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. |
Current project only have test cases for Tensorflow. We should enable tests for all rest frameworks in the pipeline.
The text was updated successfully, but these errors were encountered: