Skip to content

Cache get_decoder_class#1834

Merged
thewhaleking merged 8 commits intostagingfrom
feature/1833/cache-get-decoder-class
May 2, 2024
Merged

Cache get_decoder_class#1834
thewhaleking merged 8 commits intostagingfrom
feature/1833/cache-get-decoder-class

Conversation

@thewhaleking
Copy link
Contributor

Monkey patches in caching of scalecodec's get_decoder_class

Fixes #1833

@thewhaleking thewhaleking changed the base branch from master to staging May 1, 2024 20:57
@thewhaleking thewhaleking marked this pull request as ready for review May 2, 2024 17:44
@thewhaleking thewhaleking marked this pull request as draft May 2, 2024 17:51
…o cache hashable args, so we need to verify we're only sending strings to the cached function. This does add a bit of overhead (type checking), but is still much faster than without.
@thewhaleking thewhaleking marked this pull request as ready for review May 2, 2024 19:00
@thewhaleking thewhaleking merged commit 3b47287 into staging May 2, 2024
@thewhaleking thewhaleking deleted the feature/1833/cache-get-decoder-class branch May 2, 2024 20:37
@thealligatorking thealligatorking mentioned this pull request May 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Cache get_decoder_class

2 participants