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] Loading iris on C locale #1998

Merged
merged 2 commits into from
Feb 14, 2017
Merged

[FIX] Loading iris on C locale #1998

merged 2 commits into from
Feb 14, 2017

Conversation

kernc
Copy link
Contributor

@kernc kernc commented Feb 4, 2017

Issue

#1997

Description of changes

Load *.metadata as UTF-8. Additionally, fix reporting filename and filetype in OWFile.

Includes
  • Code changes
  • Tests
  • Documentation

@astaric
Copy link
Member

astaric commented Feb 7, 2017

The failing test on windows seems related to the change. Can you take a look?

@codecov-io
Copy link

Codecov Report

Merging #1998 into master will not impact coverage.

@@           Coverage Diff           @@
##           master    #1998   +/-   ##
=======================================
  Coverage   89.52%   89.52%           
=======================================
  Files          91       91           
  Lines        9200     9200           
=======================================
  Hits         8236     8236           
  Misses        964      964

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5970d21...05834f3. Read the comment docs.

@kernc
Copy link
Contributor Author

kernc commented Feb 7, 2017

Indeed, encoding should be set also when writing the file. Thanks.

@astaric astaric added this to the 3.3.12 milestone Feb 13, 2017
@astaric astaric merged commit 992d6fd into biolab:master Feb 14, 2017
astaric added a commit to astaric/orange3 that referenced this pull request Feb 14, 2017
[FIX] Loading iris on C locale
(cherry picked from commit 992d6fd)
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