Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

resolve all references of a of lazy import simultaneously #14357

Open
vbraun opened this issue Mar 26, 2013 · 3 comments
Open

resolve all references of a of lazy import simultaneously #14357

vbraun opened this issue Mar 26, 2013 · 3 comments
Labels
p: major / 3 t: enhancement t: performance

Comments

@vbraun
Copy link
Member

vbraun commented Mar 26, 2013

If you import a lazy import defined elsewhere and then use it, the original lazy import is not replaced. This ticket fixes this.

Depends on #14187

Component: performance

Issue created by migration from https://trac.sagemath.org/ticket/14357
@vbraun vbraun added this to the sage-5.11 milestone Mar 26, 2013
@vbraun
Copy link
Member Author

vbraun commented Mar 26, 2013

Dependencies: #14187

@vbraun
Copy link
Member Author

vbraun commented Mar 26, 2013

Attachment: trac_14357_lazy_everywhere.patch.gz

Initial patch

@jdemeyer jdemeyer modified the milestones: sage-5.11, sage-5.12 Aug 13, 2013
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.1, sage-6.2 Jan 30, 2014
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.2, sage-6.3 May 6, 2014
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.3, sage-6.4 Aug 10, 2014
@jdemeyer
Copy link

jdemeyer commented Apr 6, 2017

comment:6

I'm not really convinced that there is a problem here.

Calling gc.get_referrers() seems like a very heavy machinery to fix a non-issue.

@mkoeppe mkoeppe removed this from the sage-6.4 milestone Dec 29, 2022
@vbraun vbraun mentioned this issue Mar 26, 2013
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
p: major / 3 t: enhancement t: performance
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@vbraun @jdemeyer @mkoeppe