You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We could try to reduce the number of .match() calls we are making by giving Rules to register certain types of nodes. This will eliminate the top level assert isinstance(node, <node type>) check from their side and made our refactor loop faster.
The text was updated successfully, but these errors were encountered:
Now thinking about it, it doesn't seem to yield much in terms of performance since isinstance check is done no matter what :/ I'll re-open this if I can find a better way to solve this problem.
We could try to reduce the number of
.match()
calls we are making by givingRule
s to register certain types of nodes. This will eliminate the top levelassert isinstance(node, <node type>)
check from their side and made our refactor loop faster.The text was updated successfully, but these errors were encountered: