Skip to content

mbedOS crashes trying to run examples from Tensorflow 2.1.0-alpha #119

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

Closed
mbanzi opened this issue Jan 13, 2021 · 3 comments · Fixed by arduino/arduino-cli#1139
Closed
Assignees

Comments

@mbanzi
Copy link
Member

mbanzi commented Jan 13, 2021

Trying to run examples from the latest version of the TensorFlow library crashed mbedOS

facchinm added a commit to facchinm/arduino-cli that referenced this issue Jan 14, 2021
Precompiled bits of a library should not depend on any link time dependency (we cannot assure ABI stability).
Fixes arduino/ArduinoCore-mbed#119
@mbanzi
Copy link
Member Author

mbanzi commented Jan 26, 2021

update. I tried the "hello_world" example on Portenta M7 Core. The code compiles ok but the crashes when run. TF Lite 2.4.0 Alpha , mbedOS core 1.3.1. The red LED flashes 4 long blinks and 4 short blinks

@facchinm
Copy link
Member

@mbanzi the crash on Portenta is due to the example trying to analogWrite() on LED_BUILTIN which is not PWM capable. mbed core crashes in this case instead than falling back to digitalWrite() due to an assert in mbed itself.
I'm wondering which is the best way to proceed, if "masking" the assert or changing the example for Portenta

@mbanzi
Copy link
Member Author

mbanzi commented Jan 26, 2021

Thanks for pointing that out. :) In the other Arduino cores trying to PWM a non PWM pin we just fall back to digitalWrite
is this something we can/want to do with Portenta as well? We can always modify the example adding some ifdef for Portenta

silvanocerza pushed a commit to facchinm/arduino-cli that referenced this issue Jan 26, 2021
Precompiled bits of a library should not depend on any link time dependency (we cannot assure ABI stability).
Fixes arduino/ArduinoCore-mbed#119
silvanocerza added a commit to arduino/arduino-cli that referenced this issue Jan 26, 2021
* Skip dependency detection if library is fully precompiled

Precompiled bits of a library should not depend on any link time dependency (we cannot assure ABI stability).
Fixes arduino/ArduinoCore-mbed#119

* Add output when skipping deps detection for precompiled libs

Co-authored-by: Silvano Cerza <[email protected]>
silvanocerza added a commit to arduino/arduino-cli that referenced this issue Jan 27, 2021
* Skip dependency detection if library is fully precompiled

Precompiled bits of a library should not depend on any link time dependency (we cannot assure ABI stability).
Fixes arduino/ArduinoCore-mbed#119

* Add output when skipping deps detection for precompiled libs

Co-authored-by: Silvano Cerza <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants