Skip to content
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

feat: build and test a Python distribution package tflite_micro #2151

Merged
merged 2 commits into from
Aug 1, 2023

Conversation

rkuester
Copy link
Contributor

Add the build configuration and integrated test to generate a Python
distribution package named tflite_micro for publishing the tflm interpreter
as a Python module with a native extension.

Use the build tools provided in @rules_python, augmented by a custom rule
py_namespace for the reasons documented in python/py_namespace.bzl.

Provide an integration test at //python/tflite_micro:whl_test. Use a .tflite
model copied from the hello_world example. (Copied to avoid creating a
dependency.)

BUG=part of #1484

Add the build configuration and integrated test to generate a
Python distribution package named `tflite_micro` for publishing
the tflm interpreter as a Python module with a native extension.

Use the build tools provided in @rules_python, augmented by a
custom rule `py_namespace` for the reasons documented in
`python/py_namespace.bzl`.

Provide an integration test at `//python/tflite_micro:whl_test`.
Use a .tflite model copied from the hello_world example. (Copied
to avoid creating a dependency.)

BUG=part of tensorflow#1484
@rkuester rkuester requested a review from a team as a code owner July 31, 2023 21:25
@rkuester rkuester self-assigned this Jul 31, 2023
@TFLM-bot TFLM-bot removed the ci:run label Jul 31, 2023
Copy link
Member

@advaitjain advaitjain left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm with a small nit.

python/py_namespace.bzl Outdated Show resolved Hide resolved
@mergify mergify bot merged commit ca74563 into tensorflow:main Aug 1, 2023
68 of 69 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants