search

adafruit / Adafruit_CircuitPython_BLE

Bluetooth Low Energy (BLE) library for CircuitPython
MIT License
127 stars 58 forks source link

ble_lywsd03mmc_simpletest.py trigger Type error in BLE with 8.0.0-beta.0 #172

Closed dglaude closed 2 years ago

dglaude commented 2 years ago

I am not sure this is a bug with this library or something in the BLE library...

But ble_lywsd03mmc_simpletest.py running on:

Does trigger a Type error:

Auto-reload is on. Simply save files over USB to run them or enter REPL to disable.
code.py output:
]0;🐍BLE:Reconnecting | code.py | 8.0.0-beta.0\Scanning...
Traceback (most recent call last):
  File "code.py", line 18, in <module>
  File "adafruit_ble/__init__.py", line 282, in start_scan
  File "adafruit_ble/advertising/__init__.py", line 319, in matches
  File "adafruit_ble/advertising/__init__.py", line 329, in matches_prefixes
TypeError: extra keyword arguments given
]0;🐍BLE:Off | 329@adafruit_ble/advertising/__init_ TypeError | 8.0.0-beta.0\
Code done running.

Press any key to enter the REPL. Use CTRL-D to reload.

This is the line 18:

    for adv in ble.start_scan(Advertisement, timeout=5):
dglaude commented 2 years ago

Same error on an ItsyBitsy NRF. Here are the library I use as updated by circup:


adafruit_ble==8.3.7```
tekktrik commented 2 years ago

The all keyword must now be match_all in CircuitPython 8.0.0, transferring to Adafruit_CircuitPython_BLE!

tekktrik commented 2 years ago

I can tackle this!

dhalbert commented 2 years ago

Ref https://github.com/adafruit/circuitpython/pull/6730