-
Notifications
You must be signed in to change notification settings - Fork 306
Model Export to liteRT #2405
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
Open
pctablet505
wants to merge
62
commits into
keras-team:master
Choose a base branch
from
pctablet505:export
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Model Export to liteRT #2405
Changes from 24 commits
Commits
Show all changes
62 commits
Select commit
Hold shift + click to select a range
087b9b2
Update backbone.py
pctablet505 de830b1
Update backbone.py
pctablet505 62d2484
Update task.py
pctablet505 3b71125
Revert "Update task.py"
pctablet505 3d453ff
Revert "Update backbone.py"
pctablet505 92b1254
export
pctablet505 e46241d
refactoring
pctablet505 6e970e2
refactor
pctablet505 15ad9f3
Update registry.py
pctablet505 02ca0d9
Refactor export logic and improve error handling
pctablet505 901c233
Merge branch 'keras-team:master' into export
pctablet505 442fdd3
reformat
pctablet505 5446e2a
Add export submodule to keras_hub API
pctablet505 5c31d88
reformat
pctablet505 3290d42
now supporting export for objectDetectors
pctablet505 8b1024f
Add and refine image model exporter configs
pctablet505 8df5a75
Refactor: move keras import to module level
pctablet505 759d223
Remove debug_object_detection.py script
pctablet505 0737c93
Rename LiteRT to Litert and update exporter configs
pctablet505 c733e18
Refactor InputSpec formatting and fix import path
pctablet505 5ab911f
Refactor exporter configs and model building logic
pctablet505 c1e26dd
Refactor export initialization and improve warnings
pctablet505 6fa8379
Improve dtype handling and verbose output in exporters
pctablet505 81c6ed5
Remove get_dummy_inputs methods from exporter configs
pctablet505 d6a8dfd
Rename LitertExporter to LiteRTExporter
pctablet505 663c190
Update registry.py
pctablet505 e0d02ee
Refactor exporter registry to use model classes
pctablet505 6c98400
Remove conditional import for keras
pctablet505 b9e3789
Add comprehensive export test suites for Keras Hub
pctablet505 9f63b2a
Refactor LiteRT exporter model adapters
pctablet505 b4ce293
Merge branch 'keras-team:master' into export
pctablet505 4ebc701
Clarify type annotations in docstrings for export modules
pctablet505 298967e
testing refactor
pctablet505 bc0a8b7
refactor test
pctablet505 ab99186
Fix LiteRT export filepath and mask argument usage
pctablet505 1c06c46
Refactor LiteRTExporter model adapter calls
pctablet505 22587f1
Add warning for private TensorFlow API usage
pctablet505 765d55c
Merge branch 'export' of https://github.com/pctablet505/keras-hub int…
pctablet505 70f712a
Refactor exporter configs and remove TextModelExporterConfig
pctablet505 e47545d
Refactor trackable children filtering logic
pctablet505 0a266b4
Refactor ExporterRegistry model config lookup
pctablet505 21f6b2c
Update litert.py
pctablet505 efa25ae
Refactor tests to remove try/except and improve clarity
pctablet505 ec37ac4
Fix docstring in TextClassifierExporterConfig
pctablet505 911eb96
Update base.py
pctablet505 51b99b1
Create litert_export_design.md
pctablet505 7ef9348
Refactor LiteRT export tests for consistency and efficiency
pctablet505 2295181
Refactor LiteRT export tests to support per-output thresholds
pctablet505 4adeadf
Update litert_models_test.py
pctablet505 00f49ca
Delete litert_export_design.md
pctablet505 5fa0498
Update litert_models_test.py
pctablet505 052669d
Refactor LiteRT export tests to use pytest parametrization
pctablet505 a273e42
Refactor export registry and add direct export to Task
pctablet505 519c3b6
Update litert.py
pctablet505 2dcbf23
Merge branch 'keras-team:master' into export
pctablet505 0136c34
Update task.py
pctablet505 f8bd6fa
Merge branch 'export' of https://github.com/pctablet505/keras-hub int…
pctablet505 14cffe0
Update test_case.py
pctablet505 9267b51
Enable dynamic input shapes for LiteRT export
pctablet505 c622d8d
Improve SignatureDef handling in LiteRT export tests
pctablet505 ca6056b
Refactor LiteRT test utilities for clarity and robustness
pctablet505 d43de36
Refactor TFLite inference to use signature runner
pctablet505 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,28 @@ | ||
| """DO NOT EDIT. | ||
|
|
||
| This file was autogenerated. Do not edit it by hand, | ||
| since your modifications would be overwritten. | ||
| """ | ||
|
|
||
| from keras_hub.src.export.configs import ( | ||
| CausalLMExporterConfig as CausalLMExporterConfig, | ||
| ) | ||
| from keras_hub.src.export.configs import ( | ||
| ImageClassifierExporterConfig as ImageClassifierExporterConfig, | ||
| ) | ||
| from keras_hub.src.export.configs import ( | ||
| ImageSegmenterExporterConfig as ImageSegmenterExporterConfig, | ||
| ) | ||
| from keras_hub.src.export.configs import ( | ||
| ObjectDetectorExporterConfig as ObjectDetectorExporterConfig, | ||
| ) | ||
| from keras_hub.src.export.configs import ( | ||
| Seq2SeqLMExporterConfig as Seq2SeqLMExporterConfig, | ||
| ) | ||
| from keras_hub.src.export.configs import ( | ||
| TextClassifierExporterConfig as TextClassifierExporterConfig, | ||
| ) | ||
| from keras_hub.src.export.configs import ( | ||
| TextModelExporterConfig as TextModelExporterConfig, | ||
| ) | ||
| from keras_hub.src.export.litert import LitertExporter as LitertExporter |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,9 @@ | ||
| from keras_hub.src.export.base import ExporterRegistry | ||
| from keras_hub.src.export.base import KerasHubExporter | ||
| from keras_hub.src.export.base import KerasHubExporterConfig | ||
| from keras_hub.src.export.configs import CausalLMExporterConfig | ||
| from keras_hub.src.export.configs import Seq2SeqLMExporterConfig | ||
| from keras_hub.src.export.configs import TextClassifierExporterConfig | ||
| from keras_hub.src.export.configs import TextModelExporterConfig | ||
| from keras_hub.src.export.litert import LitertExporter | ||
| from keras_hub.src.export.litert import export_litert |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,248 @@ | ||
| """Base classes for Keras-Hub model exporters. | ||
| This module provides the foundation for exporting Keras-Hub models to various | ||
| formats. It follows the Optimum pattern of having different exporters for | ||
| different model types and formats. | ||
| """ | ||
|
|
||
| from abc import ABC | ||
| from abc import abstractmethod | ||
|
|
||
| try: | ||
| import keras | ||
|
|
||
| KERAS_AVAILABLE = True | ||
| except ImportError: | ||
| KERAS_AVAILABLE = False | ||
| keras = None | ||
|
|
||
|
|
||
| class KerasHubExporterConfig(ABC): | ||
| """Base configuration class for Keras-Hub model exporters. | ||
| This class defines the interface for exporter configurations that specify | ||
| how different types of Keras-Hub models should be exported. | ||
| """ | ||
|
|
||
| # Model type this exporter handles (e.g., "causal_lm", "text_classifier") | ||
| MODEL_TYPE = None | ||
|
|
||
| # Expected input structure for this model type | ||
| EXPECTED_INPUTS = [] | ||
|
|
||
| # Default sequence length if not specified | ||
| DEFAULT_SEQUENCE_LENGTH = 128 | ||
|
|
||
| def __init__(self, model, **kwargs): | ||
| """Initialize the exporter configuration. | ||
| Args: | ||
| model: `keras.Model`. The Keras-Hub model to export. | ||
| **kwargs: Additional configuration parameters. | ||
| """ | ||
| self.model = model | ||
| self.config_kwargs = kwargs | ||
| self._validate_model() | ||
|
|
||
| def _validate_model(self): | ||
| """Validate that the model is compatible with this exporter.""" | ||
| if not self._is_model_compatible(): | ||
| raise ValueError( | ||
| f"Model {self.model.__class__.__name__} is not compatible " | ||
| f"with {self.__class__.__name__}" | ||
| ) | ||
|
|
||
| @abstractmethod | ||
| def _is_model_compatible(self): | ||
| """Check if the model is compatible with this exporter. | ||
| Returns: | ||
| bool: True if compatible, False otherwise | ||
| """ | ||
| pass | ||
|
|
||
| @abstractmethod | ||
| def get_input_signature(self, sequence_length=None): | ||
| """Get the input signature for this model type. | ||
| Args: | ||
| sequence_length: `int` or `None`. Optional sequence length for | ||
| input tensors. | ||
| Returns: | ||
| A dictionary mapping input names to their tensor specifications. | ||
| """ | ||
| pass | ||
|
|
||
|
|
||
| class KerasHubExporter(ABC): | ||
| """Base class for Keras-Hub model exporters. | ||
| This class provides the common interface for exporting Keras-Hub models | ||
| to different formats (LiteRT, ONNX, etc.). | ||
| """ | ||
|
|
||
| def __init__(self, config, **kwargs): | ||
| """Initialize the exporter. | ||
| Args: | ||
| config: `KerasHubExporterConfig`. Exporter configuration specifying | ||
| model type and parameters. | ||
| **kwargs: Additional exporter-specific parameters. | ||
| """ | ||
| self.config = config | ||
| self.model = config.model | ||
| self.export_kwargs = kwargs | ||
|
|
||
| @abstractmethod | ||
| def export(self, filepath): | ||
| """Export the model to the specified filepath. | ||
| Args: | ||
| filepath: `str`. Path where to save the exported model. | ||
| """ | ||
| pass | ||
|
|
||
| def _ensure_model_built(self, param=None): | ||
| """Ensure the model is properly built with correct input structure. | ||
| This method builds the model using model.build() with input shapes. | ||
| This creates the necessary variables and initializes the model structure | ||
| for export without needing dummy data. | ||
| Args: | ||
| param: `int` or `None`. Optional parameter for input signature | ||
| (e.g., sequence_length for text models, image_size for vision | ||
| models). | ||
| """ | ||
| # Get input signature (returns dict of InputSpec objects) | ||
| input_signature = self.config.get_input_signature(param) | ||
|
|
||
| # Extract shapes from InputSpec objects | ||
| input_shapes = {} | ||
| for name, spec in input_signature.items(): | ||
| if hasattr(spec, "shape"): | ||
| input_shapes[name] = spec.shape | ||
| else: | ||
| # Fallback for unexpected formats | ||
| input_shapes[name] = spec | ||
|
|
||
| # Build the model using shapes only (no actual data allocation) | ||
| # This creates variables and initializes the model structure | ||
| self.model.build(input_shape=input_shapes) | ||
|
|
||
|
|
||
| class ExporterRegistry: | ||
| """Registry for mapping model types to their appropriate exporters.""" | ||
|
|
||
| _configs = {} | ||
| _exporters = {} | ||
|
|
||
| @classmethod | ||
| def register_config(cls, model_type, config_class): | ||
| """Register a configuration class for a model type. | ||
| Args: | ||
| model_type: The model type (e.g., "causal_lm") | ||
| config_class: The configuration class | ||
| """ | ||
pctablet505 marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
| cls._configs[model_type] = config_class | ||
|
|
||
| @classmethod | ||
| def register_exporter(cls, format_name, exporter_class): | ||
| """Register an exporter class for a format. | ||
| Args: | ||
| format_name: The export format (e.g., "litert") | ||
| exporter_class: The exporter class | ||
| """ | ||
| cls._exporters[format_name] = exporter_class | ||
|
|
||
| @classmethod | ||
| def get_config_for_model(cls, model): | ||
| """Get the appropriate configuration for a model. | ||
| Args: | ||
| model: The Keras-Hub model | ||
| Returns: | ||
| KerasHubExporterConfig: An appropriate exporter configuration | ||
| instance | ||
| Raises: | ||
| ValueError: If no configuration is found for the model type | ||
| """ | ||
| model_type = cls._detect_model_type(model) | ||
|
|
||
| if model_type not in cls._configs: | ||
| raise ValueError( | ||
| f"No configuration found for model type: {model_type}" | ||
| ) | ||
|
|
||
| config_class = cls._configs[model_type] | ||
| return config_class(model) | ||
|
|
||
| @classmethod | ||
| def get_exporter(cls, format_name, config, **kwargs): | ||
| """Get an exporter for the specified format. | ||
| Args: | ||
| format_name: The export format | ||
| config: The exporter configuration | ||
| **kwargs: Additional parameters for the exporter | ||
| Returns: | ||
| KerasHubExporter: An appropriate exporter instance | ||
| Raises: | ||
| ValueError: If no exporter is found for the format | ||
| """ | ||
| if format_name not in cls._exporters: | ||
| raise ValueError(f"No exporter found for format: {format_name}") | ||
|
|
||
| exporter_class = cls._exporters[format_name] | ||
| return exporter_class(config, **kwargs) | ||
|
|
||
| @classmethod | ||
| def _detect_model_type(cls, model): | ||
| """Detect the model type from the model instance. | ||
| Args: | ||
| model: The Keras-Hub model | ||
| Returns: | ||
| str: The detected model type | ||
| """ | ||
| # Import here to avoid circular imports | ||
| try: | ||
| from keras_hub.src.models.causal_lm import CausalLM | ||
| from keras_hub.src.models.image_segmenter import ImageSegmenter | ||
| from keras_hub.src.models.object_detector import ObjectDetector | ||
| from keras_hub.src.models.seq_2_seq_lm import Seq2SeqLM | ||
| except ImportError: | ||
| CausalLM = None | ||
| Seq2SeqLM = None | ||
| ObjectDetector = None | ||
| ImageSegmenter = None | ||
|
|
||
| model_class_name = model.__class__.__name__ | ||
|
|
||
| if CausalLM and isinstance(model, CausalLM): | ||
| return "causal_lm" | ||
| elif "TextClassifier" in model_class_name: | ||
| return "text_classifier" | ||
| elif Seq2SeqLM and isinstance(model, Seq2SeqLM): | ||
| return "seq2seq_lm" | ||
| elif "ImageClassifier" in model_class_name: | ||
| return "image_classifier" | ||
| elif ObjectDetector and isinstance(model, ObjectDetector): | ||
| return "object_detector" | ||
| elif "ObjectDetector" in model_class_name: | ||
| return "object_detector" | ||
| elif ImageSegmenter and isinstance(model, ImageSegmenter): | ||
| return "image_segmenter" | ||
| elif "ImageSegmenter" in model_class_name: | ||
| return "image_segmenter" | ||
| else: | ||
| # Default to text model for generic Keras-Hub models | ||
| return "text_model" | ||
pctablet505 marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
pctablet505 marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The alias
as exportis redundant here. For better readability and to follow common Python conventions, you can simplify this import. While this pattern exists in the file, it's a good opportunity to correct it for the new addition.