Skip to content
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

asinh does not comply with IEEE #5760

Open
p6rt opened this issue Oct 19, 2016 · 3 comments
Open

asinh does not comply with IEEE #5760

p6rt opened this issue Oct 19, 2016 · 3 comments

Comments

@p6rt
Copy link

@p6rt p6rt commented Oct 19, 2016

Migrated from rt.perl.org#129919 (status was 'new')

Searchable as RT129919$

@p6rt

This comment has been minimized.

Copy link
Author

@p6rt p6rt commented Oct 19, 2016

From @zoffixznet

Perl IEEE[^1]​:
1) asinh(-Inf) should return -Inf
2) Any too-large values that don't fit should be an Inf with the correct sign.
3) asinh(-0) gives -0

I fixed[^2] the first issue for num and Num, but not yet sure what the
correct solution is for Complex.
The rest of the issues are still present​:

  <ZoffixW> m​: dd asinh -1e200
  <camelia> rakudo-moar 74d0e3​: OUTPUT«Inf␤» # should be -Inf
  <ZoffixW> m​: dd asinh my num $ = -0e0
  <camelia> rakudo-moar 74d0e3​: OUTPUT«0e0␤» # should be -0e0
  <ZoffixW> m​: dd asinh -0e0
  <camelia> rakudo-moar 74d0e3​: OUTPUT«0e0␤» # should be -0e0
  <ZoffixW> m​: dd asinh -1e200+1i
  <camelia> rakudo-moar 74d0e3​: OUTPUT«<NaN+NaN\i>␤» # Not sure
what it should be; WolframAlpha just gives -Inf, with no indication of
the imaginary part

[1] http://www.csee.umbc.edu/~tsi%E2%80%8Bmo1/CMSC455/IEEE-754-2008.pdf
[2]
rakudo/rakudo@74d0e36

@p6rt

This comment has been minimized.

Copy link
Author

@p6rt p6rt commented Oct 19, 2016

From @zoffixznet

Corrected IEEE link http://www.csee.umbc.edu/~tsimo1/CMSC455/IEEE-754-2008.pdf
Relevant info are in sections 7 and 9. Try searching for asinh

@p6rt

This comment has been minimized.

Copy link
Author

@p6rt p6rt commented Oct 19, 2016

From @zoffixznet

♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥
🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁

TODO-fudged tests added as part of perl6/roast@6c31fe3

🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁🏁
♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥♥

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant
You can’t perform that action at this time.