This issue tracker has been migrated to GitHub ,
and is currently read-only.
For more information,
see the GitHub FAQs in the Python's Developer Guide.
| Author | jyasskin |
|---|---|
| Recipients | gvanrossum, jyasskin, mark.dickinson |
| Date | 2008年01月09日.07:36:22 |
| SpamBayes Score | 0.017880877 |
| Marked as misclassified | No |
| Message-id | <1199864185.54.0.551339308957.issue1682@psf.upfronthosting.co.za> |
| In-reply-to |
| Content | |
|---|---|
Thanks again for the excellent comments.
__init__: good catch.
repr(Rational): The rule for repr is "eval(repr(object)) == object".
Unfortunately, that doesn't decide between the two formats, since both
assume some particular import statements. I picked the one more likely
to be unique, and I assume Decimal picked the shorter one. I can go
either way.
_gcd's sign: It's a happy accident for me. Possibly Sjoerd Mullender
designed it that way. I've added a comment and a test.
__ceil__: I like that implementation better.
2-argument round: Fixed and tested.
equality: Very good point. I've stolen the sandbox code and added
Rational.from_float() using it. I think I also need to make this change
to the comparisons.
hashing: oops, yes these should be hashable. Decimal cheats by comparing
!= to even floats that it's equal to, so I'm going to assume that they
also want Rational(5,2) != Decimal('2.5').
The new patch is against 2.6. |
|
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2008年01月09日 07:36:25 | jyasskin | set | spambayes_score: 0.0178809 -> 0.017880877 recipients: + jyasskin, gvanrossum, mark.dickinson |
| 2008年01月09日 07:36:25 | jyasskin | set | spambayes_score: 0.0178809 -> 0.0178809 messageid: <1199864185.54.0.551339308957.issue1682@psf.upfronthosting.co.za> |
| 2008年01月09日 07:36:24 | jyasskin | link | issue1682 messages |
| 2008年01月09日 07:36:22 | jyasskin | create | |