-
Notifications
You must be signed in to change notification settings - Fork 22
Add use-wifi param #97
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
Conversation
Justin, Yes, Please let me know when i can try it. |
@rebtop you can download the files from here: https://github.com/justmobilize/Adafruit_CircuitPython_MatrixPortal/tree/add-use-wifi-param and replace what you have in Then when you create it add matrixportal = MatrixPortal(
# what ever you have currently
use_wifi=False,
) |
I will try it tomorrowSent from my iPhoneOn May 30, 2024, at 9:01 PM, Justin Myers ***@***.***> wrote:
@rebtop you can download the files from here: https://github.com/justmobilize/Adafruit_CircuitPython_MatrixPortal/tree/add-use-wifi-param and replace what you have in lib/adafruit_matrixportal with what's in the adafruit_matrixportal from the download.
Then when you create it add use_wifi:
matrixportal = MatrixPortal(
# what ever you have currently
use_wifi=False,
)
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: ***@***.***>
|
Justin |
I am assuming you want this inserted in the code itself. I will try that. |
That change takes the incompatibility out of play. Now i will work on getting Ble to work. |
@rebtop awesome! |
K Ble is working with MatrixPortal code. |
Awesome. If you are willing to share code, I can use it as a basis to build in Bluetooth support into MatrixPortal |
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.
Thanks @justmobilize for the PR and @rebtop for testing.
Updating https://github.com/adafruit/Adafruit_CircuitPython_MatrixPortal to 3.2.0 from 3.1.13: > Merge pull request adafruit/Adafruit_CircuitPython_MatrixPortal#97 from justmobilize/add-use-wifi-param Updating https://github.com/adafruit/Adafruit_CircuitPython_Bundle/circuitpython_library_list.md to NA from NA: > Updated download stats for the libraries
Justin
Let me clean it up some and I will send it to you.
From Bob Black. Sent from my iPad
On Jun 1, 2024, at 10:53 PM, Dan Halbert ***@***.***> wrote:
Merged #97<#97> into main.
—
Reply to this email directly, view it on GitHub<#97 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AOKVYXOZR2HGBZUYMR7T2M3ZFKCLZAVCNFSM6AAAAABIRVLDPSVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJTGAYTANRTGAZDGNA>.
You are receiving this because you were mentioned.Message ID: ***@***.***>
|
I am cleaning up the code and will share it shortly |
Justin |
This fixes #85 by allowing the user to specify they don't want to use WiFi. A later PR could happen that enables Bluetooth.