#20859: Simplify the logic handling the EvaluationMethods mixin class for
Expression
-------------------------------------+-------------------------------------
Reporter: jdemeyer | Owner:
Type: enhancement | Status: new
Priority: major | Milestone: sage-7.3
Component: symbolics | Resolution:
Keywords: | Merged in:
Authors: Nicolas M. Thiéry | Reviewers:
Report Upstream: N/A | Work issues:
Branch: | Commit:
u/nthiery/optimize_method_lookup_from_the_categories_for_instances_of_cython_classes|
5619a7d0f032f4273069166e6babf938b8c3f40a
Dependencies: #20825, #20686 | Stopgaps:
-------------------------------------+-------------------------------------
Comment (by nthiery):
Replying to [comment:10 jdemeyer]:
> Replying to [comment:7 nthiery]:
> > Furthermore, for our XXXMethods classes, it does not matter whether
> > they are old style or new style classes
>
> It might matter in more places than you think. There will be some
porting effort needed to transition from old-style classes to new-style
classes (some issues came up in #20686). It would be better to do this now
to avoid unexpected issues with Python 3.
>
> For this reason, I am very against this change:
> {{{
> #!diff
> - class EvaluationMethods(object):
> + class EvaluationMethods:
> }}}
I already did tests with inheriting from object in some XXXMethods,
and the category framework kept working the exact same way. I am
therefore convinced there won't be anything to change for that
specific aspect for Python 3.
Furthermore, the more consistent things will be across the library,
the easier the porting will be.
Cheers,
Nicolas
--
Ticket URL:
<https://u3351942.ct.sendgrid.net/wf/click?upn=aTs-2BwUSKwq20U-2FVxpZle9V7rZPHNFdCZn9IqCcBPbg6Wx7VTUgJoegiKQ3QL4-2BXvu5BR6r8bx-2BeBPPrDDMoydQ-3D-3D_gXX0YPkjCa6kfMda2NWALp0MQ-2FOvmULrxPdhd2nGLCadPM06G56QPRHqNsi-2BCtYoXhCE-2F-2F3aZ0WCYmlqpBvCDJHSasMO-2B-2F60WzhvdbbqmTwujAegqh8mXKClWszUwghaZV-2BtlGi1EJDDw4BDINSIufvQx9-2BitOS8bcgTVFYBxnCPb0TUjyk-2FpcSMObM4EJplWkCEKC16bAEETcSznFy6PqJ5VnnQe0P5b-2FlItaccI1Y-3D>
Sage
<https://u3351942.ct.sendgrid.net/wf/click?upn=jm4cvpnHFskDUI5PLE4HCGcqpDNkng8vhBVTwprYF6Q-3D_gXX0YPkjCa6kfMda2NWALp0MQ-2FOvmULrxPdhd2nGLCadPM06G56QPRHqNsi-2BCtYoXhCE-2F-2F3aZ0WCYmlqpBvCDKDz10yakzs4PVLmgfCHc5SwBqhSYrVRkJanixDQoVklwkqw0KnxRv0CMrHqIGViumib6hPAyGRJ5RQBo6YTESvJkO-2FTbeE8pUmwhheZMWjhR2noxYuqcmmew110Bs3xTAUHyA2V1HEOhLeDk8YGCas-3D>
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 unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at https://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.
