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

update the tensorflow.lite to use ai-edge-litert for all python scripts #2691

Closed

Conversation

ecalubaquib
Copy link
Member

@ecalubaquib ecalubaquib commented Sep 19, 2024

to update all usage from tensorflow.lite to ai-edge-litert
which has similar functionality with tflite_runtime

bug=2691

@ecalubaquib ecalubaquib requested a review from a team as a code owner September 19, 2024 21:06
Copy link

google-cla bot commented Sep 19, 2024

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@ecalubaquib ecalubaquib marked this pull request as draft September 19, 2024 23:06
@ecalubaquib ecalubaquib marked this pull request as ready for review September 19, 2024 23:30
Copy link
Collaborator

@rascani rascani left a comment

Choose a reason for hiding this comment

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

Thanks for the PR!

Copy link
Collaborator

Choose a reason for hiding this comment

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

ai-edge-litert also needs to be added to third_party/python_requirements.in. Please add that, then run bazel run //third_party:python_requirements.update

Copy link
Member Author

Choose a reason for hiding this comment

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

thanks for review. I added ai-edge-litert to requirements.in and ran the bazel update above. Committed the update as well

rascani
rascani previously approved these changes Sep 24, 2024
@ecalubaquib ecalubaquib changed the title update the tensorflow.lite to use ai-edge-litert for all python based… update the tensorflow.lite to use ai-edge-litert for all python based Sep 25, 2024
@ecalubaquib ecalubaquib changed the title update the tensorflow.lite to use ai-edge-litert for all python based update the tensorflow.lite to use ai-edge-litert for all python scripts Sep 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants