[Python-Dev] PEP 485 review (isclose()) (original) (raw)
Chris Barker chris.barker at noaa.gov
Fri Feb 27 21:07:48 CET 2015
- Previous message: [Python-Dev] PEP 485 review (isclose())
- Next message: [Python-Dev] PEP 485 review (isclose())
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Thank you Guido.
It'll be nice to see this all come to something.
Thanks to all who contributed to the discussion -- despite this being a pretty simple function, I learned a lot and far more fully appreciate the nuance of all of this.
I'll edit the text as you suggest, and then work on a patch -- I'm sure I'll have questions for Python-dev when I actually do that, but I'll get started on my own and see how far I get.
-Chris
On Fri, Feb 27, 2015 at 11:38 AM, Guido van Rossum <guido at python.org> wrote:
I think it's time to accept PEP 485. I've re-read it once more, and it looks like the text is in great shape. (My only recommendation would be to update the Abstract to state that we're specifically adding math.isclose().)
A wording question: "This implementation has a flag that lets the user select which relative tolerance test to apply -- this PEP does not suggest that that be retained, but rather than the weak test be selected." -- I think this was meant to say "... rather that the weak test be selected", right? (It would be nice if the sample implementation defaulted to the choice in the PEP.) However, those are just minor edits, and I hereby approve the PEP. Thanks Chris and everyone else for the fruitful discussion (and thanks especially to Chris for eventually ending the bikeshedding and writing a PEP that explains each of the choices). Congrats! -- --Guido van Rossum (python.org/~guido)
--
Christopher Barker, Ph.D. Oceanographer
Emergency Response Division NOAA/NOS/OR&R (206) 526-6959 voice 7600 Sand Point Way NE (206) 526-6329 fax Seattle, WA 98115 (206) 526-6317 main reception
Chris.Barker at noaa.gov -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://mail.python.org/pipermail/python-dev/attachments/20150227/a0a864f9/attachment.html>
- Previous message: [Python-Dev] PEP 485 review (isclose())
- Next message: [Python-Dev] PEP 485 review (isclose())
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]