[issue33402] Change the fractions.Fraction class to convert to a unicode fraction string

2018-05-02 Thread Gabe Appleton
Change by Gabe Appleton : -- resolution: -> rejected stage: patch review -> resolved status: open -> closed ___ Python tracker

[issue33402] Change the fractions.Fraction class to convert to a unicode fraction string

2018-05-02 Thread Tim Peters
Tim Peters added the comment: -1. We should stop pretending this _ might_ happen ;-) -- nosy: +tim.peters ___ Python tracker ___

[issue33402] Change the fractions.Fraction class to convert to a unicode fraction string

2018-05-02 Thread STINNER Victor
STINNER Victor added the comment: I dislike using non-ASCII characters for the default implementation. I don't think that this formatting is popular, and I expect many troubles on each platform. IMHO it's very easy to put such short function in your favorite module, or

[issue33402] Change the fractions.Fraction class to convert to a unicode fraction string

2018-05-02 Thread Raymond Hettinger
Raymond Hettinger added the comment: In the Monaco font, the superscripts ⁰¹²³⁴⁵⁶⁷⁸⁹ don't all line-up. The 1, 2, and 3 are lower which makes the fraction look weird. -- nosy: +rhettinger ___ Python tracker

[issue33402] Change the fractions.Fraction class to convert to a unicode fraction string

2018-05-02 Thread Mark Dickinson
Mark Dickinson added the comment: -1 from me. Apart from anything else, the output for a general fraction looks (to my eyes) ugly in a fixed-width font: the tiny digits are harder to read, and there's too much space between successive numerator (or denominator) digits:

[issue33402] Change the fractions.Fraction class to convert to a unicode fraction string

2018-05-01 Thread Steven D'Aprano
Steven D'Aprano added the comment: Strings in Python 3 are already unicode. Looking at the patch, I see a lot of fractions which display as the missing glyph white square. For example, instead of seeing 1/9, which displays perfectly everywhere, I see a mysterious

[issue33402] Change the fractions.Fraction class to convert to a unicode fraction string

2018-05-01 Thread R. David Murray
R. David Murray added the comment: I vote -1. It's cute, but I'd much rather have a consistently ascii representation of something that is easily represented in ascii. -- nosy: +r.david.murray ___ Python tracker

[issue33402] Change the fractions.Fraction class to convert to a unicode fraction string

2018-05-01 Thread Eric V. Smith
Eric V. Smith added the comment: I'd bring it up on python-ideas, and point the discussion to this issue. I think the primary complain will be using non-ASCII characters in a function that normally doesn't return non-ASCII. But maybe people will be willing to accept it.

[issue33402] Change the fractions.Fraction class to convert to a unicode fraction string

2018-05-01 Thread Gabe Appleton
Gabe Appleton added the comment: Would it be workable if I instead just changed the __str__() method? I'm willing to go either way, but I feel like it's a bit nicer to have it output a nice fraction that way. -- ___ Python

[issue33402] Change the fractions.Fraction class to convert to a unicode fraction string

2018-05-01 Thread Eric V. Smith
Eric V. Smith added the comment: Your patch would break the usual and useful behavior of x == eval(repr(x)) >>> f = Fraction(1,2) >>> repr(f) 'Fraction(1, 2)' >>> eval(repr(f)) Fraction(1, 2) >>> f == eval(repr(f)) True Plus, I'm sure there's working code that would break

[issue33402] Change the fractions.Fraction class to convert to a unicode fraction string

2018-05-01 Thread Gabe Appleton
Change by Gabe Appleton : -- keywords: +patch pull_requests: +6372 stage: -> patch review ___ Python tracker ___

[issue33402] Change the fractions.Fraction class to convert to a unicode fraction string

2018-05-01 Thread Gabe Appleton
New submission from Gabe Appleton : Currently it has a __repr__() which returns `Fraction(x, y)`, and a __str__() which returns `x/y`. I have a ready pull request to change this to a scheme where both return unicode fractions. -- components: Library (Lib)