Skip to content
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

Fix inference CLI #762

Merged
merged 1 commit into from
Mar 4, 2019
Merged

Fix inference CLI #762

merged 1 commit into from
Mar 4, 2019

Conversation

ClemDoum
Copy link
Collaborator

@ClemDoum ClemDoum commented Mar 4, 2019

Description:

  • In Python 2.7, UTF-8 str input must be converted to unicode otherwise we can't type UTF-8 characters in the inference CLI

@ClemDoum ClemDoum requested a review from adrienball March 4, 2019 13:03
@codecov-io
Copy link

Codecov Report

Merging #762 into develop will decrease coverage by 0.03%.
The diff coverage is 60%.

@@             Coverage Diff             @@
##           develop     #762      +/-   ##
===========================================
- Coverage     88.1%   88.07%   -0.04%     
===========================================
  Files           74       74              
  Lines         4497     4502       +5     
  Branches       863      864       +1     
===========================================
+ Hits          3962     3965       +3     
- Misses         398      400       +2     
  Partials       137      137

@adrienball adrienball merged commit d2d62a2 into develop Mar 4, 2019
@adrienball adrienball deleted the fix/inference-cli branch March 4, 2019 13:41
Sign up for free to 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.

3 participants