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

Adding vibration and gyro functionality #36

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

Conversation

BeanCounterTop
Copy link

Added vibration and gyro functionality.
Added BYTEPATTERNS class to constants.py to name, contain, and dynamically generate bytepatterns (e.g., b'\x01\x00') using named hex strings also stored in class (e.g. '0100')
Added support for queued writes to allow for multithreading (sending vibrations while reading gyro data, for example)
Changes assume python3 due to the use of string formatting.

This is an initial PR, I've not yet incorporated the new bytepattern usage into the existing functions for auth, etc. I have done this in my derivative project, but I wanted to give you the chance to review these changes in isolation before moving forward there to see if they fit with your design goals.

@satcar77
Copy link
Owner

I quickly went through your commits and found it impressive. Although I haven't run the code or tested it, your project looks promising. I would gladly accept the PR once you finish off the functions you intend to implement. Thank you for the contributions to the repo.

@Skypow2012
Copy link

@NateSchoolfield Hello, is this gyro worked?I find my miband4 can not write in 00000002-0000-3512-2118-0009af100700

@Skypow2012
Copy link

This is worked, I got the gyro data from my mi band 4, @NateSchoolfield thanks!

@eduardopp
Copy link

@Skypow2012 were you able to make it work? mine is stuck in "Starting gyro..." - no output comes out.
Also @BeanCounterTop could you please share the final version (if it exists)?
Thank you both :)

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.

4 participants