Skip to content

Add TextInput support - #189

Merged
terrytangyuan merged 3 commits into
tensorflow:masterfrom
yongtang:text_input
Apr 17, 2019
Merged

Add TextInput support#189
terrytangyuan merged 3 commits into
tensorflow:masterfrom
yongtang:text_input

Conversation

@yongtang

@yongtangyongtang commented Apr 15, 2019

Copy link
Copy Markdown
Member

This PR add TextInput support, with expanded archive support.

Signed-off-by: Yong Tang yong.tang.github@outlook.com

@yongtang

Copy link
Copy Markdown
MemberAuthor

The build fail with Ubuntu 16.04 and 18.04, likely caused by the gcc version mismatch of the binary in tensorflow.

The gdb debug shows the segmentation fault happens in TypeId() call when marshaling/unmarshaling the Variant type. This is likely similar to #173

@yongtangyongtang changed the title Add TextInput support[WIP] Add TextInput supportApr 15, 2019
@yongtang
yongtangforce-pushed the text_input branch 2 times, most recently from 355a2fc to 54804baCompareApril 16, 2019 15:49
@yongtangyongtang changed the title [WIP] Add TextInput supportAdd TextInput supportApr 16, 2019
@yongtang

Copy link
Copy Markdown
MemberAuthor

Removed building on Ubuntu 16.04 and 18.04 with tensorflow binaries, as it will not work. Our CI is still fully covered, as we build our binary on Ubuntu 14.04, and test on Ubuntu 16.04 and 18.04.

This PR is ready for review.

/cc @terrytangyuan

@yongtang

Copy link
Copy Markdown
MemberAuthor

@terrytangyuan All test finally passed 🎉

@terrytangyuan

Copy link
Copy Markdown
Member

Oops seems like there's conflict from the recently merged PR. Could you resolve it?

This PR add TextInput support, with expanded archive support.
Signed-off-by: Yong Tang <yong.tang.github@outlook.com>
with the same version of gcc with tensorflow
Signed-off-by: Yong Tang <yong.tang.github@outlook.com>
Signed-off-by: Yong Tang <yong.tang.github@outlook.com>
@yongtang

Copy link
Copy Markdown
MemberAuthor

@terrytangyuan The PR has been rebased and all tests passed now.

@terrytangyuanterrytangyuan left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks!

@terrytangyuan
terrytangyuan merged commit dc0c188 into tensorflow:masterApr 17, 2019
@yongtang
yongtang deleted the text_input branch April 17, 2019 20:25
i-ony pushed a commit to i-ony/io that referenced this pull request Feb 8, 2021
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@yongtang@terrytangyuan