[FRONTEND][TFLITE] TFLite frontend ported from Relay by srkreddy1238 · Pull Request #18685 · apache/tvm

srkreddy1238 · 2026-01-23T11:04:41Z

Verified for entire range of classification nets
Quantization is disabled at the moment
There exists few unspoorted ops in convertion maps which is need to be mapped in future when relax op inventory grows.

gemini-code-assist · 2026-01-23T11:04:58Z

Summary of Changes

Hello @srkreddy1238, I'm Gemini Code Assist¹! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

This pull request integrates a TensorFlow Lite (TFLite) frontend into the TVM Relax framework, allowing for the conversion and execution of TFLite models. The implementation includes a specialized FlexBuffer parser to handle TFLite's custom options and is validated by a broad range of tests, ensuring compatibility with common TensorFlow operations and popular Keras classification models. This lays the groundwork for broader TFLite model support in Relax.

Highlights

New TFLite Frontend: Introduced a new frontend for TensorFlow Lite (TFLite) within the TVM Relax framework, enabling the conversion of TFLite models into Relax graphs.
FlexBuffer Decoder: Implemented a partial FlexBuffer decoder to correctly parse custom options embedded within TFLite models.
Comprehensive Test Coverage: Added an extensive test suite covering various TensorFlow operations (e.g., binary, element-wise, logical, reduce, conv2d, pooling) and end-to-end validation with several Keras classification networks.
Current Limitations: Quantization support is currently disabled, and a few TFLite operations are not yet mapped, pending future expansion of the Relax op inventory.

🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console.

Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature	Command	Description
Code Review	`/gemini review`	Performs a code review for the current pull request in its current state.
Pull Request Summary	`/gemini summary`	Provides a summary of the current pull request in its current state.
Comment	@gemini-code-assist	Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help	`/gemini help`	Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution. ↩

gemini-code-assist

Code Review

This pull request introduces the TFLite frontend for Relax, enabling conversion of TFLite graphs into Relax graphs. It includes a Flexbuffer decoder for custom options and a comprehensive set of tests for various TensorFlow operations and Keras applications. While the overall structure is good, there are several critical issues related to missing imports, incorrect handling of data types, and potential bugs in the Flexbuffer deserialization and quantized operations that need to be addressed before merging. Additionally, some security and maintainability concerns were identified.

python/tvm/relax/frontend/tflite/tflite_flexbuffer.py

gemini-code-assist · 2026-01-23T11:07:26Z

tests/python/relax/test_frontend_tflite.py

+
+import ssl
+
+ssl._create_default_https_context = ssl._create_unverified_context


Disabling SSL certificate verification globally using ssl._create_unverified_context is a security risk and generally discouraged. For testing purposes, consider using a more targeted approach, such as a context manager, or ensuring the test environment has proper certificates.

python/tvm/relax/frontend/tflite/tflite_flexbuffer.py

gemini-code-assist · 2026-01-23T11:07:26Z

tests/python/relax/test_frontend_tflite.py

+        # Limiting the tests for CI
+        (keras_app.Xception, (1, 299, 299, 3)),
+        # (keras_app.VGG16, (1, 224, 224, 3)),
+        # (keras_app.VGG19, (1, 224, 224, 3)),
+        (keras_app.ResNet50, (1, 224, 224, 3)),
+        # (keras_app.ResNet50V2, (1, 224, 224, 3)),
+        # (keras_app.ResNet101, (1, 224, 224, 3)),
+        # (keras_app.ResNet101V2, (1, 224, 224, 3)),
+        # (keras_app.ResNet152, (1, 224, 224, 3)),
+        # (keras_app.ResNet152V2, (1, 224, 224, 3)),
+        (keras_app.InceptionResNetV2, (1, 299, 299, 3)),
+        # (keras_app.MobileNet, (1, 224, 224, 3)),
+        (keras_app.MobileNetV2, (1, 224, 224, 3)),
+        (keras_app.DenseNet121, (1, 224, 224, 3)),
+        # (keras_app.DenseNet169, (1, 224, 224, 3)),
+        # (keras_app.DenseNet201, (1, 224, 224, 3)),
+        (keras_app.NASNetMobile, (1, 224, 224, 3)),
+        # (keras_app.NASNetLarge, (1, 331, 331, 3)),
+        (keras_app.EfficientNetB0, (1, 224, 224, 3)),
+        # (keras_app.EfficientNetB1, (1, 240, 240, 3)),
+        # (keras_app.EfficientNetB2, (1, 260, 260, 3)),
+        # (keras_app.EfficientNetB3, (1, 300, 300, 3)),
+        # (keras_app.EfficientNetB4, (1, 380, 380, 3)),
+        # (keras_app.EfficientNetB5, (1, 456, 456, 3)),
+        # (keras_app.EfficientNetB6, (1, 528, 528, 3)),
+        # (keras_app.EfficientNetB7, (1, 600, 600, 3)),
+        (keras_app.EfficientNetV2B0, (1, 224, 224, 3)),
+        # (keras_app.EfficientNetV2B1, (1, 240, 240, 3)),
+        # (keras_app.EfficientNetV2B2, (1, 260, 260, 3)),
+        # (keras_app.EfficientNetV2B3, (1, 300, 300, 3)),
+        # (keras_app.EfficientNetV2S, (1, 384, 384, 3)),
+        # (keras_app.EfficientNetV2M, (1, 480, 480, 3)),
+        # (keras_app.EfficientNetV2L, (1, 480, 480, 3)),
+        (keras_app.ConvNeXtTiny, (1, 224, 224, 3)),
+        # (keras_app.ConvNeXtSmall, (1, 224, 224, 3)),
+        # (keras_app.ConvNeXtBase, (1, 224, 224, 3)),
+        # (keras_app.ConvNeXtLarge, (1, 224, 224, 3)),
+        # (keras_app.ConvNeXtXLarge, (1, 224, 224, 3)),
+    ],


Many network tests are commented out with the note "Limiting the tests for CI". While this might be necessary for CI resource constraints, it means a significant portion of the TFLite frontend's functionality for these Keras models is not being tested. This could lead to regressions or undetected issues. Consider enabling these tests for local development or in a less constrained CI environment, or adding a tracking issue to re-enable them.

python/tvm/relax/frontend/tflite/tflite_frontend.py

mshr-h · 2026-02-02T05:15:18Z

tests/python/relax/test_frontend_tflite.py

+    return mod
+
+
+def verify(TestClass, expected=None):


Please avoid e2e tests to reduce CI pressure. Structual equality is better. Nightly might be okay.

Do we have test marker now to skip while in CI ?

You can put nightly tests in tests/python/nightly.

Instead of duplicating the under tests/python/nightly managed calling tflite tests from tests/scripts/task_python_nightly.sh with different environment

python/tvm/relax/frontend/tflite/tflite_frontend.py

mshr-h · 2026-02-02T05:36:48Z

tests/python/relax/test_frontend_tflite.py

+
+import ssl
+
+ssl._create_default_https_context = ssl._create_unverified_context


Why is this line needed?

The network tests where the keras tries to download the model weights.

mshr-h · 2026-02-02T15:04:39Z

python/tvm/relax/frontend/tflite/tflite_frontend.py

+    def __init__(self, model, subgraph, exp_tab, ctx):
+
+        try:
+            from tflite.ActivationFunctionType import ActivationFunctionType


I find it a bit annoying that the codebase has so many duplicate import statements. I’d prefer consolidating imports at the top level.

The front ends are not importing corresponding frameworks globally. Tried to clean by removing exception handling and removing unnecessary imports with in op handlers.

Verified for entire range of classification nets Quantization is disabled at the moment There exists few unspoorted ops in convertion maps which is need to be mapped in future when relax op inventory grows.

tests/python/relax/test_frontend_tflite.py

gemini-code-assist bot reviewed Jan 23, 2026

View reviewed changes

srkreddy1238 changed the title ~~[FRONTEND][TFLITE] TFLite frontend ported from Relax~~ [FRONTEND][TFLITE] TFLite frontend ported from Relay Jan 27, 2026

srkreddy1238 requested a review from yongwww January 28, 2026 06:18

mshr-h requested changes Feb 2, 2026

View reviewed changes

mshr-h reviewed Feb 2, 2026

View reviewed changes

srkreddy1238 added 4 commits February 3, 2026 10:30

[FRONTEND][TFLITE] TFLite frontend ported from Relax

5a59c9a

Verified for entire range of classification nets Quantization is disabled at the moment There exists few unspoorted ops in convertion maps which is need to be mapped in future when relax op inventory grows.

import

9fc9b59

GPU tests commented

9767290

CI tests

a057ca9

mshr-h reviewed Feb 3, 2026

View reviewed changes

tests/python/relax/test_frontend_tflite.py Outdated Show resolved Hide resolved

srkreddy1238 force-pushed the tflite_frontend branch from 03918e8 to 8e93e22 Compare February 3, 2026 17:45

Review

34d07ae

srkreddy1238 force-pushed the tflite_frontend branch from 8e93e22 to 34d07ae Compare February 3, 2026 17:48

srkreddy1238 added 3 commits February 4, 2026 09:49

Nightly tests

cf738d0

nightly

5937713

import

37e2ac0


		import ssl

		ssl._create_default_https_context = ssl._create_unverified_context

Conversation

srkreddy1238 commented Jan 23, 2026

Uh oh!

gemini-code-assist bot commented Jan 23, 2026

Summary of Changes

Highlights

Footnotes

Uh oh!

gemini-code-assist bot left a comment

Choose a reason for hiding this comment

Code Review

Uh oh!

Uh oh!

Uh oh!

gemini-code-assist bot Jan 23, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

gemini-code-assist bot Jan 23, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

mshr-h Feb 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

mshr-h Feb 2, 2026 •

edited

Loading