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

Add wide_deep to the official models #2554

Merged
merged 4 commits into from Oct 19, 2017

Conversation

6 participants
@nealwu
Copy link
Member

nealwu commented Oct 19, 2017

No description provided.

@tensorflow-jenkins

This comment has been minimized.

Copy link
Collaborator

tensorflow-jenkins commented Oct 19, 2017

Can one of the admins verify this patch?

@nealwu nealwu requested a review from k-w-w Oct 19, 2017

@googlebot googlebot added the cla: yes label Oct 19, 2017

@nealwu

This comment has been minimized.

Copy link
Member

nealwu commented Oct 19, 2017

Jenkins, test this please.

@k-w-w

k-w-w approved these changes Oct 19, 2017

@nealwu nealwu requested review from itsmeolivia and jhseu as code owners Oct 19, 2017

@jhseu

jhseu approved these changes Oct 19, 2017

@av8ramit

This comment has been minimized.

Copy link
Member

av8ramit commented Oct 19, 2017

Jenkins, test this please.

@nealwu

This comment has been minimized.

Copy link
Member

nealwu commented Oct 19, 2017

Tests pass for me locally (both Python2 and Python3); going to merge.

@nealwu nealwu merged commit b4cf230 into master Oct 19, 2017

1 check passed

cla/google All necessary CLAs are signed

@nealwu nealwu deleted the wide_deep branch Oct 19, 2017

@nealwu nealwu added this to To Do in TensorFlow Official Models via automation Nov 7, 2017

@nealwu nealwu moved this from To Do to Done in TensorFlow Official Models Nov 7, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment