On Sat, May 23, 2015 at 12:53 AM, Chris Barker chris.barker@noaa.gov wrote:
On Fri, May 22, 2015 at 2:33 PM, Larry Hastings larry@hastings.org wrote:
On 05/22/2015 02:29 PM, Chris Barker wrote:
Is it too late to get the isclose() code (PEP 485) into 3.5?
...
Hopefully you can find a core dev familiar enough with the issues involved that they can (quickly!) guide it through the process of getting it checked in.
Ping! Anyone willing to sponsor this?
Hi Chris,
Thanks for the PEP and the implementation!
You'll get more attention If you open an issue with a patch at bugs.python.org. Having a GitHub repository is good, but the isclose() code(with tests and documentation) needs to be integrated into the CPython code base:
--Berker
On Sat, May 23, 2015 at 1:34 AM, Berker Peksağ berker.peksag@gmail.com wrote:
On Sat, May 23, 2015 at 12:53 AM, Chris Barker chris.barker@noaa.gov wrote:
On Fri, May 22, 2015 at 2:33 PM, Larry Hastings larry@hastings.org wrote:
On 05/22/2015 02:29 PM, Chris Barker wrote:
Is it too late to get the isclose() code (PEP 485) into 3.5?
...
Hopefully you can find a core dev familiar enough with the issues involved that they can (quickly!) guide it through the process of getting it checked in.
Ping! Anyone willing to sponsor this?
...
- The C implementation should be in Modules/mathmodule.c
- Tests should be in Lib/test/test_math.py
- Documentation should be in Doc/library/math.rst
- Add an entry to Doc/whatsnew/3.5.rst
- If I remember correctly, we don't need the Python implementation and its tests
I'll happily review the patch once it's on the bug tracker as Berker described.
On 23 May 2015 at 19:29, Tal Einat taleinat@gmail.com wrote:
On Sat, May 23, 2015 at 1:34 AM, Berker Peksağ berker.peksag@gmail.com wrote:
On Sat, May 23, 2015 at 12:53 AM, Chris Barker chris.barker@noaa.gov wrote:
On Fri, May 22, 2015 at 2:33 PM, Larry Hastings larry@hastings.org wrote:
On 05/22/2015 02:29 PM, Chris Barker wrote:
Is it too late to get the isclose() code (PEP 485) into 3.5?
...
Hopefully you can find a core dev familiar enough with the issues involved that they can (quickly!) guide it through the process of getting it checked in.
Ping! Anyone willing to sponsor this?
...
- The C implementation should be in Modules/mathmodule.c
- Tests should be in Lib/test/test_math.py
- Documentation should be in Doc/library/math.rst
- Add an entry to Doc/whatsnew/3.5.rst
- If I remember correctly, we don't need the Python implementation and its tests
I'll happily review the patch once it's on the bug tracker as Berker described.
I filed http://bugs.python.org/issue24270 to track this, but there's a fair bit of work to be done to integrate the changes into the existing math module's code, tests and documentation.
And correct, there's no need for a pure Python implementation - Guido rejected the idea of a pure Python fallback for the math module a while back (http://bugs.python.org/issue23595)
Regards, Nick.
On 05/23/2015 06:25 AM, Nick Coghlan wrote:
I filed http://bugs.python.org/issue24270 to track this, but there's a fair bit of work to be done to integrate the changes into the existing math module's code, tests and documentation.
I'm willing to consider a feature freeze exception for this, as long as
Somebody, please take that as an encouragement to get this cleaned up and ready for checkin.
//arry/
p.s. Would it make sense to add a form of isclose to unittest?
On Sat, May 23, 2015 at 4:25 PM, Nick Coghlan ncoghlan@gmail.com wrote:
On 23 May 2015 at 19:29, Tal Einat taleinat@gmail.com wrote:
On Sat, May 23, 2015 at 1:34 AM, Berker Peksağ berker.peksag@gmail.com wrote:
- The C implementation should be in Modules/mathmodule.c
- Tests should be in Lib/test/test_math.py
- Documentation should be in Doc/library/math.rst
- Add an entry to Doc/whatsnew/3.5.rst
- If I remember correctly, we don't need the Python implementation and its tests
I'll happily review the patch once it's on the bug tracker as Berker described.
I filed http://bugs.python.org/issue24270 to track this, but there's a fair bit of work to be done to integrate the changes into the existing math module's code, tests and documentation.
Done. Patch attached to the issue. Awaiting review!