Opened 3 years ago
Last modified 3 years ago
#24604 new defect
No evaluation with some functions
Reported by:  rws  Owned by:  

Priority:  major  Milestone:  sage8.2 
Component:  symbolics  Keywords:  
Cc:  Merged in:  
Authors:  Ralf Stephan  Reviewers:  
Report Upstream:  N/A  Work issues:  
Branch:  u/rws/24604 (Commits)  Commit:  f963efa53d3a109fec373ce2c5b49a2295198bc3 
Dependencies:  #24630, #24668  Stopgaps: 
Description
In a systematic search the following cases were found:
sage: arctan2(7,CDF(.92)) arctan2(7, 0.92) sage: arctan2(7,CBF(.92)) arctan2(7, [0.9200000000000000 +/ 4.00e17]) sage: elliptic_eu(7, CIF(1.7)) elliptic_eu(7, 1.7000000000000000?) sage: elliptic_eu(7, complex(1.7)) elliptic_eu(7, (1.7+0j)) sage: jacobi_am(7, CIF(1.7)) jacobi_am(7, 1.7000000000000000?) sage: jacobi_am(7, complex(1.7)) jacobi_am(7, (1.7+0j))
The systematic test will be committed with #24299.
Change History (7)
comment:1 Changed 3 years ago by
 Branch set to u/rws/24604
 Commit set to 0ab03cb2e1adc06aeb231aeb228b8ebeee69e557
comment:2 Changed 3 years ago by
 Dependencies set to pynac0.7.16
The beta
cases are fixed in Pynac0.7.16.
comment:3 Changed 3 years ago by
 Dependencies changed from pynac0.7.16 to #24630, pynac0.7.16
For jacobi_am
the eval function was enhanced. This revealed the conversion problem of #24630 which we now depend on.
comment:4 Changed 3 years ago by
 Commit changed from 0ab03cb2e1adc06aeb231aeb228b8ebeee69e557 to 4045fd5a6d8f0f90de062b9a8c5ed5cc5b59eb7d
Branch pushed to git repo; I updated commit sha1. New commits:
4045fd5  24604: fix jacobi_am eval

comment:5 Changed 3 years ago by
 Commit changed from 4045fd5a6d8f0f90de062b9a8c5ed5cc5b59eb7d to f963efa53d3a109fec373ce2c5b49a2295198bc3
comment:6 Changed 3 years ago by
Same applies to elliptic_eu
.
comment:7 Changed 3 years ago by
 Dependencies changed from #24630, pynac0.7.16 to #24630, #24668
Note: See
TracTickets for help on using
tickets.
The arctan cases were due to a case in
py_atan2
not converting to the commonP
.New commits:
24604: py_atan2 fixes