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

Change MPU6050 motion detection to act on ANY axis, not ALL axis. #4029

Closed
TD-er opened this issue May 2, 2022 · 0 comments · Fixed by #4042
Closed

Change MPU6050 motion detection to act on ANY axis, not ALL axis. #4029

TD-er opened this issue May 2, 2022 · 0 comments · Fixed by #4042
Labels
Category: Plugin Related to supported sensors Type: Enhancement Improve something already present

Comments

@TD-er
Copy link
Member

TD-er commented May 2, 2022

At the moment, MPU6050 motion detection works in such a way that the statement that motion has occurred is based on the finding that activation has occurred in ALL enabled axes.
In the case of a washing machine, it turns out that it is better to check that the stimulation occurred in ANY axis.
I wrote a script to implement this using rules. It works much better than the original motion detection included in the driver.
My suggestion is that there should be, for example, a checkbox toggling between ANY/ALL axes.
Below is a proposition of the driver part _P045_MPU6050.ino for a function that executes ANY instead of ALL axes.

See forum thread

@TD-er TD-er added Type: Enhancement Improve something already present Category: Plugin Related to supported sensors labels May 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Category: Plugin Related to supported sensors Type: Enhancement Improve something already present
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant