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

Lazy attribute fixes and improvements #5783

Closed
nthiery opened this issue Apr 14, 2009 · 7 comments
Closed

Lazy attribute fixes and improvements #5783

nthiery opened this issue Apr 14, 2009 · 7 comments
Assignees
@nthiery
Labels
p: major / 3 t: bug
Milestone
sage-4.0

Comments

@nthiery
Copy link
Contributor

nthiery commented Apr 14, 2009

  • fix infinite recursion bug
  • adds support for cpdefs methods
  • fix ReST doc (indentation, ::, ...)
  • fix introspection (hopefuly)
  • adds a hook for this in sage.misc.sageinspect.sage_getsourcelines

CC: @sagetrac-sage-combinat

Component: misc

Issue created by migration from https://trac.sagemath.org/ticket/5783
@nthiery nthiery assigned nthiery and unassigned sagetrac-cwitty May 18, 2009
@nthiery
Copy link
Contributor Author

nthiery commented May 19, 2009

Attachment: lazy_attributes-fixes-5783-final.patch.gz

@nthiery

This comment has been minimized.

Sign in to view
@nthiery nthiery changed the title Lazy attributes: fix infinite recursion bug + support for cpdefs methods Lazy attribute fixes and improvements May 19, 2009
@nthiery
Copy link
Contributor Author

nthiery commented May 19, 2009

comment:4

Updated patch:

  • fix ReST doc (indentation, ::, ...)
  • fix introspection
  • adds a hook for this in sage.misc.sageinspect.sage_getsourcelines

@nthiery

This comment has been minimized.

Sign in to view
@roed314
Copy link
Contributor

roed314 commented May 19, 2009

comment:5

Adds lots of good documentation, fixes some problems, though additional feature requests remain. Positive review.

@sagetrac-mabshoff
Copy link
Mannequin

sagetrac-mabshoff mannequin commented May 19, 2009

comment:6

David,

please remember to change the summary if you do reviews of tickets.

Michael

@sagetrac-mabshoff
Copy link
Mannequin

sagetrac-mabshoff mannequin commented May 19, 2009

comment:7

Merged in Sage 4.0.rc0.

Cheers,

Michael

@sagetrac-mabshoff sagetrac-mabshoff mannequin added this to the sage-4.0 milestone May 19, 2009
@sagetrac-mabshoff sagetrac-mabshoff mannequin closed this as completed May 19, 2009
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nthiery nthiery

Labels
p: major / 3 t: bug
Projects
None yet
Milestone
sage-4.0
Development

No branches or pull requests
2 participants
@roed314 @nthiery