Skip to content

Added entity instances caching. #291

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 8 commits into from
Nov 26, 2019
Merged

Added entity instances caching. #291

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
bb03086
Added entity caching.
jordanbriere Nov 25, 2019
d8cef55
Made the cache lookup slightly faster.
jordanbriere Nov 25, 2019
285287f
Fixed a rare case where a base class initialisation would be skipped.
jordanbriere Nov 25, 2019
2aab39b
No need to wrap the classes that are not using the cache.
jordanbriere Nov 25, 2019
58fec31
Greatly improved the caching implementation.
jordanbriere Nov 25, 2019
d7cc213
Updated docstrings, etc.
jordanbriere Nov 25, 2019
d9f87dc
Moved the "cache" property to the Entity class so that it gets added …
jordanbriere Nov 25, 2019
52075fc
Moved the "cache" property back to the caching class so that it can b…
jordanbriere Nov 26, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
No need to wrap the classes that are not using the cache.
  • Loading branch information
@jordanbriere
jordanbriere committed Nov 25, 2019
commit 2aab39bad1457c94dfabec8a22e64df30f0e0104
7 changes: 5 additions & 2 deletions addons/source-python/packages/source-python/entities/_base.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -87,10 +87,13 @@ def __new__(metaclass, classname, bases, attributes, caching=True):
cls = super().__new__(
metaclass, classname, bases, attributes
)
# No need to override the instance creation/initialisation if we
# are not caching them
if not caching:
return cls

# New instances of this class will be cached in that dictionary
if caching:
metaclass.cache[cls] = {}
metaclass.cache[cls] = {}

def __init__(self, index):
# Does nothing, so we don't re-initialize cached instances, etc.
Expand Down