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.

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

Get rid of ResourcePools dependency #6

Open
maxfreu opened this issue May 4, 2022 · 1 comment
Open

Get rid of ResourcePools dependency #6

maxfreu opened this issue May 4, 2022 · 1 comment

Comments

@maxfreu
Copy link

maxfreu commented May 4, 2022

That issue maybe comes from not knowing enough, but to me it looks like the resource pool stuff should be decoupled from a camera interface definition. It's not clear why an AbstractAcquiredImage should be some pooled dense array.

@stemann
Copy link
Owner

stemann commented May 5, 2022

Good to give it some thought.

Without having thought too much about it (since some years ago): The reason for the dependency is for integrating cameras into a (soft) real-time system - with low-latency requirements - such that one should expect that images returned from a camera is from some pool of shared images, and the user is expected to call release! to give the image back to the pool - or retain! if the user needs to hold onto the image.

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

No branches or pull requests

2 participants