commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From S├ębastien Brisard (Commented) (JIRA) <>
Subject [jira] [Commented] (MATH-733) Add LogNormal distribution
Date Mon, 16 Jan 2012 06:49:40 GMT


S├ębastien Brisard commented on MATH-733:

Hi Dennis,
thanks for this contribution. I've looked into it, and am uncomfortable with the naming conventions
{{mean}} and {{standardDeviation}} which are *not* the mean and sd of the log-normal distribution,
but rather the underlying normal distribution.
Would you object to my renaming all these parameters and class variables to {{scale}} and
{{shape}}, as you yourself mention these naming conventions in the javadoc? I would then move
your explanation of the definition of these parameters to the class's documentation comment.
>From the user's perspective (heavily relying on automatic completion) I think it is dangerous
that {{getMean()}} should *not* return the mean value of the current distribution. Of course,
your javadoc comments are very clear, but... Also, my experience in materials science is that
people are always a bit "fuzzy" about log-normal distributions. What they call "sigma" is
never very clear...
Please let me know what you think.
> Add LogNormal distribution
> --------------------------
>                 Key: MATH-733
>                 URL:
>             Project: Commons Math
>          Issue Type: New Feature
>            Reporter: Dennis Hendriks
>              Labels: distribution
>             Fix For: 3.0
>         Attachments: lognormal_distribution.patch
> It would be nice to have a log-normal distribution.

This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:!default.jspa
For more information on JIRA, see:


View raw message