-
Notifications
You must be signed in to change notification settings - Fork 102
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add (and unify) Julia Lambert W function? #27
Comments
Thats a good idea. I haven't followed recent discussion on core library, metapackages, etc. But, LambertW should be in some kind of "special functions" package or metapackage so people don't have to scrounge around for this or that function..... I just realized that this comment is in SpecialFunctions.jl and not my LambertW.jl repo. So, yes, I am for incorporating the jlapeyre LambertW repo into SpecialFunctions.jl . There is also jlapeyre/FunctionZeros.jl I forgot about the robertdj LambertW.jl. It appears to predate this one. I just did some quick tests on large arrays and found that either one of the LambertW's is consistently faster than the other, depending on both argument and branch. Usually by less than a factor of two. They both appear accurate more or less to machine precision. But, the jlapeyre LambertW also handles all the branches, complex arguments, and BigFloats. The robertdj LambertW is only useful for two branches and Float64 arguments. From my point of view, it is perhaps worth incorporating some |
I may be able to work on a PR this weekend. |
It sounds like a good idea to incorporate the Lambert W in this broader package! My version definitely has a more modest scope, but let me know if I can help out. |
Bump :-) |
@mschauer I felt that bump. Must have traveled slowly. :) |
LambertW.jl (from jlapeyre) has been moved to https://github.com/JuliaMath/LambertW.jl |
There are two packages for the Lambert W function:
LambertW.jl, and
LambertW.jl.
Surely the Lambert W function is special enough for the special club.
CC @jlapeyre, @robertdj.
The text was updated successfully, but these errors were encountered: