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

Thomas Neidhart commented on MATH-1066:
---------------------------------------

Sorry for the late feedback, you surely put a lot of effort into this patch.
We had some bad experience with code translated too literally from fortran as 
it tends to be very difficult to understand / maintain.

Having said that and quickly looking at your patch, I would say that this 
functionality is definitely interesting for CM but would still need a lot of 
work to make it ready for committing.

The main problem I see with the rjbesl method, and I do not know if this could 
be re-written in a more Java-friendly way without breaking functionality.

> Add Bessel functions of the first kind
> --------------------------------------
>
>                 Key: MATH-1066
>                 URL: https://issues.apache.org/jira/browse/MATH-1066
>             Project: Commons Math
>          Issue Type: Improvement
>    Affects Versions: 3.2
>            Reporter: Brian Wignall
>              Labels: newbie, patch, special-functions
>         Attachments: patch_besselj
>
>   Original Estimate: 168h
>  Remaining Estimate: 168h
>
> According to the Wish List, special functions are a priority item to be added 
> to ACM. As Bessel functions (http://en.wikipedia.org/wiki/Bessel_function) 
> are listed by name, I have translated some Fortran code from Netlib -- 
> specifically, http://www.netlib.org/specfun/rjbesl -- for Bessel functions of 
> the first kind.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to