Opened 8 years ago
Closed 8 years ago
#17403 closed enhancement (fixed)
implement PowerSeries.inversion() as wrapper for ^1
Reported by:  Ralf Stephan  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 8 years ago by
Branch:  → u/rws/implement_powerseries_inversion___as_wrapper_for___1 

comment:2 Changed 8 years ago by
Authors:  → Ralf Stephan 

Commit:  → 6107bd2a8470cd984c06ef23a88d7f3f6311357b 
Status:  new → needs_review 
comment:3 Changed 8 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 8 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 Changed 8 years ago by
comment:6 Changed 8 years ago by
Commit:  6107bd2a8470cd984c06ef23a88d7f3f6311357b → 94b3ca373a6800e1e5600c955f86f012b1dd0e12 

Branch pushed to git repo; I updated commit sha1. New commits:
94b3ca3  17403: rename to inverse

comment:7 Changed 8 years ago by
Reviewers:  → Bruno Grenet 

Status:  needs_review → positive_review 
Looks good to me!
comment:8 Changed 8 years ago by
Branch:  u/rws/implement_powerseries_inversion___as_wrapper_for___1 → 94b3ca373a6800e1e5600c955f86f012b1dd0e12 

Resolution:  → fixed 
Status:  positive_review → closed 
New commits:
17403: convenience wrapper inversion() for power and Laurent series