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

Configurability of unicode/console encoding #1654

Closed
wants to merge 1 commit into from
Closed

Configurability of unicode/console encoding #1654

wants to merge 1 commit into from

Conversation

changhiskhan
Copy link
Contributor

#1620

checks whether there are non-ascii characters and tries to convert them if possible
allows encoding to be set in set_printoptions (default UTF8)

@wesm
Copy link
Member

wesm commented Jul 20, 2012

Crap, I fixed this on the plane. Looks like you did a better job by
adding a configuration option

Sent from my mobile device

On Jul 20, 2012, at 6:20 PM, Chang She
reply@reply.github.com
wrote:

checks whether there are non-ascii characters and tries to convert them if possible
allows encoding to be set in set_printoptions (default UTF8)

You can merge this Pull Request by running:

git pull https://github.com/changhiskhan/pandas nonascii

Or you can view, comment on it, or merge it online at:

#1654

-- Commit Summary --

-- File Changes --

M pandas/core/common.py (8)
M pandas/core/format.py (33)
M pandas/tests/test_format.py (7)

-- Patch Links --

https://github.com/pydata/pandas/pull/1654.patch
https://github.com/pydata/pandas/pull/1654.diff


Reply to this email directly or view it on GitHub:
#1654

@changhiskhan
Copy link
Contributor Author

Doh, I claimed it but forgot to push yesterday, my bad

On Jul 20, 2012, at 6:24 PM, Wes McKinney wrote:

Crap, I fixed this on the plane. Looks like you did a better job by
adding a configuration option

Sent from my mobile device

On Jul 20, 2012, at 6:20 PM, Chang She
reply@reply.github.com
wrote:

checks whether there are non-ascii characters and tries to convert them if possible
allows encoding to be set in set_printoptions (default UTF8)

You can merge this Pull Request by running:

git pull https://github.com/changhiskhan/pandas nonascii

Or you can view, comment on it, or merge it online at:

#1654

-- Commit Summary --

-- File Changes --

M pandas/core/common.py (8)
M pandas/core/format.py (33)
M pandas/tests/test_format.py (7)

-- Patch Links --

https://github.com/pydata/pandas/pull/1654.patch
https://github.com/pydata/pandas/pull/1654.diff


Reply to this email directly or view it on GitHub:
#1654


Reply to this email directly or view it on GitHub:
#1654 (comment)

@takluyver
Copy link
Contributor

My standard comment for anything unicode related - are we sure this won't mess things up in Python 3? I'm not sure when these functions get called, but, for instance, if _conv_unicode() is called on a bytes string, it will always return False.

@wesm
Copy link
Member

wesm commented Jul 21, 2012

Hey @changhiskhan I pushed the fix I'd cooked up ae70acc and 4e94fba. Unit test suite is passing-- it would be nice to incorporate the configurability of the encoding in this PR, so I'm going to leave it open for now.

@ghost ghost assigned changhiskhan Jul 23, 2012
@ghost
Copy link

ghost commented Oct 11, 2012

#2005 also fixes this, replaces the small helper functions with the generic
print_thing() function, and fixes up the repr to always use the encoding in
print_config on py2.x, and leave things well alone on py3.

unless I missed something , vote to close?

@wesm
Copy link
Member

wesm commented Oct 11, 2012

Agreed-- it looks like the encoding has already been made configurable in set_printoptions so we can close this.

@wesm wesm closed this Oct 11, 2012
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