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

Implement hysteresis for inference computation #41

Open
peddybeats opened this issue Apr 5, 2020 · 1 comment
Open

Implement hysteresis for inference computation #41

peddybeats opened this issue Apr 5, 2020 · 1 comment

Comments

@peddybeats
Copy link
Owner

  • We should add a inference state variable
  • Each inference made, should update that variable by max(epsilon, | new - old | ) in the correct direction
  • Be disciplined about how we return 0s vs errors vs undefineds ex
@Berkmann18
Copy link
Collaborator

Berkmann18 commented May 1, 2020

One thing to bear in mind is that the inference text is being repainted every single frame (even when the inference state is set to "Off").
This can be observed by going on a browser's devtool by either:

  • enabling paint flashing and then one should see at least 2 green boxes (representing repainted areas on the page)
  • navigating to the container Element such that the camera view and inference text elements are shown, what could be observed is a constant change of at least 2-3 HTML elements.

@misterpeddy I suppose (judging by the fact this issue is "In progress") you are working on this?

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