Skip to content

Revert "Change default optimization level to All (from Basic)"#2776

Merged
snnn merged 1 commit intomasterfrom
revert-2745-askhade/update_graph_transformer_level
Jan 3, 2020
Merged

Revert "Change default optimization level to All (from Basic)"#2776
snnn merged 1 commit intomasterfrom
revert-2745-askhade/update_graph_transformer_level

Conversation

@snnn
Copy link
Contributor

@snnn snnn commented Jan 3, 2020

Reverts #2745

A regression was found in converter pipelines.

@snnn snnn requested a review from a team as a code owner January 3, 2020 21:36
@tianleiwu
Copy link
Contributor

What is the regression in converter pipeline? Could the fix be done in converter pipeline instead?

@jiafatom
Copy link
Contributor

jiafatom commented Jan 3, 2020

What is the regression in converter pipeline? Could the fix be done in converter pipeline instead?

It seems that the fusion has some issues, the error is:
onnxruntime.capi.onnxruntime_pybind11_state.InvalidGraph: [ONNXRuntimeError] : 10 : INVALID_GRAPH : This is an invalid model. Error in Node:fused gemm_0_9 : Attribute 'alpha' appeared multiple times.

The converted model does not contain Gemm op, this is introduced from fusion, not converter.

@snnn snnn merged commit 013642e into master Jan 3, 2020
@snnn snnn deleted the revert-2745-askhade/update_graph_transformer_level branch January 3, 2020 23:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants