Opened 3 years ago
Closed 3 years ago
#25251 closed defect (fixed)
Doctest: Certain products cause pynac to deadloop
Reported by:  embray  Owned by:  

Priority:  major  Milestone:  sage8.3 
Component:  symbolics  Keywords:  
Cc:  Merged in:  
Authors:  Ralf Stephan  Reviewers:  Frédéric Chapoton 
Report Upstream:  N/A  Work issues:  
Branch:  495e9f4 (Commits, GitHub, GitLab)  Commit:  495e9f47ba235f743ddf62741a5bc12d9e8f0eb5 
Dependencies:  #24838  Stopgaps: 
Description (last modified by )
As discussed in https://groups.google.com/d/msg/sagedevel/a_2RU4hTdM8/NApTFCiMBgAJ
The example I found (and there might be others) was:
sage: e1 = sqrt(2)*I  sqrt(2)  2 sage: e2 = sqrt(2) sage: e1 * e2
According to rws the following fix to pynac, not yet in Sage, fixes the issue: https://github.com/pynac/pynac/commit/c4b2a4a42c6f01f0db6c3d54d49cccc3a049806e
The commit is contained in pynac0.7.17 (introduced with #24838).
Change History (9)
comment:1 Changed 3 years ago by
 Dependencies set to #24838
 Description modified (diff)
comment:2 Changed 3 years ago by
 Description modified (diff)
comment:3 Changed 3 years ago by
comment:4 Changed 3 years ago by
 Branch set to u/rws/25251
comment:5 Changed 3 years ago by
 Commit set to 495e9f47ba235f743ddf62741a5bc12d9e8f0eb5
 Report Upstream changed from Fixed upstream, but not in a stable release. to N/A
 Status changed from new to needs_review
 Summary changed from Certain products cause pynac to deadloop to Doctest: Certain products cause pynac to deadloop
New commits:
495e9f4  25251: doctest

comment:6 Changed 3 years ago by
 Reviewers set to Frédéric Chapoton
ok (fill the Authors field, please, then you can set to positive)
comment:7 Changed 3 years ago by
 Status changed from needs_review to positive_review
comment:8 Changed 3 years ago by
Thanks.
comment:9 Changed 3 years ago by
 Branch changed from u/rws/25251 to 495e9f47ba235f743ddf62741a5bc12d9e8f0eb5
 Resolution set to fixed
 Status changed from positive_review to closed
Note: See
TracTickets for help on using
tickets.
See also #25252,.