-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
Update shape inference documentation #4163
Conversation
Signed-off-by: Ganesan Ramalingam <grama@microsoft.com>
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 like this document enhancement -- especially the documentation for shape inference utilities.
Signed-off-by: Ganesan Ramalingam <grama@microsoft.com>
handle the possibility that the dimension may not have a statically | ||
known value. | ||
|
||
There are several utility functions in [shape_inference.h](/onnx/defs/shape_inference.h) |
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.
Should we make sure users are using the utility functions to throw exceptions if one shape has an unexpecgted value?
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.
Yes, but, unfortunately, we are dependent on reviewers to check for this. Any other suggestions?
Signed-off-by: Ganesan Ramalingam <grama@microsoft.com>
* Update shape inference documentation Signed-off-by: Ganesan Ramalingam <grama@microsoft.com> * Add an example. Signed-off-by: Ganesan Ramalingam <grama@microsoft.com> * Move sub-section Signed-off-by: Ganesan Ramalingam <grama@microsoft.com> Signed-off-by: Liqun Fu <liqfu@microsoft.com>
* Update shape inference documentation Signed-off-by: Ganesan Ramalingam <grama@microsoft.com> * Add an example. Signed-off-by: Ganesan Ramalingam <grama@microsoft.com> * Move sub-section Signed-off-by: Ganesan Ramalingam <grama@microsoft.com>
Signed-off-by: Ganesan Ramalingam grama@microsoft.com
Description
Update the shape-inference documentation.
Motivation and Context
Helps developers who write shape-inference code for operators avoid some common mistakes.