Skip to content
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

Jump to bottom

Device Caching #29

Closed
teddyrendahl opened this issue Dec 21, 2017 · 0 comments · Fixed by #45
Closed

Device Caching #29

teddyrendahl opened this issue Dec 21, 2017 · 0 comments · Fixed by #45
Labels
enhancement Medium Priority

Comments

@teddyrendahl
Copy link
Contributor

Expected Behavior

The device loading should keep track of which devices it has loaded so far. This way we do not attempt to load the same device twice.

Current Behavior

No caching done.

Possible Solution

Have a happi.cache that has a mapping of loaded devices to containers. This way it is possible to go find the container information for a prior device. Also if a device is requested twice we just return a reference to the first one.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement Medium Priority
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

ENH: Cache
1 participant
@teddyrendahl