Opened 7 years ago

Last modified 3 years ago

#15936 needs_info enhancement

Improve the memory footprint of cached functions and methods

Reported by: hivert Owned by:
Priority: major Milestone: sage-6.4
Component: misc Keywords: cachedfunction memory
Cc: SimonKing, nthiery Merged in:
Authors: Reviewers:
Report Upstream: N/A Work issues:
Branch: u/hivert/ticket/15936 (Commits, GitHub, GitLab) Commit: fd17ee11b7ce2d249674681226f33e6dc3214703
Dependencies: 15931 Stopgaps:

Status badges

Description

Some experiment shows that this is possible to reduce the size of all cached function by changing the seldom attribute __module__ to a property. This reduce in particular the size of the CachedMethodCallerNoArgs which is used in a lot of places. One probably can do even better...

Change History (9)

comment:1 Changed 7 years ago by hivert

  • Branch set to u/hivert/ticket/15936
  • Created changed from 03/13/14 16:35:41 to 03/13/14 16:35:41
  • Modified changed from 03/13/14 16:35:41 to 03/13/14 16:35:41

comment:2 Changed 7 years ago by hivert

  • Cc SimonKing nthiery added
  • Commit set to ce764be21076acd827a571e50e61bb69624c8515
  • Dependencies set to 15931
  • Keywords cachedfunction memory added

New commits:

e8fe5ebTrac 15931: implement a proper hash function for (combinatorial) free module elements
ce764beexperiment

comment:3 Changed 7 years ago by git

  • Commit changed from ce764be21076acd827a571e50e61bb69624c8515 to fd17ee11b7ce2d249674681226f33e6dc3214703

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

fd17ee1Added missing self.

comment:4 Changed 7 years ago by vbraun_spam

  • Milestone changed from sage-6.2 to sage-6.3

comment:5 Changed 7 years ago by vbraun_spam

  • Milestone changed from sage-6.3 to sage-6.4

comment:6 Changed 6 years ago by SimonKing

Ping to myself (I should review it at some point...)

comment:7 Changed 6 years ago by SimonKing

Oops! I just see that it is not set to needs_review. Should it be?

comment:8 Changed 4 years ago by SimonKing

Again ping to the author: Is this ticket supposed to be reviewed?

comment:9 Changed 3 years ago by SimonKing

  • Status changed from new to needs_info

Some questions to Florent:

  • Is there evidence that the memory footprint of cached methods currently is a problem?
  • How would one measure that footprint?
  • Do I understand correctly that the current branch won't merge?

Note: See TracTickets for help on using tickets.