Repository navigation
Add generic operator package rewrites and integration tests - #2756
Closed
Akshay Sonawane (apsonawane) wants to merge 2 commits into
Closed
Akshay Sonawane (apsonawane) wants to merge 2 commits into
Akshay Sonawane (apsonawane) wants to merge 2 commits into
Conversation
Exclude benchmark, profiling and evaluation scripts from this stack.
This was referenced Oct 19, 2026
Include main through the graph runtime merge. Keep upstream fixes and validate the optimizer package rewrites with 26 unit tests and 81 subtests. Resolve package-script lint without changing rewrite semantics.
Contributor
Author
|
Closing this draft to rebuild the changes from current main rather than the divergent feature branch. Existing branches are preserved; replacement PRs will be linked here. |
Contributor
Author
|
Clean replacement opened directly against main: #2759. The old branch is preserved. Both replacements are independent main-based PRs; end-to-end package execution still needs the companion runtime and ORT operators. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Main integration update
Merged repository
mainat4b34a64af8b763c3bb2d8781b45a26f0c3841118. Published head:f0665f7571eee9fdf8a789c291e4c0d8569441c9. Existing PR bases remain unchanged.qwen-flashconda environment; ranlintrunner -abefore each new commit, followed by clean lint checks. Final branch-vs-main lint passes.This update supersedes earlier notes saying main has not been integrated, but does not remove the draft/build/runtime qualification requirements.
Summary
Add package constructors and guarded generic-contrib graph rewrites, with 26 model-independent optimizer unit tests and integration documentation.
Stack and main target
Depends on #2755.
This is layer 2/2 of the microsoft/onnxruntime-genai stack. The current review base is
asonawane/ported-qwen-graph-runtimeso this PR shows only its incremental group. The final destination is upstreammain: land/reconcile prerequisites first, then rebase and retarget this layer tomain. Keep this PR in draft until that base-specific validation succeeds. Do not merge the entire stack through an unvalidated intermediate base.Cross-repository dependency: GenAI package/runtime integration requires the companion ORT contrib operators.
Validation and blockers
python -m unittest discover -s tools/qwen38_graph -p test_optimize_ported_package.py: 26 tests pass.Scope
Stack navigation
Companion repository stack: