-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Rewrite ResNet sample by GCP components #1018
Conversation
Do you think they might be still be useful for the CMLE container training? |
Alexey, I am not sure where it is going to be used, do you know any user is using that? |
Probably no one is using them. I thought they might be useful to demonstrate using the same container to train on both CMLE and TFJob. /lgtm |
Can you please add a bit more information here about what code is just moved, what code is remove and what code is added. I see that some component code is just moved, but I'm not completely sure about some other code files. Thanks. |
Done and need another LGTM. Thanks. |
/retest |
/lgtm |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: hongye-sun The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: hongye-sun The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
CloudBuild is failing:
When deleting files we should make sure they're not mentioned anywhere: |
kubeflow#1018) * Genericized component logic and cleaned up valdation and other interfaces * Fixed a validation bug for custom components * Moved component validation logic to component.go
The PR includes:
This change is