Skip to content

Commit

Permalink
removing default op_domain
Browse files Browse the repository at this point in the history
Signed-off-by: Jason Wang <jasowang@microsoft.com>
  • Loading branch information
memoryz committed Jun 1, 2022
1 parent 0fd6b9b commit 9b98131
Showing 1 changed file with 0 additions and 9 deletions.
9 changes: 0 additions & 9 deletions onnxmltools/convert/sparkml/operator_converters/k_means.py
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,6 @@ def convert_sparkml_k_means_model(scope: Scope, operator: Operator, container: M
op_type="ReduceSumSquare",
inputs=[operator.inputs[0].full_name],
outputs=[input_row_squared_sum_variable_name],
op_domain="ai.onnx",
**reduce_sum_square_attrs
)

Expand All @@ -66,7 +65,6 @@ def convert_sparkml_k_means_model(scope: Scope, operator: Operator, container: M
op_type="Gemm",
inputs=[operator.inputs[0].full_name, centers_variable_name, input_row_squared_sum_variable_name],
outputs=[gemm_output_variable_name],
op_domain="ai.onnx",
**gemm_attrs
)

Expand All @@ -76,7 +74,6 @@ def convert_sparkml_k_means_model(scope: Scope, operator: Operator, container: M
op_type="Add",
inputs=[gemm_output_variable_name, centers_row_squared_sum_variable_name],
outputs=[distance_output_variable_name],
op_domain="ai.onnx",
)
elif op.getDistanceMeasure() == "cosine":
# centers_row_norm2: [1 x K]
Expand All @@ -100,7 +97,6 @@ def convert_sparkml_k_means_model(scope: Scope, operator: Operator, container: M
op_type="ReduceL2",
inputs=[operator.inputs[0].full_name],
outputs=[input_row_norm2_variable_name],
op_domain="ai.onnx",
**reduce_l2_attrs
)

Expand All @@ -115,7 +111,6 @@ def convert_sparkml_k_means_model(scope: Scope, operator: Operator, container: M
op_type="Gemm",
inputs=[operator.inputs[0].full_name, centers_variable_name],
outputs=[gemm_output_variable_name],
op_domain="ai.onnx",
**gemm_attrs
)

Expand All @@ -125,15 +120,13 @@ def convert_sparkml_k_means_model(scope: Scope, operator: Operator, container: M
op_type="Div",
inputs=[gemm_output_variable_name, input_row_norm2_variable_name],
outputs=[div_output_variable_name],
op_domain="ai.onnx",
op_version=7, # Setting to version 7 for broadcasting support
)
cosine_similarity_output_variable_name = scope.get_unique_variable_name("cosine_similarity_output")
container.add_node(
op_type="Div",
inputs=[div_output_variable_name, centers_row_norm2_variable_name],
outputs=[cosine_similarity_output_variable_name],
op_domain="ai.onnx",
op_version=7, # Setting to version 7 for broadcasting support
)

Expand All @@ -142,7 +135,6 @@ def convert_sparkml_k_means_model(scope: Scope, operator: Operator, container: M
op_type="Neg",
inputs=[cosine_similarity_output_variable_name],
outputs=[distance_output_variable_name],
op_domain="ai.onnx",
)
else:
raise ValueError(f"Distance measure {op.getDistanceMeasure()} not supported")
Expand All @@ -156,7 +148,6 @@ def convert_sparkml_k_means_model(scope: Scope, operator: Operator, container: M
op_type="ArgMin",
inputs=[distance_output_variable_name],
outputs=[operator.outputs[0].full_name],
op_domain="ai.onnx",
**argmin_attrs
)

Expand Down

0 comments on commit 9b98131

Please sign in to comment.