Skip to content

Expose build_function and build_graph in public API#2913

Merged
gramalingam merged 1 commit into
mainfrom
rama/api
May 8, 2026
Merged

Expose build_function and build_graph in public API#2913
gramalingam merged 1 commit into
mainfrom
rama/api

Conversation

@gramalingam
Copy link
Copy Markdown
Collaborator

Add build_function and build_graph to onnxscript's top-level exports, allowing downstream packages (e.g. mobius) to import them directly as onnxscript.build_function / onnxscript.build_graph instead of reaching into onnxscript._internal.builder.

Add build_function and build_graph to onnxscript's top-level exports,
allowing downstream packages (e.g. mobius) to import them directly as
onnxscript.build_function / onnxscript.build_graph instead of reaching
into onnxscript._internal.builder.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Signed-off-by: G Ramalingam <grama@microsoft.com>
Copy link
Copy Markdown
Contributor

Copilot AI left a 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 pull request exposes build_function and build_graph as part of onnxscript’s top-level public API so downstream consumers can import these utilities directly from onnxscript instead of reaching into onnxscript._internal.builder.

Changes:

  • Add build_function and build_graph to onnxscript.__all__.
  • Re-export build_function and build_graph from onnxscript/__init__.py by importing them from onnxscript._internal.builder.

@codecov
Copy link
Copy Markdown

codecov Bot commented May 8, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 72.61%. Comparing base (e8d744f) to head (b71a23e).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2913   +/-   ##
=======================================
  Coverage   72.61%   72.61%           
=======================================
  Files         259      259           
  Lines       31597    31597           
  Branches     2973     2973           
=======================================
  Hits        22945    22945           
  Misses       7643     7643           
  Partials     1009     1009           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@github-project-automation github-project-automation Bot moved this from Todo to Done in ONNX Script Review Board May 8, 2026
@gramalingam gramalingam merged commit 029441f into main May 8, 2026
36 of 37 checks passed
@gramalingam gramalingam deleted the rama/api branch May 8, 2026 17:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

3 participants