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

Firmware update #9

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

Conversation

error404notfound
Copy link

This is updating the example and library to included functionality in the firmware 1.5.4.132 update

  • layout: additional buttons for testing the light pattern and LRA mode, shuffled the items up to fit everything in.
  • neosensory blessed library: addition of gets and sets for LRA mode, motor thresholds and LEDs

Expected behaviour running the app:
connect to buzz process is unchanged. button response is set to be on.
when light pattern button selected: LED pattern thread is started and it runs through a rainbow of hues
When LRA mode button is pressed: it toggles between open and closed loop.
When physical buttons are pressed, the name of the pressed button is displayed where the CLI information was displayed previously.

Todo: switch from using the on read notify function to handle button presses to having a specific handler that is called from the library side.

@GreatScott
Copy link
Contributor

Sorry---I'm just getting to this. This is amazing work!!! Tested and the changes look mostly good! Only thing is I wasn't able to get the CLI/app to respond to button presses -- this is using the latest firmware updated through the official Neosensory app.

I saw briefly that the code should write to the CLI output -- wanted to verify this is indeed the case before taking a deeper dive, thanks!

@GreatScott
Copy link
Contributor

Oh, another suggestion: would be to peg the version of BLESSED in the build.gradle to a static/the latest release instead of the master-SNAPSHOT to avoid their future updates accidentally breaking the SDK.

…sting to the notification space, linked the blessed library as static
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.

2 participants