-
-
Notifications
You must be signed in to change notification settings - Fork 567
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.
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
Bindable classes #12953
Comments
comment:1
For the record: all tests passed for me with 5.0.rc0 and the sage-combinat queue applied up to this patch. |
Reviewer: Franco Saliola |
This comment has been minimized.
This comment has been minimized.
Changed keywords from none to days38 |
comment:2
Attachment: trac_12953_bindable_class-nt.patch.gz On behalf of Franco: Comment: A variant would be to implement this feature using a class decorator. Practice will tell if we would need that variant. One advantage (or maybe sometimes an inconvenient?) of doing it by inheritance as implemented here is that all subclasses benefit from it, like, e.g., for UniqueRepresentation. Positive review! |
Merged: sage-5.1.beta1 |
From the documentation:
This feature will be extensively used for implementing SetsWithRealizations; see e.g. #12959 and the upcoming NCSF #8899 patch
CC: @sagetrac-sage-combinat
Component: misc
Keywords: days38
Author: Nicolas M. Thiéry
Reviewer: Franco Saliola
Merged: sage-5.1.beta1
Issue created by migration from https://trac.sagemath.org/ticket/12953
The text was updated successfully, but these errors were encountered: