-
Notifications
You must be signed in to change notification settings - Fork 17
Create examples for using IR passes #300
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
Conversation
Signed-off-by: Justin Chu <[email protected]>
Signed-off-by: Justin Chu <[email protected]>
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #300 +/- ##
==========================================
- Coverage 78.58% 78.56% -0.02%
==========================================
Files 43 43
Lines 5365 5365
Branches 1089 1089
==========================================
- Hits 4216 4215 -1
Misses 841 841
- Partials 308 309 +1 ☔ View full report in Codecov by Sentry. |
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.
Pull request overview
This PR adds documentation examples demonstrating how to use IR passes in the onnx_ir library. The changes include adding a comprehensive example to the documentation file and an inline docstring example to the Sequential class.
Key Changes
- Added example code in the API documentation showing how to use
Sequential, individual passes, andPassManager - Added inline docstring example to the
Sequentialclass demonstrating basic usage - Updated documentation to use proper Sphinx module reference for
onnx_ir.passes.common
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| docs/api/ir_passes.md | Added three code examples demonstrating basic pass usage, individual pass execution, and PassManager orchestration |
| src/onnx_ir/passes/_pass_infra.py | Added inline docstring example to the Sequential class showing how to chain multiple passes |
Signed-off-by: Justin Chu <[email protected]>
Signed-off-by: Justin Chu <[email protected]>
Co-authored-by: Copilot <[email protected]> Signed-off-by: Justin Chu <[email protected]>
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.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.
Signed-off-by: Justin Chu <[email protected]>
Signed-off-by: Justin Chu <[email protected]>
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.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
Signed-off-by: Justin Chu <[email protected]>
Signed-off-by: Justin Chu <[email protected]>
Signed-off-by: Justin Chu <[email protected]>
Signed-off-by: Justin Chu <[email protected]>
Signed-off-by: Justin Chu <[email protected]>
Signed-off-by: Justin Chu <[email protected]>
Signed-off-by: Justin Chu <[email protected]>
No description provided.