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 mark.dickinson
Recipients mark.dickinson, rhettinger, serhiy.storchaka, stutzbach, tim.peters
Date 2019-06-02.12:14:18
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1559477658.94.0.118231914365.issue37132@roundup.psfhosted.org>
In-reply-to
Content
I'm pretty much out of core-dev cycles for this weekend; I'm happy to review GH-13741, but won't have time to do so before next weekend.

I'm overall -0 on this change; there's a minor benefit in the separation, but for me it's outweighted by the duplication of `gcd` and `factorial` (or in the case of `gcd`, triplication, since it's also in the `fractions` module).

I'm out of time, so I'll be brief, but if this does go into 3.8, here are my thoughts:

- please remove the imath.as_integer_ratio function; it needs more discussion, and it has quite a different character from the other functions; it's not at all clear to me that it belongs. We might end up putting it back in eventually, but it needs discussion first, and we don't have time for that discussion before feature freeze.
- please add the pure Python version of imath that PEP 399 requires, or get Brett's confirmation that PEP 399 doesn't apply in this case.
- please get agreement from one (preferably both) of Tim and Raymond

Raymond, Tim: thoughts?
History
Date User Action Args
2019-06-02 12:14:18mark.dickinsonsetrecipients: + mark.dickinson, tim.peters, rhettinger, stutzbach, serhiy.storchaka
2019-06-02 12:14:18mark.dickinsonsetmessageid: <1559477658.94.0.118231914365.issue37132@roundup.psfhosted.org>
2019-06-02 12:14:18mark.dickinsonlinkissue37132 messages
2019-06-02 12:14:18mark.dickinsoncreate