Bug 19283 – [std.mathspecial] documentation for normal distribution doesn't list parameters

Status
RESOLVED
Resolution
FIXED
Severity
trivial
Priority
P3
Component
phobos
Product
D
Version
D2
Platform
All
OS
All
Creation time
2018-10-02T19:43:12Z
Last change time
2019-10-16T15:37:49Z
Keywords
bootcamp, pull
Assigned to
dcuan17
Creator
Ray Kulhanek

Comments

Comment #0 by kulhanek.5 — 2018-10-02T19:43:12Z
The documentation for normalDistribution and normalDistributionInverse does not state what mean and standard deviation are used. Presumably mean=0.0 and stdev=1.0; this is consistent with the cumulative distribution equation listed in the normalDistribution docs and the code for the z<1 case in std.internal.math.errorfunction.normalDistributionImpl. Still, it'd probably be a good idea for someone familiar with std.internal.math.errorfunction to double check that before updating the docs; there's some numerical stability code (I think) in there that I was unable to follow.
Comment #1 by dlang-bot — 2019-10-16T14:26:15Z
@berni44 created dlang/phobos pull request #7233 "Fix Issue 19283 - [std.mathspecial] documentation for normal distribution doesn't list parameters" fixing this issue: - Fix Issue 19283 - [std.mathspecial] documentation for normal distribution doesn't list parameters https://github.com/dlang/phobos/pull/7233
Comment #2 by dlang-bot — 2019-10-16T15:37:49Z
dlang/phobos pull request #7233 "Fix Issue 19283 - [std.mathspecial] documentation for normal distribution doesn't list parameters" was merged into master: - eb9889abd590c76737973a532a00dfafedbbbf95 by Bernhard Seckinger: Fix Issue 19283 - [std.mathspecial] documentation for normal distribution doesn't list parameters https://github.com/dlang/phobos/pull/7233