edgeimpulse / firmware-arduino-nano-33-ble-sense

Edge Impulse firmware for the Arduino Nano 33 BLE Sense development board
Apache License 2.0
61 stars 20 forks source link

Support request for Nano 33 BLE Sense Rev2 #10

Closed ShawnHymel closed 11 months ago

ShawnHymel commented 1 year ago

I don't have a Rev2 yet, so I don't know what the issue might be. However, we are seeing a user request support for the Rev2 board: https://forum.edgeimpulse.com/t/arduino-nano-33-ble-sense-rev2-support/6234.

The Arduino Nano 33 firmware doesn’t seem to work properly with the most recent Rev 2 board. The firmware uploads, and the microphone works, but the inertial sensors don’t. I didn’t try the environmental sensors. The inertial sensors worked on the Rev 1 board.

tahunus commented 1 year ago

I don't have a Rev2 yet, so I don't know what the issue might be. However, we are seeing a user request support for the Rev2 board: https://forum.edgeimpulse.com/t/arduino-nano-33-ble-sense-rev2-support/6234.

Rev2 uses different sensors: Arduino changed all STMicro-based sensors from Rev1:

Replacement of IMU from LSM9DS1 (9 axis) for a combination of two IMUs (BMI270 - 6 axis IMU and BMM150 - 3 axis IMU). Replacement of temperature and humidity sensor from HTS221 for HS3003. Replacement of microphone from MP34DT05 to MP34DT06JTR

All Arduino sketches I've tested (in Arduino IDE 2) work simply by updating the #include library statements with the new IMU and Temp/Humidity libraries. Microphone doesn't need a new library.

New IMU Library: https://github.com/arduino-libraries/Arduino_BMI270_BMM150 New Temp & Hum library: https://github.com/arduino-libraries/Arduino_HS300x

The question is if this approach works with Edge Impulse's firmware: https://github.com/edgeimpulse/firmware-arduino-nano-33-ble-sense. It would seem changes in arduino-build.sh, ei_environmentsensor.cpp and ei_lsm9ds1.h would fix this?

kbrunik commented 1 year ago

I'm not sure if this is the same issue but when I use the new IMUs (BMI270 - 6 axis IMU and BMM150 - 3 axis IMU) the output acceleration is several thousand Gs and the gyroscope and magnetometer readings seem off as well.

automatiek commented 11 months ago

Support for rev 2 added in commit 5b8e11f