Opened 6 years ago
Closed 6 years ago
#17403 closed enhancement (fixed)
implement PowerSeries.inversion() as wrapper for ^1
Reported by:  rws  Owned by:  

Priority:  major  Milestone:  sage6.5 
Component:  commutative algebra  Keywords:  inverse, series 
Cc:  Merged in:  
Authors:  Ralf Stephan  Reviewers:  Bruno Grenet 
Report Upstream:  N/A  Work issues:  
Branch:  94b3ca3 (Commits, GitHub, GitLab)  Commit:  94b3ca373a6800e1e5600c955f86f012b1dd0e12 
Dependencies:  Stopgaps: 
Description
People try to TABdiscover a method for inversion of PowerSeries
but can't find it (http://ask.sagemath.org/question/24904/howtocomputethemultiplicativeinverseofaseries/), so we need a wrapper for ^1
, named analogously to PowerSeries.reversion()
.
The same applies to LaurentSeries
Change History (8)
comment:1 Changed 6 years ago by
 Branch set to u/rws/implement_powerseries_inversion___as_wrapper_for___1
comment:2 Changed 6 years ago by
 Commit set to 6107bd2a8470cd984c06ef23a88d7f3f6311357b
 Status changed from new to needs_review
comment:3 Changed 6 years ago by
The name inverse()
instead of inversion()
would be more appropriate to my mind, but actually I would say the same for reverse()
rather than reversion()
. This is more consistent with other names in Sage, such as derivative()
(and not derivation()
) or truncate()
(and not truncation()
).
comment:4 followup: ↓ 5 Changed 6 years ago by
I actually agree. If you open a ticket for reverse
then I'll make this one dependent on it and do the development/deprecation work.
comment:5 in reply to: ↑ 4 Changed 6 years ago by
comment:6 Changed 6 years ago by
 Commit changed from 6107bd2a8470cd984c06ef23a88d7f3f6311357b to 94b3ca373a6800e1e5600c955f86f012b1dd0e12
Branch pushed to git repo; I updated commit sha1. New commits:
94b3ca3  17403: rename to inverse

comment:7 Changed 6 years ago by
 Reviewers set to Bruno Grenet
 Status changed from needs_review to positive_review
Looks good to me!
comment:8 Changed 6 years ago by
 Branch changed from u/rws/implement_powerseries_inversion___as_wrapper_for___1 to 94b3ca373a6800e1e5600c955f86f012b1dd0e12
 Resolution set to fixed
 Status changed from positive_review to closed
New commits:
17403: convenience wrapper inversion() for power and Laurent series