Skip to content

Allow opset_version to be set explicitly when exporting #2615

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.

Sign up for GitHub

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

Merged
justinchuby merged 5 commits into from
Oct 14, 2025
Merged

Allow opset_version to be set explicitly when exporting #2615

Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
e7e632d
Allow `opset_version` to be set explicitly when exporting
NoRaincheck Oct 9, 2025
a6cd17e
Update irbuilder.py
NoRaincheck Oct 9, 2025
9ac844e
Update irbuilder.py
NoRaincheck Oct 11, 2025
28cf729
fix style
NoRaincheck Oct 11, 2025
75baf54
fix lint
NoRaincheck Oct 13, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 5 additions & 2 deletions onnxscript/irbuilder.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -321,6 +321,7 @@ def to_model_proto(
input_types: Optional[Sequence[ONNXType]] = None,
output_types: Optional[Sequence[ONNXType]] = None,
value_infos: dict[str, ONNXType] | None = None,
opset_version: int | None = None,
**kwargs,
) -> onnx.ModelProto:
"""Converts this instance into a `onnx.ModelProto`.
Expand All @@ -336,6 +337,8 @@ def to_model_proto(
are set to be of the corresponding type in this list.
value_infos: A dictionary mapping intermediate variable names to ONNX types.
Used to set value_info for intermediate variables.
opset_version: The standard opset version to use for the model if it
cannot be inferred. Otherwise defaults to the current opset version.
kwargs: Additional parameters given to function :func:`onnx.helper.make_model`.

Returns:
Expand Down Expand Up @@ -393,8 +396,8 @@ def to_proto(f):

if "" not in opsets:
# No operator is using the standard opset.
# A default value is given.
opsets[""] = onnx_opset_version()
# Use the specified version if provided or the default value.
opsets[""] = opset_version if opset_version is not None else onnx_opset_version()

if "ir_version" not in kwargs:
kwargs["ir_version"] = select_ir_version(opsets[""])
Expand Down
Loading