singledispatchmethod significantly slower than singledispatch

[CaselIT]

BPO	40988
Nosy	@mental32, @CaselIT
PRs	gh-85160: Optimized singledispatchmethod access (noticeable improvement). #23213

^{Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.}

Show more details

GitHub fields:

assignee = None
closed_at = None
created_at = <Date 2020-06-15.20:14:13.251>
labels = ['3.8', 'library', 'performance']
title = 'singledispatchmethod significantly slower than singledispatch'
updated_at = <Date 2020-11-10.05:12:05.587>
user = 'https://github.com/CaselIT'

bugs.python.org fields:

activity = <Date 2020-11-10.05:12:05.587>
actor = 'mental'
assignee = 'none'
closed = False
closed_date = None
closer = None
components = ['Library (Lib)']
creation = <Date 2020-06-15.20:14:13.251>
creator = 'CaselIT'
dependencies = []
files = []
hgrepos = []
issue_num = 40988
keywords = ['patch']
message_count = 1.0
messages = ['371594']
nosy_count = 2.0
nosy_names = ['mental', 'CaselIT']
pr_nums = ['23213']
priority = 'normal'
resolution = None
stage = 'patch review'
status = 'open'
superseder = None
type = 'performance'
url = 'https://bugs.python.org/issue40988'
versions = ['Python 3.8']

Linked PRs

• gh-85160: improve performance of singledispatchmethod #106448
• gh-85160: improve performance of singledispatchmethod #107148
• gh-85160: Reduce memory usage of singledispatchmethod when owner instances cannot be weakref'd #107706
• gh-127750: Improve caching in singledispatchmethod #127751

[CaselIT]

The implementation of singledispatchmethod is significantly slower (~4x) than the normal singledispatch version

Using timeit to test this example case:

    from functools import singledispatch, singledispatchmethod
    import timeit

    class Test:
        @singledispatchmethod
        def go(self, item, arg):
            print('general')
        
        @go.register
        def _(self, item:int, arg):
            return item + arg

    @singledispatch
    def go(item, arg):
        print('general')

    @go.register
    def _(item:int, arg):
        return item + arg

print(timeit.timeit('t.go(1, 1)', globals={'t': Test()}))
print(timeit.timeit('go(1, 1)', globals={'go': go}))

Prints on my system.

3.118346
0.713173

Looking at the singledispatchmethod implementation I believe that most of the difference is because a new function is generated every time the method is called.

Maybe an implementation similar to cached_property could be used if the class has __dict__ attribute?
Trying this simple patch

diff --git a/Lib/functools.py b/Lib/functools.py
index 5cab497..e42f485 100644
--- a/Lib/functools.py
+++ b/Lib/functools.py
@@ -900,6 +900,7 @@ class singledispatchmethod:

            self.dispatcher = singledispatch(func)
            self.func = func
    +        self.attrname = None

        def register(self, cls, method=None):
            """generic_method.register(cls, func) -> func
    @@ -908,6 +909,10 @@ class singledispatchmethod:
            """
            return self.dispatcher.register(cls, func=method)

+    def __set_name__(self, owner, name):
+        if self.attrname is None:
+            self.attrname = name
+
    def __get__(self, obj, cls=None):
        def _method(*args, **kwargs):
            method = self.dispatcher.dispatch(args[0].__class__)
@@ -916,6 +921,7 @@ class singledispatchmethod:
        _method.__isabstractmethod__ = self.__isabstractmethod__
        _method.register = self.register
        update_wrapper(_method, self.func)
+        obj.__dict__[self.attrname] = _method
        return _method

    @property

improves the performance noticeably

0.9720976
0.7269078

[iritkatriel]

I see a similar time difference on main.