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

devicemotion #37

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Conversation

tinybeachthor
Copy link

No description provided.

@smarquez1
Copy link

@WhoMeNope is your branch working?

@tinybeachthor
Copy link
Author

@smarquez1 just checked, yep it works.

@jamiechong
Copy link

This PR works, but it seems to be way too sensitive. Any suggestions on how to tame it down a bit? I'm thinking we'd want to process fewer devicemotion events.

@tinybeachthor
Copy link
Author

I was thinking about adding some threshold values, some filtering (so the changes are more gradual) and limit the number of events.
That should do it. I will try it tmrw.

@tinybeachthor
Copy link
Author

@jamiechong try it now, it should be much more pleasant

@Sahero
Copy link

Sahero commented Jun 18, 2018

@WhoMeNope The image is shaking while using the mobile device. The sensitivity is still high. Any chance it can be improved?

@justrealmilk
Copy link

@Sahero 👍

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

Successfully merging this pull request may close these issues.

None yet

5 participants