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

doctest fix for: Re/Im(tanh) wrong formula #20098

Closed
rwst opened this issue Feb 23, 2016 · 7 comments
Closed

doctest fix for: Re/Im(tanh) wrong formula #20098

rwst opened this issue Feb 23, 2016 · 7 comments

Comments

@rwst
Copy link

rwst commented Feb 23, 2016

In analogy to #19791:

sage: tanh(1+2*I).n()
1.16673625724092 - 0.243458201185725*I
sage: tanh(1+2*I).real().n()
0.202053272732553
sage: tanh(1+2*I).imag().n()
-0.579697798322816
sage: tanh(x).real()
tanh(real_part(x))/(tan(imag_part(x))^2*tanh(real_part(x))^2 + 1)
sage: tanh(x).imag()
tan(imag_part(x))/(tan(imag_part(x))^2*tanh(real_part(x))^2 + 1)

See for example http://dlmf.nist.gov/4.35.E36

Will be fixed in Pynac-0.6.3 and should be doctested here.

CC: @behackl

Component: symbolics

Author: Ralf Stephan

Branch/Commit: 9eafdee

Reviewer: Marc Mezzarobba

Issue created by migration from https://trac.sagemath.org/ticket/20098

@rwst rwst added this to the sage-7.1 milestone Feb 23, 2016
@rwst

This comment has been minimized.

@rwst
Copy link
Author

rwst commented Mar 25, 2016

Branch: u/rws/re_im_tanh__wrong_formula

@rwst
Copy link
Author

rwst commented Mar 25, 2016

New commits:

9eafdee20098: doctest fix for Re/Im(tanh) wrong formula

@rwst
Copy link
Author

rwst commented Mar 25, 2016

Commit: 9eafdee

@rwst
Copy link
Author

rwst commented Mar 25, 2016

Author: Ralf Stephan

@rwst rwst changed the title Re/Im(tanh) wrong formula doctest fix for: Re/Im(tanh) wrong formula Mar 25, 2016
@mezzarobba
Copy link
Member

Reviewer: Marc Mezzarobba

@vbraun
Copy link
Member

vbraun commented Mar 28, 2016

Changed branch from u/rws/re_im_tanh__wrong_formula to 9eafdee

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants