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

deprecated_function_alias for Cython classes #11320

Closed
vbraun opened this issue May 10, 2011 · 4 comments
Closed

deprecated_function_alias for Cython classes #11320

vbraun opened this issue May 10, 2011 · 4 comments
Assignees
@jasongrout
Labels
p: major / 3 t: enhancement
Milestone
sage-4.7.1

Comments

@vbraun
Copy link
Member

vbraun commented May 10, 2011

Currently you can't use deprecated_function_alias with Cython classes because they do not have a __dict__ attribute. This patch implements a try/except block as suggested by Florent Hivert in https://groups.google.com/d/topic/sage-devel/DayMPddlOds/discussion

CC: @hivert @burcin

Component: misc

Author: Volker Braun

Reviewer: Mike Hansen

Merged: sage-4.7.1.alpha3

Issue created by migration from https://trac.sagemath.org/ticket/11320
@vbraun vbraun added this to the sage-4.7.1 milestone May 10, 2011
@vbraun
Copy link
Member Author

vbraun commented May 10, 2011

Attachment: trac_11320_deprecated_function_alias_in_cython.patch.gz

Initial patch

@mwhansen
Copy link
Contributor

Reviewer: Mike Hansen

@mwhansen
Copy link
Contributor

comment:2

Looks good to me.

@jdemeyer
Copy link
Contributor

jdemeyer commented Jun 8, 2011

Merged: sage-4.7.1.alpha3

@jdemeyer jdemeyer closed this as completed Jun 8, 2011
@hivert hivert mentioned this issue Jul 5, 2011
Closed
This was referenced Jun 15, 2011
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jasongrout jasongrout

Labels
p: major / 3 t: enhancement
Projects
None yet
Milestone
sage-4.7.1
Development

No branches or pull requests
4 participants
@jasongrout @mwhansen @vbraun @jdemeyer