Co-authored-by: NAdvait Jain <advaitjain@users.noreply.github.com>

Co-authored-by: NDmitry Zakharov <47519863+dzakhar@users.noreply.github.com>

There isn't any need to have a special define for the target. In fact,
having such a define can create trouble when the TARGET and
OPTIMIZED_KERNEL_DIR have the same name.

Priot to this change, -D<target> and -D<optimized_kernel_dir> could
result in the same define which would interfere with the kernel variant
definitions (which are tied to the optimized_kernel_dir and are
independent of the target).

Manually tested that the following command fails without this change, and passes with it:
```
make -f tensorflow/lite/micro/tools/make/Makefile TARGET=xtensa TARGET_ARCH=hifi4 XTENSA_CORE=F1_190305_swupgrade -j8 keyword_benchmark
```

BUG=http://b/197889069

This is not caught in presubmit because it requires manual trigger to
run

BUG=http://b/196637015

* Make size check workflow error out if memory footprint increase
between head of main and this PR.

This error can be disabled by putting MEM_CHANGE_OK= tag.
In either case, a memory footprint comparison report will be produced
as part of this workflow action.

The plan is for the code reviewer to audit the report when the
PR increase memory footprint and only
allow the PR that increase memory footprint to be merged with
MEM_CHANGE_OK=what_is_the_reason_for_this_increase_and_why_this_is_ok

The PR shall be up to date for the comparison to be accurate.
So before applying MEM_CHANGE_OK=, please make the PR branch up to date.

BUG=http://b/196637015

* Fix an if condition

Bumps [tensorflow](https://github.com/tensorflow/tensorflow) from 2.5.0 to 2.5.1.
- [Release notes](https://github.com/tensorflow/tensorflow/releases)
- [Changelog](https://github.com/tensorflow/tensorflow/blob/master/RELEASE.md)
- [Commits](https://github.com/tensorflow/tensorflow/compare/v2.5.0...v2.5.1)

---
updated-dependencies:
- dependency-name: tensorflow
  dependency-type: direct:production
...
Signed-off-by: Ndependabot[bot] <support@github.com>
Co-authored-by: Ndependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Updating downloaded CMSIS version.
* Removing -Wno-strict-aliasing for CMSIS-NN.

Change-Id: I41a99fd8d5aa3bf5167a42adfc1ae02449a071a0
Co-authored-by: NAdvait Jain <advaitjain@users.noreply.github.com>

This reverts commit 1ca28598, reversing
changes made to b703c9f9.

Revert created with the following command:
```
git revert 1ca28598 -m 2
```

Merge branch 'delete-arduino' of https://github.com/advaitjain/tflite-micro into advaitjain-delete-arduino

Co-authored-by: NChris Knorowski <chris.knorowski@sensiml.com>

* Replace std::partial_sort with merge_sort in PPD

TFL has changed to std::stable_sort but TFLM can not use it because it
is using the heap. Instead TFLM use merge sort.

Change-Id: I015ee890214b65ebb14cac3b1e888b8f4213ebfb

* Add unit test for stable sort in detection_postprocess
Co-authored-by: NAdvait Jain <advaitjain@users.noreply.github.com>

With https://github.com/tensorflow/tflite-micro/pull/337 we can now run these targets via a bazel build.

BUG=http://b/186676432 and http://b/174680668

Arduino and Sparkfun workflows are now run from their respective GitHub
repositories:
 * [Arduino](https://github.com/tensorflow/tflite-micro-arduino-examples/blob/f2960cf5fc2e483386dbca15529eb3eca57b8038/.github/workflows/ci.yml)
 * [Sparkfun Edge](https://github.com/advaitjain/tflite-micro-sparkfun-edge-examples/blob/0cc8217949d3069d32187c2f2bb62aaedd73c669/.github/workflows/ci.yml)

And the build badges from the TFLM repo also link to the external
Arduino and Sparkfun Edge repositories.

Folow-up changes will delete the Arduio and SParkfun Edge code from the
TFLM repository.

BUG=http://b/196257581 http://b/193823889 and https://github.com/tensorflow/tflite-micro/issues/407

Co-authored-by: NChris Knorowski <chris.knorowski@sensiml.com>

* Fix size_check failure for ci:test label

* Add debug message to debug live

* Update comment

* Remove redundant checkout and add HEAD

This is an attempt to fix size_check workflow not
working for ci:test label

BUG=http://b/196637015Co-authored-by: NAdvait Jain <advaitjain@users.noreply.github.com>

And restore -layering_check to a few places where it got removed with https://github.com/tensorflow/tflite-micro/pull/435/

BUG=http://b/177257332

* Fix misc formatting and spelling errors.

* Re-enable layering check.
Co-authored-by: NAdvait Jain <advaitjain@users.noreply.github.com>

Currently script uses /tmp directory for the the xtensa downloads.
Modifying sciprt to resolve the issue, mktemp creates unique folder.

BUG=usage of /tmp directory for the xtensa downloads.

BUG=http://b/196637015

* Build all the benchmarks as part of a `make build` command.

* Build benchmark as part of `make build`.

BUG=http://b/197017812

* Fix the build

* Add a separate `make build` to the CI scripts.

* Update size_check.yml and rename fusion_f1.json to more general
binary_size.json.

Updte the size_check workflow to be able to launched it
manually for test. Also update it so that it can build image.

The next step is to test whether this will lead to two
PR to have merge conflict always.

BUG=http://b/196637015

* Remove currently unused code

* Create a test used for binary size test.
Also create a script to generate size report of a such binary.
Modify the size_comp.py to be more generic (not limited to xtensa).

what is tensorflow/lite/micro/tools/ci_build/binary_size_test will
be eventually removed once we verify the feature.

The next PR will update the yml file for the size_check.yml.

BUG=http://b/196637015

* Restore lines that are deleted by mistakes

* Update comments

* Add Pillow to Dockerfile.micro

Needed with the changes from https://github.com/tensorflow/tflite-micro/pull/410

BUG=http://b/196630709

* address review comments.

BUG=cleanup

* Enable micro_interpreter_test and two other tests for stm32f4

TESTED=make -j8 -f tensorflow/lite/micro/tools/make/Makefile TARGET=
stm32f4 test

BUG=http://b/174707181

* Update todo

BUG=http://b/158651472

* REF_CODE_REFACTOR: Leaky_Relu
refactoring the reference code for leaky_relu operator.

BUG=refactoring existing code.

* small nits.

* fix the formatting.
Co-authored-by: NAdvait Jain <advaitjain@google.com>

* REF_CODE_REFACTOR: Circular_buffer
refactoring the refernce code for circular_buffer operator.

BUG=refactoring existing code.

* Formatted the code.
Co-authored-by: NAdvait Jain <advaitjain@users.noreply.github.com>
Co-authored-by: NAdvait Jain <advaitjain@google.com>