Skip to content
This repository was archived by the owner on Jul 7, 2023. It is now read-only.

Simplify is_unicode() #1318

Merged
merged 1 commit into from
Jan 4, 2019
Merged

Simplify is_unicode() #1318

merged 1 commit into from
Jan 4, 2019

Conversation

cclauss
Copy link
Contributor

@cclauss cclauss commented Dec 21, 2018

@googlebot googlebot added the cla: yes PR author has signed CLA label Dec 21, 2018
@afrozenator
Copy link
Contributor

thanks a lot @cclauss -- much appreciated!

@afrozenator afrozenator merged commit ef12bee into tensorflow:master Jan 4, 2019
@cclauss cclauss deleted the patch-3 branch January 4, 2019 22:59
tensorflow-copybara pushed a commit that referenced this pull request Jan 4, 2019
PiperOrigin-RevId: 227921102
@afrozenator
Copy link
Contributor

@cclauss - I see both the functions correctly at master - https://github.com/tensorflow/tensor2tensor/blob/master/tensor2tensor/data_generators/text_encoder.py

@cclauss
Copy link
Contributor Author

cclauss commented Jan 4, 2019

You are correct. I was mistaken. Thanks.

@afrozenator
Copy link
Contributor

I've been tripped up in the past by this, it is very confusing.

kpe pushed a commit to kpe/tensor2tensor that referenced this pull request Mar 2, 2019
kpe pushed a commit to kpe/tensor2tensor that referenced this pull request Mar 2, 2019
PiperOrigin-RevId: 227921102
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla: yes PR author has signed CLA
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants