-
Notifications
You must be signed in to change notification settings - Fork 238
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
Add More Motion #732
Add More Motion #732
Conversation
call it motion plus, its just too easy |
YESSSS MOTION+ I WAS WAITING FOR THIS |
(distance between (x1) (y1) and (x2) (y2)) |
This comment was marked as spam.
This comment was marked as spam.
@NexusKitten are you ok lol? |
please keep the pull request names reasonable if you want them to be looked at. |
This comment was marked as spam.
This comment was marked as spam.
It should just be called Motion Plus to match the other category expansion extensions |
This comment was marked as spam.
This comment was marked as spam.
I made some changes
the last 2 are significant breaking changes, so let me know if you need the old behavior back |
"touching" as a concept is a sensing thing. Wouldn't it make more sense within sensing+? "Motion" describes movement, and the reporters in the motion category typically get an axis of movement, not a condition. Otherwise, distance would've been in motion. |
More Motion is an extension to add several utility blocks pertaining to the "Motion" category, such as blocks for distance, stepping, and checking position. Although it's blocks are somewhat limited as of currently, I wanted to set the groundwork for a motion extension since we don't currently have one yet.