Opened 8 years ago

Closed 8 years ago

#5449 closed defect (fixed)

[with patch, positive review] Implements a variant of @cached_method with cache stored in the parent

Reported by: nthiery Owned by: nthiery
Priority: major Milestone: sage-4.0
Component: misc Keywords: cached_method, cache
Cc: sage-combinat Merged in:
Authors: Reviewers:
Report Upstream: Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description (last modified by nthiery)

When the elements of a parent do not have unique representation, it can be desirable to store the cache of (some of) the methods in the parent rather than in the element.

This patches implements a variant @cached_in_parent_method of the decorator @cached_method which does just this.

Attachments (1)

cached_in_parent_method-5449-submitted.patch (4.2 KB) - added by nthiery 8 years ago.

Download all attachments as: .zip

Change History (6)

comment:1 Changed 8 years ago by nthiery

  • Cc sage-combinat added

Changed 8 years ago by nthiery

comment:2 Changed 8 years ago by nthiery

  • Description modified (diff)
  • Keywords cached_method cache added
  • Milestone changed from sage-combinat to sage-4.0
  • Summary changed from Implement @cache_method with cache stored in the parent to [with patch, needs review] Implements a variant of @cached_method with cache stored in the parent

comment:3 Changed 8 years ago by roed

  • Summary changed from [with patch, needs review] Implements a variant of @cached_method with cache stored in the parent to [with patch, positive review] Implements a variant of @cached_method with cache stored in the parent

Passes doctests, good documentation.

comment:4 Changed 8 years ago by nthiery

  • Status changed from new to assigned

comment:5 Changed 8 years ago by mabshoff

  • Milestone changed from sage-4.0.1 to sage-4.0
  • Resolution set to fixed
  • Status changed from assigned to closed

Merged in Sage 4.0.rc1.

Cheers,

Michael

Note: See TracTickets for help on using tickets.