classification
Title: Add C fastpath for statistics.NormalDist.inv_cdf()
Type: Stage: patch review
Components: Extension Modules Versions: Python 3.9
process
Status: open Resolution:
Dependencies: Superseder:
Assigned To: Nosy List: corona10, rhettinger, steven.daprano
Priority: normal Keywords: easy (C), patch

Created on 2019-08-09 04:43 by rhettinger, last changed 2019-08-14 06:36 by corona10.

Pull Requests
URL Status Linked Edit
PR 15266 open corona10, 2019-08-14 06:36
Messages (3)
msg349273 - (view) Author: Raymond Hettinger (rhettinger) * (Python committer) Date: 2019-08-09 04:43
Create new module:  Modules/_statisticsmodule.c

Add a function:    _normal_dist_inv_cdf(p, mu, sigma) |-> x

Mostly, it should be a cut-and-paste from the pure Python version, just add argument processing and semi-colons.

Expect to measure a manyfold speedup.
msg349521 - (view) Author: Dong-hee Na (corona10) * Date: 2019-08-13 05:30
@rhettinger

Hi, Can I work on this issue?
It might be my first C module addition task for CPython.
msg349527 - (view) Author: Raymond Hettinger (rhettinger) * (Python committer) Date: 2019-08-13 06:33
> Can I work on this issue?  
> It might be my first C module addition task for CPython.

Yes.  This issue would be a good place to start.
History
Date User Action Args
2019-08-14 06:36:11corona10setkeywords: + patch
stage: patch review
pull_requests: + pull_request14987
2019-08-13 06:33:48rhettingersetmessages: + msg349527
2019-08-13 05:30:22corona10setnosy: + corona10
messages: + msg349521
2019-08-09 04:43:26rhettingercreate