[ 
https://issues.apache.org/jira/browse/NUMBERS-167?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17427679#comment-17427679
 ] 

Gilles Sadowski commented on NUMBERS-167:
-----------------------------------------

bq. {{ComplementaryFunctions}} interface [...] should be elevated to the 
package level.

>From a programming POV certainly, but I'm not sure about the proper naming...

Another (unrelated) issue: I think that we should really have
{code}
public enum RegularizedGamma {
    P(...),
    Q(...);

    // Methods for generating "ComplementaryFunctions" instances.
}
{code}
instead of the current
{code}
public class RegularizedGamma {
    public enum RG {
        P(...),
        Q(...);

        // ...
    }

    public static class P { ... }

    public static class Q { ... }
}
{code}

Do you know whether the change could be done while retaining BC?

> RegularizedGamma.P with precomputed LogGamma value
> --------------------------------------------------
>
>                 Key: NUMBERS-167
>                 URL: https://issues.apache.org/jira/browse/NUMBERS-167
>             Project: Commons Numbers
>          Issue Type: Wish
>          Components: gamma
>            Reporter: Gilles Sadowski
>            Priority: Minor
>             Fix For: 1.1
>
>         Attachments: pr_106.patch
>
>          Time Spent: 0.5h
>  Remaining Estimate: 0h
>
> We have
> {code:java}
> double v = RegularizedGamma.P.value(a, x);
> {code}
> where method {{value}} internally calls {{LogGamma.value(a)}}.
> There is a use-case for
> {code:java}
> double logGammaA = LogGamma.value(a);
> double v = RegularizedGamma.P.value(a, x, logGammaA);
> {code}
> for when the user varies {{x}} but not {{a}}.
> Method name TBD:  Another overload of {{value}} may be confusing (?).



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to