search

raspberrypi / pico-tflmicro

Pico TensorFlow Lite Port
641 stars 98 forks source link

Updated CMake for latest Pico-SDK, including Pico2 support (issue #18) #19

Open sean-lawless opened 1 month ago

sean-lawless commented 1 month ago

... also includes hello_world fix (issue #17). Upgraded C++ compiler to 2017. NOTE: Four (4) tests are now skipped due to missing C++ linker dependencies - if merged please open an issue for these.

lurch commented 1 month ago

NOTE: Four (4) tests are now skipped due to missing C++ linker dependencies

As GitHub isn't doing a very good job of rendering the diff here, it might be worth mentioning on this PR which of the tests are now skipped?

sean-lawless commented 1 month ago

NOTE: Four (4) tests are now skipped due to missing C++ linker dependencies

As GitHub isn't doing a very good job of rendering the diff here, it might be worth mentioning on this PR which of the tests are now skipped?

Apologies, it looks like I introduced a line ending issue with the CMakeLists.txt (thank you Windows)...

The commented tests are:

add_subdirectory("tests/flatbuffer_utils_test")

add_subdirectory("tests/kernels_elementwise_test")

add_subdirectory("tests/kernels_l2_pool_2d_test")

add_subdirectory("tests/kernels_l2norm_test")

The errors are link related, For example:

[flatbuffer_utils_test]: undefined reference to `std::__throw_length_error(char const*)
...
[kernels_l2norm_test]: undefined reference to `sqrtf'
sean-lawless commented 1 month ago

I fixed the line endings so CMakeLists.txt shows changes.