diff options
author | Benjamin Klimczak <benjamin.klimczak@arm.com> | 2023-07-19 16:35:57 +0100 |
---|---|---|
committer | Benjamin Klimczak <benjamin.klimczak@arm.com> | 2023-10-11 16:06:17 +0100 |
commit | 3cd84481fa25e64c29e57396d4bf32d7a3ca490a (patch) | |
tree | ad81fb520a965bd3a3c7c983833b7cd48f9b8dea /tests/utils | |
parent | f3e6597dd50ec70f043d692b773f2d9fd31519ae (diff) | |
download | mlia-3cd84481fa25e64c29e57396d4bf32d7a3ca490a.tar.gz |
Bug-fixes and re-factoring for the rewrite module
- Fix input shape of rewrite replacement:
During and after training of the replacement model for a rewrite the
Keras model is converted and saved in TensorFlow Lite format. If the
input shape does not match the teacher model exactly, e.g. if the
batch size is undefined, the TFLiteConverter adds extra operators
during conversion.
- Fix rewritten model output
- Save the model output with the rewritten operator in the output dir
- Log MAE and NRMSE of the rewrite
- Remove 'verbose' flag from rewrite module and rely on the logging
mechanism to control verbose output.
- Re-factor utility classes for rewrites
- Merge the two TFLiteModel classes
- Move functionality to load/save TensorFlow Lite flatbuffers to
nn/tensorflow/tflite_graph
- Fix issue with unknown shape in datasets
After upgrading to TensorFlow 2.12 the unknown shape of the
TFRecordDataset is causing problems when training the replacement models
for rewrites. By explicitly setting the right shape of the tensors we
can work around the issue.
- Adapt default parameters for rewrites. The training steps especially
had to be increased significantly to be effective.
Resolves: MLIA-895, MLIA-907, MLIA-946, MLIA-979
Signed-off-by: Benjamin Klimczak <benjamin.klimczak@arm.com>
Change-Id: I887ad165aed0f2c6e5a0041f64cec5e6c5ab5c5c
Diffstat (limited to 'tests/utils')
-rw-r--r-- | tests/utils/rewrite.py | 18 |
1 files changed, 18 insertions, 0 deletions
diff --git a/tests/utils/rewrite.py b/tests/utils/rewrite.py index 4264b4b..739bb11 100644 --- a/tests/utils/rewrite.py +++ b/tests/utils/rewrite.py @@ -3,8 +3,12 @@ """Common test utils for the rewrite tests.""" from __future__ import annotations +from typing import Any + from tensorflow.lite.python.schema_py_generated import ModelT +from mlia.nn.rewrite.core.train import TrainingParameters + def models_are_equal(model1: ModelT, model2: ModelT) -> bool: """Check that the two models are equal.""" @@ -25,3 +29,17 @@ def models_are_equal(model1: ModelT, model2: ModelT) -> bool: return False # Tensor from graph1 not found in other graph.") return True + + +class TestTrainingParameters( + TrainingParameters +): # pylint: disable=too-few-public-methods + """ + TrainingParameter class for rewrites with different default values. + + To speed things up for the unit tests. + """ + + def __init__(self, *args: Any, steps: int = 32, **kwargs: Any) -> None: + """Initialize TrainingParameters with different defaults.""" + super().__init__(*args, steps=steps, **kwargs) # type: ignore |