You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Eduardo Ponce / @edponce:
I consider having (1) ln, (2) log10, and (3) log2 is a good starting point, and the missing variants are:
(4) log(x, base) = log(x) / log(base), note that log(base) is a constant w.r.t. to Datum x so optimizations can be made
(5) log1p(x) = log(1+x)
Note that (4) is the most general variant as it can represent any of the other variants, so it might be worthwhile to consider adding it to Arrow.
As additional context, here is the partial landscape of log functions across library/languages:
David Li / @lidavidm:
Thanks for looking all these over. I think we can add #5 easily. For #4, would we pass the log base as a kernel option? Or as another argument? (We'd only be able to optimize in the former case, I guess, unless we required a scalar argument)
ln, log, log2?
Reporter: David Li / @lidavidm
Assignee: David Li / @lidavidm
Related issues:
PRs and other links:
Note: This issue was originally created as ARROW-13096. Please see the migration documentation for further details.
The text was updated successfully, but these errors were encountered: