#9265: Remove `CombinatorialClass` from sage.combinat.tableau
---------------------------------+------------------------------------------
Reporter: jbandlow | Owner: sage-combinat
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-5.3
Component: combinatorics | Resolution:
Keywords: | Work issues:
Report Upstream: N/A | Reviewers:
Authors: Jason Bandlow | Merged in:
Dependencies: | Stopgaps:
---------------------------------+------------------------------------------
Comment (by andrew.mathas):
Thanks Anne!
This one confused no end:) It seems that sage keeps pickles of old objects
and then checks that new code is still able to unpickle the saved pickles.
One way to fix this error would be be to make a new pickle jar which would
remove the references to these renamed classes and presumably solve this
problem. Another option would be to include deprecation warnings for all
of the old class names, which would probably lead to doctest warnings
instead of failures when this test is run(?) -- I am not sure but would
this lead to deprecation warnings during the unpickling meaning that these
tests would still fail?
I am happy to deprecate all of the old class name if you like, although it
does seem a little strange to deprecate classes that were never officially
part of sage. Other the other hand, the patch has been around for a while
so people may be using the old names in their own code, so it might be
worth doing because of this.
I am happy to refer to your expertise as to what is the best course of
action. Please advise.
Andrew
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/9265#comment:24>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en.
