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

'Cache' properties in Kiro plugin #45

Open
TimoDritschler opened this issue Aug 13, 2014 · 2 comments
Open

'Cache' properties in Kiro plugin #45

TimoDritschler opened this issue Aug 13, 2014 · 2 comments

Comments

@TimoDritschler
Copy link
Member

Since the Kiro plugin only forwards all properties from and to the TANGO world, it does not actually hold any property values itself.
However, 'recording' is dependent on several values such as frames-per-second, roi-width, roi-height, sensor-width and so on.
Therefore, the Kiro Plugin somehow needs to buffer these properties locally. But if the property gets changed on the TANGO side by someone other than the Kiro Plugin itself, then the plugin is currently unaware of these changes which might lead to inconsistencies and undefined behavior.
A possible approach to this problem would be to use TANGOS Event System to get notifications every time a property changes and react to those changes accordingly.

@matze
Copy link
Contributor

matze commented Aug 13, 2014

There might be some overlap with #13. But that's lingering around for almost a year now ;-)

@TimoDritschler
Copy link
Member Author

Ah, yeah, you are right!
I will take this into consideration. Thanks for noticing!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants