leap-model-parser 0.1.199.dev2__py3-none-any.whl → 0.1.228__py3-none-any.whl

leap_model_parser/contract/graph.py

@@ -47,6 +47,7 @@ class Node:
     outputs: Dict[str, ConnectionOutput] = field(default_factory=dict)
     pruning_plan_id: Optional[str] = None
     wrapper: Optional[WrapperData] = None
+    shape: Optional[List[str]] = None
 
     def __key(self):
         return (self.id, self.name)

leap_model_parser/keras_json_model_import.py

@@ -5,7 +5,6 @@ from typing import Set, Dict, Any, List, Type, Optional, Tuple
 
 import tensorflow as tf  # type: ignore
 from keras.engine.keras_tensor import KerasTensor  # type: ignore
-from keras.engine.node import Node  # type: ignore
 from keras.engine.node import Node as keras_node # type: ignore
 from keras.layers import Layer # type: ignore
 from keras.layers.convolutional.base_conv import Conv # type: ignore
@@ -206,7 +205,7 @@ class KerasJsonModelImport:
                            'serialized_call_kwargs': serialized_call_kwargs})
 
     def generate_regular_node(self, layer: Dict[str, Any], layer_metadata: Dict[str, Any], node_key: str,
-                              layer_name_to_inbound_nodes: Dict[str, List[Node]]):
+                              layer_name_to_inbound_nodes: Dict[str, List[keras_node]]):
         data = layer['config']
         if layer['class_name'] in ('TFOpLambda', 'SlicingOpLambda') or layer['class_name'] in self.custom_layers:
             call_args = layer_name_to_inbound_nodes[layer['config']
@@ -221,11 +220,17 @@ class KerasJsonModelImport:
 
         self.layer_data_adjustments(data, layer_metadata)
         node = Node(id=str(self.id), name=layer_metadata.get(
-            "class_name", layer["class_name"]), data=data, position=[0, 0])
+            "class_name", layer["class_name"]), data=data, position=[0, 0],
+                    shape=self._convert_layer_shape_to_string(layer['output_shape']))
         if 'wrapper' in layer:
             node.wrapper = layer['wrapper']
         self.nodes_cache[node_key] = node
 
+    @staticmethod
+    def _convert_layer_shape_to_string(s: List[Optional[int]]) -> List[str]:
+        return [str(dim) if dim is not None else 'dynamic_shape' for dim in s]
+
+
     @classmethod
     def handle_wrapper_layer(cls, layer):
         wrapped_layer = layer['config']['layer']
@@ -332,7 +337,8 @@ class KerasJsonModelImport:
         self.connected_inputs.append(input_info)
         input_copy = {
             'type': 'Input',
-            'output_name': input_name
+            'output_name': input_name,
+            'batch_input_shape_origin': shape
         }
         inputNode = Node(
             id=str(self.id), name='Input', data=input_copy, position=[0, 0])

leap_model_parser/leap_graph_editor.py

@@ -31,7 +31,9 @@ class LeapGraphEditor:
                 f"Prediction node with name {prediction_node.name} not found in model graph"
             prediction_node_id = prediction_mapping_node.id
             self.model_graph[prediction_node_id].data['prediction_type'] = prediction_labels_name
-        if node_connection.node.type == NodeMappingType.Visualizer:
+        elif 'Input' in node_connection.node.type.value:
+            self._find_or_add_input_node(node_connection.node)
+        elif node_connection.node.type == NodeMappingType.Visualizer:
             new_visualizer_node_id = self._add_visualizer_node(
                 node_connection.node.name, node_connection.node.sub_type,
                 node_connection.node.user_unique_name, node_connection.node.arg_names)
@@ -47,6 +49,7 @@ class LeapGraphEditor:
                 self._add_connection_to_node(new_metric_node_id, input_name, input_node_id)
         elif node_connection.node.type in (NodeMappingType.Loss, NodeMappingType.CustomLoss):
             new_loss_node_id = self._add_loss_node(node_connection.node.name,
+                                                   node_connection.node.user_unique_name,
                                                    node_connection.node.type == NodeMappingType.CustomLoss,
                                                    node_connection.node.arg_names)
             for input_name, node in node_connection.node_inputs.items():
@@ -71,6 +74,8 @@ class LeapGraphEditor:
 
     def _find_input_node_by_origin_name(self, origin_name: str) -> Optional[Node]:
         for node in self.model_graph.values():
+            if node.data.get('original_output_name') == origin_name:
+                return node
             if node.data.get('output_name') == origin_name:
                 return node
         return None
@@ -122,6 +127,22 @@ class LeapGraphEditor:
         self.model_graph[new_node_id] = ground_truth_node
         return new_node_id
 
+    def _add_input_encoder_not_connected_to_the_model_node(self, input_name: str) -> str:
+        new_node_id = self._generate_new_node_id()
+        ground_truth_node = Node(
+            new_node_id,
+            'Input',
+            position=[0, 0],
+            data={'name': input_name, 'output_name': input_name,
+                  'type': 'Input', "selected": input_name},
+            inputs={},
+            outputs={
+                f'{new_node_id}-{input_name}': ConnectionOutput([])
+            }
+        )
+        self.model_graph[new_node_id] = ground_truth_node
+        return new_node_id
+
     def _add_visualizer_node(self, visualizer_name: str, visualizer_type: str,
                              user_unique_name: str, arg_names: List[str]) -> str:
         new_node_id = self._generate_new_node_id()
@@ -155,7 +176,7 @@ class LeapGraphEditor:
         self.model_graph[new_node_id] = metric_node
         return new_node_id
 
-    def _add_loss_node(self, loss_name: str, is_custom_loss: bool, arg_names: Optional[List[str]]=None) -> str:
+    def _add_loss_node(self, loss_name: str, user_unique_name:str, is_custom_loss: bool, arg_names: Optional[List[str]]=None) -> str:
         new_node_id = self._generate_new_node_id()
 
         loss_type = 'CustomLoss' if is_custom_loss else 'Loss'
@@ -165,7 +186,7 @@ class LeapGraphEditor:
             new_node_id,
             loss_node_name,
             position=[0, 0],
-            data={'type': loss_type, 'selected': loss_name, 'name': loss_name},
+            data={'type': loss_type, 'selected': loss_name, 'name': loss_name, 'user_unique_name': user_unique_name},
             inputs={},
             outputs={
                 f'{new_node_id}-loss': ConnectionOutput([])
@@ -210,7 +231,10 @@ class LeapGraphEditor:
         input_node_by_origin = self._find_input_node_by_origin_name(origin_name)
         assert input_node_by_origin is not None, f"Input node with origin name {origin_name} not found in model graph"
         input_node_id = input_node_by_origin.id
+        if 'original_output_name' not in self.model_graph[input_node_id].data:
+            self.model_graph[input_node_id].data['original_output_name'] = self.model_graph[input_node_id].data['output_name']
         self.model_graph[input_node_id].data['output_name'] = input_node.name
+
         output_keys = list(self.model_graph[input_node_id].outputs.keys())
         for output_key in output_keys:
             new_output_key = f'{input_node_id}-{input_node.name}'
@@ -232,6 +256,8 @@ class LeapGraphEditor:
             if input_node_id is None:
                 if input_node.type == NodeMappingType.GroundTruth:
                     input_node_id = self._add_ground_truth_node(input_node.name)
+                elif input_node.type == NodeMappingType.Input:
+                    input_node_id = self._add_input_encoder_not_connected_to_the_model_node(input_node.name)
                 else:
                     raise Exception(f'Couldnt find input node name {input_node.name}')
         elif 'Input' in input_node.type.value:

leap_model_parser/model_parser.py

@@ -1,3 +1,5 @@
+# mypy: ignore-errors
+
 import glob
 import json
 import ntpath
@@ -6,9 +8,8 @@ import tempfile
 from importlib.util import find_spec
 from pathlib import Path
 
-
 import tensorflow as tf  # type: ignore
-from code_loader.contract.mapping import NodeConnection  # type: ignore
+from code_loader.contract.mapping import NodeConnection, NodeMapping  # type: ignore
 from keras import Model  # type: ignore
 from keras_data_format_converter import convert_channels_first_to_last  # type: ignore
 from leap_model_rebuilder import rebuild_model  # type: ignore
@@ -33,6 +34,7 @@ if spec is not None:
 
     onnx_imported = True
 
+
 class ModelParser:
     def __init__(self, should_transform_inputs_and_outputs=False,
                  custom_layers=None,
@@ -53,8 +55,14 @@ class ModelParser:
             ImportModelTypeEnum.PB_TF2.value: self.convert_pb_model,
         }
 
+    @staticmethod
+    def _add_output_node_shape_to_model_schema(model_schema: Dict, keras_model: Model):
+        for i, layer in enumerate(keras_model.layers):
+            model_schema['config']['layers'][i]['output_shape'] = list(layer.output_shape)
+
     def get_keras_model_and_model_graph(
-            self, model_path: Path, model_type: ImportModelTypeEnum) -> Tuple[Dict[str, Node], List[InputInfo], Optional[Model], Optional[str]]:
+            self, model_path: Path, model_type: ImportModelTypeEnum) -> Tuple[
+        Dict[str, Node], List[InputInfo], Optional[Model], Optional[str]]:
         model_to_keras_converter: Optional[Callable[[str], Tuple[Dict[str, Node], Model, Optional[str]]]] = \
             self._model_types_converter.get(model_type.value)
         if model_to_keras_converter is None:
@@ -63,6 +71,9 @@ class ModelParser:
 
         file_path = str(model_path)
         model_schema, keras_model_with_weights, error_info = model_to_keras_converter(file_path)
+
+        self._add_output_node_shape_to_model_schema(model_schema, keras_model_with_weights)
+
         model_generator = KerasJsonModelImport(self.custom_layers)
 
         keras_model = keras_model_with_weights
@@ -82,7 +93,6 @@ class ModelParser:
 
         return graph, connected_inputs, keras_model_with_weights, error_info
 
-
     def _get_k_model_from_pb_path(self, file_path: str):
         tar_file = tarfile.open(file_path)
         with tempfile.TemporaryDirectory() as temp_dir:
@@ -97,7 +107,8 @@ class ModelParser:
             k_model = self._load_keras_model_with_custom_layers(pb_folder_path)
         return k_model
 
-    def generate_model_graph(self, model_path: Path, model_type: ImportModelTypeEnum) -> Tuple[Dict[str, Node], List[InputInfo]]:
+    def generate_model_graph(self, model_path: Path, model_type: ImportModelTypeEnum) -> Tuple[
+        Dict[str, Node], List[InputInfo]]:
         model_graph, connected_inputs, _, error_info = self.get_keras_model_and_model_graph(
             model_path, model_type)
         return model_graph, connected_inputs
@@ -122,23 +133,56 @@ class ModelParser:
             node.name for node in onnx_model.graph.initializer]
         input_names = list(set(input_all) - set(input_initializer))
         converted_response: ConvertedResponse = onnx_to_keras(onnx_model, input_names=input_names,
-                                name_policy='attach_weights_name', allow_partial_compilation=False)
+                                                              name_policy='attach_weights_name',
+                                                              allow_partial_compilation=False)
         return self.convert_to_keras_model(converted_response.converted_model, converted_response.error_info)
 
     def _load_keras_model_with_custom_layers(self, file_path: str):
         custom_objects = {}
         if self.custom_layers is not None:
             custom_objects = self.custom_layers
-        return load_model(file_path, custom_objects=custom_objects, compile=False)
+
+        try:
+            return load_model(file_path, custom_objects=custom_objects, compile=False)
+        except OSError as e:
+            if 'signature' in str(e):
+                raise Exception('Unable to open model file. The model might be corrupted or not a valid.')
+            else:
+                raise e
+
 
     def convert_h5_model(self, file_path: str) -> Tuple[Dict[str, Node], Model, Optional[str]]:
         imported_model = self._load_keras_model_with_custom_layers(file_path)
         return self.convert_to_keras_model(imported_model)
 
-    def convert_to_keras_model(self, k_model, error_info: Optional[str] = None) -> Tuple[Dict[str, Node], Model, Optional[str]]:
+    def convert_to_keras_model(self, k_model, error_info: Optional[str] = None) -> Tuple[
+        Dict[str, Node], Model, Optional[str]]:
         converted_k_model = convert_channels_first_to_last(
             k_model, self._should_transform_inputs_and_outputs, self.custom_layers)
 
+        from keras.saving.legacy.saved_model import json_utils  # type: ignore
+        import numpy as np
+
+        _orig_get_json_type = json_utils.get_json_type
+
+
+        def _patched_get_json_type(obj):  # type: ignore
+            # Handle numpy dtype explicitly
+            if isinstance(obj, np.dtype):
+                return obj.name  # e.g. "int64"
+            # Make sure common numpy scalars/containers are handled robustly
+            if isinstance(obj, np.integer):
+                return int(obj)
+            if isinstance(obj, np.floating):
+                return float(obj)
+            if isinstance(obj, np.bool_):
+                return bool(obj)
+            if isinstance(obj, np.ndarray):
+                return obj.tolist()
+            return _orig_get_json_type(obj)
+
+        json_utils.get_json_type = _patched_get_json_type
+
         model_schema = json.loads(converted_k_model.to_json())
 
         return model_schema, converted_k_model, error_info

{leap_model_parser-0.1.199.dev2.dist-info → leap_model_parser-0.1.228.dist-info}/METADATA

@@ -1,8 +1,9 @@
-Metadata-Version: 2.3
+Metadata-Version: 2.4
 Name: leap-model-parser
-Version: 0.1.199.dev2
+Version: 0.1.228
 Summary: 
 License: MIT
+License-File: LICENSE
 Author: idan
 Author-email: idan.yogev@tensorleap.ai
 Requires-Python: >=3.8,<3.11
@@ -11,12 +12,12 @@ Classifier: Programming Language :: Python :: 3
 Classifier: Programming Language :: Python :: 3.8
 Classifier: Programming Language :: Python :: 3.9
 Classifier: Programming Language :: Python :: 3.10
-Requires-Dist: code-loader (==1.0.92.dev3)
+Requires-Dist: code-loader (>=1.0.127)
 Requires-Dist: keras-data-format-converter (==0.1.22)
 Requires-Dist: leap-model-rebuilder (==0.1.7)
 Requires-Dist: numpy (>=1.22.3,<2.0.0)
 Requires-Dist: onnx (==1.13.0)
-Requires-Dist: onnx2kerastl (==0.0.174)
+Requires-Dist: onnx2kerastl (==0.0.177)
 Requires-Dist: tensorflow (==2.12.0) ; platform_machine == "x86_64"
 Requires-Dist: tensorflow-io-gcs-filesystem (==0.34.0)
 Requires-Dist: tensorflow-macos (==2.12.0) ; platform_machine == "arm64"

{leap_model_parser-0.1.199.dev2.dist-info → leap_model_parser-0.1.228.dist-info}/RECORD

@@ -1,12 +1,12 @@
 leap_model_parser/__init__.py,sha256=OAU7rFHAVVWUM-cDtQ4Ohum567KN8M-YTkHZp5KiYbo,132
 leap_model_parser/contract/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-leap_model_parser/contract/graph.py,sha256=c5rSlJJQ3JzLhyeniglzrP5GH0fp8UZDQKZw1fcrNe8,1132
+leap_model_parser/contract/graph.py,sha256=lbs_IUnlkvoVjiARBMJee-aNbqeNXNPsUeOEwQliKgU,1170
 leap_model_parser/contract/importmodelresponse.py,sha256=GlvnKS8xrebU2Sj0dxqtEhAOOo3DlOtT7AMJ2BlzH9E,145
 leap_model_parser/contract/nodedata.py,sha256=1_ML0nzp3QUZ0_9mGSLhfO4_hqjYMwi0DWLwymUnWEs,43326
 leap_model_parser/contract/ui_components.json,sha256=0lsxwOLElW1E-imCcdh3zKPWgzFuQ_bApG6aHvYfTvo,410591
-leap_model_parser/keras_json_model_import.py,sha256=x7HOH6iaASfzJgwMRHgF5SQS-iFOF5j9yCG0mDC9HEA,16794
-leap_model_parser/leap_graph_editor.py,sha256=ToPnBUh31hXrA83BMTo631DLQm36PgFE_uTYB-boFAM,12983
-leap_model_parser/model_parser.py,sha256=kodF6aCPXP5FzZR9To2txqu6YdwklmhdJbyQKO_hghM,6587
+leap_model_parser/keras_json_model_import.py,sha256=h9-eb41W67goutRV5c3VkStxvntgb6EmPFdPeshhtsY,17058
+leap_model_parser/leap_graph_editor.py,sha256=s6iVO06xM6LJ8ZjSxkr1srYxsHhGl2KnQjQoc9g4_iE,14329
+leap_model_parser/model_parser.py,sha256=W8FGKCBOBClIl0sdBlTdoSRT8sIdVrlbO8veNMouJ7o,8169
 leap_model_parser/utils/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 leap_model_parser/utils/layerpedia/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 leap_model_parser/utils/layerpedia/layerpedia.py,sha256=1syubfXBTB630TVkgcQ-Ge7Qe9Zbr6EtZRreuqCJnQ8,9292
@@ -17,8 +17,8 @@ leap_model_parser/utils/uicomponents/generatenodedata.py,sha256=LRaPlO5jJ9pUtkvL
 leap_model_parser/utils/uicomponents/tensorflowinscpection.py,sha256=ym613z9iQKPDBpr0RYD35bTABdm1L-Ez86G47BYT7qw,6775
 leap_model_parser/utils/uicomponents/ui_components.json,sha256=0lsxwOLElW1E-imCcdh3zKPWgzFuQ_bApG6aHvYfTvo,410591
 leap_model_parser/utils/uicomponents/ui_components_config.yaml,sha256=cRH8T-c3TAL0nfefRvt9pFsjbTWNEg38NRyHR7RpJsk,19534
-leap_model_parser-0.1.199.dev2.dist-info/LICENSE,sha256=qIwWjdspQeSMTtnFZBC8MuT-95L02FPvzRUdWFxrwJY,1067
-leap_model_parser-0.1.199.dev2.dist-info/METADATA,sha256=uN9eAx9NG4bMszvrxtLDlbzzfBPpuB472G72bJt_zkM,1125
-leap_model_parser-0.1.199.dev2.dist-info/WHEEL,sha256=b4K_helf-jlQoXBBETfwnf4B04YC67LOev0jo4fX5m8,88
-leap_model_parser-0.1.199.dev2.dist-info/entry_points.txt,sha256=ZvV6EuQt1uAqwapNg5Lo2qjJM9ZG5g2wfzZoLh_Ztyk,77
-leap_model_parser-0.1.199.dev2.dist-info/RECORD,,
+leap_model_parser-0.1.228.dist-info/METADATA,sha256=4mXS089Qn_kVo07OcvrVsHvQnbu6clRRnJXircAOEG0,1138
+leap_model_parser-0.1.228.dist-info/WHEEL,sha256=zp0Cn7JsFoX2ATtOhtaFYIiE2rmFAD4OcMhtUki8W3U,88
+leap_model_parser-0.1.228.dist-info/entry_points.txt,sha256=ZvV6EuQt1uAqwapNg5Lo2qjJM9ZG5g2wfzZoLh_Ztyk,77
+leap_model_parser-0.1.228.dist-info/licenses/LICENSE,sha256=qIwWjdspQeSMTtnFZBC8MuT-95L02FPvzRUdWFxrwJY,1067
+leap_model_parser-0.1.228.dist-info/RECORD,,

{leap_model_parser-0.1.199.dev2.dist-info → leap_model_parser-0.1.228.dist-info}/WHEEL

@@ -1,4 +1,4 @@
 Wheel-Version: 1.0
-Generator: poetry-core 2.1.3
+Generator: poetry-core 2.2.1
 Root-Is-Purelib: true
 Tag: py3-none-any