-
Notifications
You must be signed in to change notification settings - Fork 1
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
SNS - Optical Flow Sensor (HYPE-33) #48
base: main
Are you sure you want to change the base?
Conversation
7778b9e
to
ac0ebdc
Compare
ac0ebdc
to
21782d0
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor things. @prakhars0206 iirc, the optical flow sensor was not soldered properly, can you confirm if that has been fixed?
Ideally you want to get to to the stage of testing the code asap with the actual sensor.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
has this been tested with the sensor yet?
Things to try next:
|
Tested and working 20/20/25 :) Main problem I think was the CS line not being used. We had additional logic errors and MSB/LSB mixup which were resolved. Supports:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
doubling the number of comments lmao
This implements the exact logic defined in:
https://github.com/pimoroni/pmw3901-python/blob/main/pmw3901/__init__.py
WIP
Also closes HYPE-57.