Opened 5 years ago

Closed 5 years ago

#17403 closed enhancement (fixed)

implement PowerSeries.inversion() as wrapper for ^-1

Reported by: rws 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) Commit: 94b3ca373a6800e1e5600c955f86f012b1dd0e12
Dependencies: Stopgaps:

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 5 years ago by rws

  • Branch set to u/rws/implement_powerseries_inversion___as_wrapper_for___1

comment:2 Changed 5 years ago by rws

  • Authors set to Ralf Stephan
  • Commit set to 6107bd2a8470cd984c06ef23a88d7f3f6311357b
  • Status changed from new to needs_review

New commits:

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

comment:3 Changed 5 years ago by bruno

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 follow-up: Changed 5 years ago by 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.

comment:5 in reply to: ↑ 4 Changed 5 years ago by bruno

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 5 years ago by git

  • Commit changed from 6107bd2a8470cd984c06ef23a88d7f3f6311357b to 94b3ca373a6800e1e5600c955f86f012b1dd0e12

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

94b3ca317403: rename to inverse

comment:7 Changed 5 years ago by bruno

  • Reviewers set to Bruno Grenet
  • Status changed from needs_review to positive_review

Looks good to me!

comment:8 Changed 5 years ago by vbraun

  • 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
Note: See TracTickets for help on using tickets.