-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
Use JuliaLibm log functions in Main.Math. #24750
Conversation
Float64 The same thing goes for timings So Slightly slower around 1. cc: @simonbyrne |
Should the JuliaLibm-module part of it be deprecated as it's been mentioned in the docstrings? |
I doubt anyone has been relying on |
Alright, I was only wondering since it's actually mentioned in the docstring of |
I would say it is better to be conservative and deprecate them properly. |
Is there a special syntax that lets you deprecate unexported functions from modules? |
|
bd9bc4c
to
cc47850
Compare
@@ -2168,6 +2168,11 @@ end | |||
|
|||
@deprecate merge!(repo::LibGit2.GitRepo, args...; kwargs...) LibGit2.merge!(repo, args...; kwargs...) | |||
|
|||
# Remember to delete the module when removing this |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You could just move the whole definition here:
@eval Base.Math module JuliaLibm
Base.@deprecate log Base.log
end
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will fix!
Errors seem unrelated? |
Shameless bump. This is approved by two people, and tests seem to pass, so I am wondering if there is more I need to do. |
These should be just as precise as openlibm (own experience and https://tpapp.github.io/post/log1p/ ) but I will ofc provide solid numbers on this.
@stevengj raised the point that it's concerning that log1p seems a bit slower for small values as per the blog post above, and these are the values where you really want log1p. I'll benchmark this further and if it is the case, my idea is to do openlibm port for the small x part and tang's for the larger x's.
Should the old ones (those in a module) be deprecated? They've been mentioned in the docstrings, so...