Opened 7 years ago

Closed 7 years ago

#19931 closed enhancement (fixed)

exact_part for asymptotic expansions

Reported by: Benjamin Hackl Owned by:
Priority: major Milestone: sage-7.1
Component: asymptotic expansions Keywords:
Cc: Daniel Krenn, Clemens Heuberger Merged in:
Authors: Benjamin Hackl Reviewers: Clemens Heuberger
Report Upstream: N/A Work issues:
Branch: 7276172 (Commits, GitHub, GitLab) Commit: 72761726f089fbf1edc5797d8cd114a4da84815b
Dependencies: Stopgaps:

Status badges

Description (last modified by Benjamin Hackl)

For various reasons, having a method expansion.exact_part that returns the asymptotic expansion that consists of all ExactTerm-summands would be very useful.

This ticket provides such a method.

Change History (6)

comment:1 Changed 7 years ago by Benjamin Hackl

Authors: Benjamin Hackl
Branch: u/behackl/asy/exact_part
Commit: 8aa55e2fe0bd8c448658de35a4e384809fe00d42
Description: modified (diff)
Status: newneeds_review

New commits:

8aa55e2implement exact_part

comment:2 Changed 7 years ago by Clemens Heuberger

Status: needs_reviewneeds_work

I'd prefer to copy the expression and to delete all non-exact elements. In general, this should be more efficient than recreating a poset with almost all elements --- after all, in general, there will be more exact terms than error terms.

comment:3 Changed 7 years ago by git

Commit: 8aa55e2fe0bd8c448658de35a4e384809fe00d4272761726f089fbf1edc5797d8cd114a4da84815b

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

7276172delete terms from copy instead of building new poset

comment:4 in reply to:  2 Changed 7 years ago by Benjamin Hackl

Status: needs_workneeds_review

Replying to cheuberg:

I'd prefer to copy the expression and to delete all non-exact elements. In general, this should be more efficient than recreating a poset with almost all elements --- after all, in general, there will be more exact terms than error terms.

I agree, this strategy seems to be more efficient than building the entire expansion again. I've adapted the implementation.

comment:5 Changed 7 years ago by Clemens Heuberger

Reviewers: Clemens Heuberger
Status: needs_reviewpositive_review

Doctests pass according to patchbot, documentation builds, code now fine.

comment:6 Changed 7 years ago by Volker Braun

Branch: u/behackl/asy/exact_part72761726f089fbf1edc5797d8cd114a4da84815b
Resolution: fixed
Status: positive_reviewclosed
Note: See TracTickets for help on using tickets.