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
fix shape inference for onnx-1.8 #1165
Conversation
Signed-off-by: Guenther Schmuelling <guschmue@microsoft.com>
This pull request introduces 2 alerts when merging 0b293b5 into 1bc3079 - view on LGTM.com new alerts:
|
tf2onnx/schemas.py
Outdated
@@ -156,7 +156,7 @@ def build_onnx_op(node): | |||
try: | |||
inferred_model = shape_inference.infer_shapes(model_proto) | |||
except Exception: # pylint: disable=broad-except | |||
logger.warning( | |||
logger.info( |
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.
I have liked that this is a warning since it often indicates a problem with freezing.
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.
done
@@ -16,7 +16,7 @@ stages: | |||
- template: 'templates/job_generator.yml' | |||
parameters: | |||
python_versions: ['3.7'] | |||
tf_versions: ['1.14.0','1.15.2','2.1.0','2.2.0'] | |||
tf_versions: ['1.14.0','1.15.2','2.3.0'] |
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.
Add 2.2.0
tf_versions: ['1.14.0','1.15.2','2.3.0'] | |
tf_versions: ['1.14.0','1.15.2','2.2.0','2.3.0'] |
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.
done
Signed-off-by: Guenther Schmuelling <guschmue@microsoft.com>
/azp run |
Azure Pipelines successfully started running 3 pipeline(s). |
This pull request introduces 2 alerts when merging 6b11bbc into 373908e - view on LGTM.com new alerts:
|
/azp run |
Azure Pipelines successfully started running 3 pipeline(s). |
ci pipeline completed but notify to github seems missing ... forced the merge. |
Signed-off-by: Guenther Schmuelling guschmue@microsoft.com
we used to set subgraphs as attributes after calling make_node() but called shape inference in make_node() which makes it hard for shape inference to do something useful. Pass the subgraphs to make_node() so they are set before calling shape inference.