numexpr icon indicating copy to clipboard operation
numexpr copied to clipboard

Add support for Bessel functions

Open rnorris7756 opened this issue 8 years ago • 2 comments

I would be willing to work on this. The C++ standard library does not have great support for Bessel functions, but the boost library has a good implementation here. What is this project's policy on including pieces of external libraries like boost?

rnorris7756 avatar Nov 30 '16 19:11 rnorris7756

Well, I think that evaluating Bessel functions inside numexpr is really too specific. numexpr comes with a virtual machine for evaluating expressions that has to lookup for many operators and functions already. Adding more functions is going to create a noticeable performance impact for all the other evaluations (even if they do not use Bessel functions). So I am -1 on this.

FrancescAlted avatar Dec 01 '16 08:12 FrancescAlted

Fair enough.

rnorris7756 avatar Dec 01 '16 18:12 rnorris7756

Message to comment on stale issues. If none provided, will not mark issues stale

github-actions[bot] avatar Feb 21 '24 01:02 github-actions[bot]