-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
[BugFix] fix exp returns incorrect value #47842
Conversation
a95af6a
to
a28172c
Compare
One concern is that should exp returns NULL when std::exp calculates infinity for output that's greater than the max value of double type. |
Feels like it should return inf rather than null when the output value of std::exp is greater than the max value of double type, just like other math functions. |
e97da13
to
88d7394
Compare
ignore the approve, should return NULL when overflow the max value, and the change doesn't explain |
@Seaven The reason that exp(7) returns NULL is ExpCheck function actually compares the output of exp(7) with |
I get it, I think you find a bug about |
Understood, will revise the code shortly. |
5afcb6d
to
424d504
Compare
…on returns NULL when output value is infinity Signed-off-by: Yi Wang <[email protected]>
[FE Incremental Coverage Report]✅ pass : 0 / 0 (0%) |
[BE Incremental Coverage Report]✅ pass : 3 / 3 (100.00%) file detail
|
@Seaven Pls have a look again~ |
✅ Backports have been created
|
✅ Backports have been created
|
✅ Backports have been created
|
✅ Backports have been created
|
✅ Backports have been created
|
Signed-off-by: Yi Wang <[email protected]> (cherry picked from commit e89d2e6)
Signed-off-by: Yi Wang <[email protected]> (cherry picked from commit e89d2e6)
Signed-off-by: Yi Wang <[email protected]> (cherry picked from commit e89d2e6) # Conflicts: # be/src/exprs/math_functions.cpp
Signed-off-by: Yi Wang <[email protected]> (cherry picked from commit e89d2e6) # Conflicts: # be/src/exprs/math_functions.cpp
Signed-off-by: Yi Wang <[email protected]> (cherry picked from commit e89d2e6) # Conflicts: # be/src/exprs/vectorized/math_functions.cpp
Co-authored-by: Yi <[email protected]>
Signed-off-by: Yi Wang <[email protected]> (cherry picked from commit e89d2e6) Signed-off-by: Yi <[email protected]>
Signed-off-by: Yi Wang <[email protected]> (cherry picked from commit e89d2e6) Signed-off-by: Yi <[email protected]>
Signed-off-by: Yi Wang <[email protected]> (cherry picked from commit e89d2e6) Signed-off-by: Yi <[email protected]>
Co-authored-by: Yi <[email protected]>
Signed-off-by: Yi <[email protected]> Co-authored-by: Yi <[email protected]>
Signed-off-by: Yi <[email protected]> Co-authored-by: Yi <[email protected]>
Signed-off-by: Yi Wang <[email protected]> (cherry picked from commit e89d2e6) Signed-off-by: Yi <[email protected]>
Signed-off-by: Yi <[email protected]> Co-authored-by: Yi <[email protected]>
Why I'm doing:
exp func returns NULL when input greater than 6
What I'm doing:
Fixes #issue
What type of PR is this:
Does this PR entail a change in behavior?
If yes, please specify the type of change:
Checklist:
Bugfix cherry-pick branch check: