Opened 5 years ago
Closed 5 years ago
#20134 closed defect (fixed)
Upgrade to pynac0.6.4
Reported by:  rws  Owned by:  

Priority:  major  Milestone:  sage7.2 
Component:  packages: standard  Keywords:  
Cc:  Merged in:  
Authors:  Ralf Stephan  Reviewers:  Jeroen Demeyer 
Report Upstream:  N/A  Work issues:  
Branch:  9cbedc7 (Commits, GitHub, GitLab)  Commit:  9cbedc71e156b9758e1226f36d8659d7674677ec 
Dependencies:  Stopgaps: 
Description (last modified by )
Newest pynac does:
 fix
Re/Im(tanh)
(#20098) (R. Stephan)  fix positive+integer assumption conflicts (#20132) (A. Thakkar)
 from GiNaC: [bugfix] fix elusive bug in quo, rem,... (R. Kreckel)
 from GiNaC: [bugfix] fix sqrfree(poly) for zero polynomials in disguise. (R. Kreckel)
 sin/cos/tan of numeric: reduce mod
2*pi
(#20099) (B. Hackl)  more trigonometric simplifications (#20099) (B. Hackl, R. Stephan)
 always expand function arguments (B. Hackl)
 exp of numeric: reduce mod
2*pi*I
(#20099) (B. Hackl)  performance: speed up sin/cos/tan/coth/acoth evaluation (R. Stephan, A. Thakkar)
 performance: GiNaC functions for coth/sech/csch/acoth/asech/acsch (R. Stephan, A. Thakkar)
 AUTHORS updated
https://github.com/pynac/pynac/releases/download/pynac0.6.4/pynac0.6.4.tar.bz2
Change History (15)
comment:1 Changed 5 years ago by
 Branch set to u/rws/upgrade_to_pynac_0_6_3
comment:2 Changed 5 years ago by
 Commit set to 2fab819852116989ae3676d1a4ff61ecd4adc9b9
 Description modified (diff)
 Status changed from new to needs_review
comment:3 Changed 5 years ago by
The interface changes on this ticket are relatively straightforward, and I could easily review this. However, I'm not sure whether I should, as I've contributed to the pynac release. What do you think?
comment:4 Changed 5 years ago by
In principle I have already had a look at your changes, but I would like to see a third opinion as well.
comment:5 Changed 5 years ago by
 Branch changed from u/rws/upgrade_to_pynac_0_6_3 to u/rws/201341
comment:6 Changed 5 years ago by
 Commit changed from 2fab819852116989ae3676d1a4ff61ecd4adc9b9 to f061a28f3229cc85c96a6e9921ea63353c22a304
 Description modified (diff)
 Summary changed from Upgrade to pynac0.6.3 to Upgrade to pynac0.6.4
New commits:
fcb8d95  a42f798 still fails three in expression.pyx

bf73f41  adapt doctest to have a different arcsech() expression

65023a5  Merge branch 'develop' into forpynac0.6.3

1de52c3  expand doctest change

075374f  add doctest

6af66d6  add doctests for issue 144

1f61a24  sech/csch changes

f061a28  pynac0.6.4 pkg version/chksum

comment:7 Changed 5 years ago by
The whole point of this doctest is lost:
Here arccoth doesn't have 1 in its domain, so we just hold the expression: sage: elliptic_e(arccoth(1), x^2*e) elliptic_e(arccoth(1), x^2*e)
I would remove the sentence "Here arccoth doesn't have 1 in its domain, so we just hold the expression:"
comment:8 Changed 5 years ago by
 Description modified (diff)
comment:9 Changed 5 years ago by
 Status changed from needs_review to needs_work
Unless I'm missing something, this doctest looks gone:
sage: diff(asech(x), x) 1/((x + 1)*x*sqrt((x  1)/(x + 1)))
comment:11 Changed 5 years ago by
 Commit changed from f061a28f3229cc85c96a6e9921ea63353c22a304 to 0791a6178c020a6f0d7bc388d95ea20564393995
Branch pushed to git repo; I updated commit sha1. New commits:
0791a61  Merge branch 'develop' into t/20134/201341

comment:12 Changed 5 years ago by
 Commit changed from 0791a6178c020a6f0d7bc388d95ea20564393995 to 9cbedc71e156b9758e1226f36d8659d7674677ec
Branch pushed to git repo; I updated commit sha1. New commits:
9cbedc7  20134: fix/add doctests

comment:13 Changed 5 years ago by
 Milestone changed from sage7.1 to sage7.2
 Status changed from needs_work to needs_review
comment:14 Changed 5 years ago by
 Reviewers set to Jeroen Demeyer
 Status changed from needs_review to positive_review
comment:15 Changed 5 years ago by
 Branch changed from u/rws/201341 to 9cbedc71e156b9758e1226f36d8659d7674677ec
 Resolution set to fixed
 Status changed from positive_review to closed
New commits:
package version / checksum
coth/acoth BuiltinFunction > GinacFunction and fixes
doctest fixes