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

Recognition should be on a seperate thread #5

Closed
sr229 opened this issue Aug 6, 2020 · 2 comments · Fixed by #24
Closed

Recognition should be on a seperate thread #5

sr229 opened this issue Aug 6, 2020 · 2 comments · Fixed by #24
Assignees
Labels
bug Something isn't working

Comments

@sr229
Copy link
Member

sr229 commented Aug 6, 2020

Since Face recog inferenceing may block the main thread, it's best we implement it is in a seperate thread - may be it a worker or using green threading.

@sr229
Copy link
Member Author

sr229 commented Aug 18, 2020

@LeNitrous pointed out using the Update() method would allow us to do this.

@sr229 sr229 self-assigned this Aug 18, 2020
@sr229 sr229 added the bug Something isn't working label Sep 3, 2020
@LeNitrous
Copy link
Member

LeNitrous commented Sep 3, 2020

Correction, doing such a heavy task in Component.Update() would be taxing for the scene graph causing slowdowns in the scene itself. Consider using System.Threading.Tasks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants