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: sage-6.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:

Status badges

Description

People try to TAB-discover a method for inversion of PowerSeries but can't find it (http://ask.sagemath.org/question/24904/how-to-compute-the-multiplicative-inverse-of-a-series/), 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 Ralf Stephan

Branch: u/rws/implement_powerseries_inversion___as_wrapper_for___1

comment:2 Changed 8 years ago by Ralf Stephan

Authors: Ralf Stephan
Commit: 6107bd2a8470cd984c06ef23a88d7f3f6311357b
Status: newneeds_review

New commits:

6107bd217403: convenience wrapper inversion() for power and Laurent series

comment:3 Changed 8 years ago by Bruno Grenet

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 Changed 8 years ago by Ralf Stephan

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 8 years ago by Bruno Grenet

Replying to rws:

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.

This is now #17724.

comment:6 Changed 8 years ago by git

Commit: 6107bd2a8470cd984c06ef23a88d7f3f6311357b94b3ca373a6800e1e5600c955f86f012b1dd0e12

Branch pushed to git repo; I updated commit sha1. New commits:

94b3ca317403: rename to inverse

comment:7 Changed 8 years ago by Bruno Grenet

Reviewers: Bruno Grenet
Status: needs_reviewpositive_review

Looks good to me!

comment:8 Changed 8 years ago by Volker Braun

Branch: u/rws/implement_powerseries_inversion___as_wrapper_for___194b3ca373a6800e1e5600c955f86f012b1dd0e12
Resolution: fixed
Status: positive_reviewclosed
Note: See TracTickets for help on using tickets.