Opened 7 years ago
Closed 7 years ago
#20134 closed defect (fixed)
Upgrade to pynac-0.6.4
Reported by: | rws | Owned by: | |
---|---|---|---|
Priority: | major | Milestone: | sage-7.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/pynac-0.6.4/pynac-0.6.4.tar.bz2
Change History (15)
comment:1 Changed 7 years ago by
Branch: | → u/rws/upgrade_to_pynac_0_6_3 |
---|
comment:2 Changed 7 years ago by
Authors: | → Ralf Stephan |
---|---|
Commit: | → 2fab819852116989ae3676d1a4ff61ecd4adc9b9 |
Description: | modified (diff) |
Status: | new → needs_review |
comment:3 Changed 7 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 7 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 7 years ago by
Branch: | u/rws/upgrade_to_pynac_0_6_3 → u/rws/20134-1 |
---|
comment:6 Changed 7 years ago by
Commit: | 2fab819852116989ae3676d1a4ff61ecd4adc9b9 → f061a28f3229cc85c96a6e9921ea63353c22a304 |
---|---|
Description: | modified (diff) |
Summary: | Upgrade to pynac-0.6.3 → Upgrade to pynac-0.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 for-pynac-0.6.3
|
1de52c3 | expand doctest change
|
075374f | add doctest
|
6af66d6 | add doctests for issue 144
|
1f61a24 | sech/csch changes
|
f061a28 | pynac-0.6.4 pkg version/chksum
|
comment:7 Changed 7 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 7 years ago by
Description: | modified (diff) |
---|
comment:9 Changed 7 years ago by
Status: | needs_review → 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 7 years ago by
Commit: | f061a28f3229cc85c96a6e9921ea63353c22a304 → 0791a6178c020a6f0d7bc388d95ea20564393995 |
---|
Branch pushed to git repo; I updated commit sha1. New commits:
0791a61 | Merge branch 'develop' into t/20134/20134-1
|
comment:12 Changed 7 years ago by
Commit: | 0791a6178c020a6f0d7bc388d95ea20564393995 → 9cbedc71e156b9758e1226f36d8659d7674677ec |
---|
Branch pushed to git repo; I updated commit sha1. New commits:
9cbedc7 | 20134: fix/add doctests
|
comment:13 Changed 7 years ago by
Milestone: | sage-7.1 → sage-7.2 |
---|---|
Status: | needs_work → needs_review |
comment:14 Changed 7 years ago by
Reviewers: | → Jeroen Demeyer |
---|---|
Status: | needs_review → positive_review |
comment:15 Changed 7 years ago by
Branch: | u/rws/20134-1 → 9cbedc71e156b9758e1226f36d8659d7674677ec |
---|---|
Resolution: | → fixed |
Status: | positive_review → closed |
New commits:
package version / checksum
coth/acoth BuiltinFunction --> GinacFunction and fixes
doctest fixes