Skip to content

Commit 701e8ba

Browse files
authored
Update CHANGELOG.rst (#112)
1 parent 45a60a1 commit 701e8ba

File tree

3 files changed

+17
-14
lines changed

3 files changed

+17
-14
lines changed

CHANGELOG.rst

+9-2
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,13 @@
77
Change log
88
==========
99

10+
0.9.2 (2023-10-20)
11+
------------------
12+
13+
**Other changes**
14+
15+
- Fix a deprecation warning from one of Spox's dependencies.
16+
1017
0.9.1 (2023-10-05)
1118
------------------
1219

@@ -19,7 +26,7 @@ Change log
1926
0.9.0 (2023-06-12)
2027
------------------
2128

22-
**New features**
29+
**New feature**
2330

2431
- The opset ``ai.onnx@19`` (ONNX 1.14) is now shipped with Spox.
2532

@@ -36,7 +43,7 @@ Change log
3643
0.8.1 (2023-05-xx)
3744
------------------
3845

39-
**Bug fixes**
46+
**Bug fixe**
4047

4148
- An explicit error is now raised when local subgraph arguments are leaked to an outer scope. This may happen when the subgraph callback uses side effects saving local variables, which would produce later a confusing error message.
4249

src/spox/_graph.py

+4-6
Original file line numberDiff line numberDiff line change
@@ -150,15 +150,13 @@ def __repr__(self):
150150

151151
def __post_init__(self):
152152
if any(not isinstance(var, Var) for var in self._results.values()):
153-
raise TypeError(
154-
f"Graph results must be Vars, not {set(type(obj) for obj in self._results.values()) - {Var}}."
155-
)
153+
seen_types = {type(obj) for obj in self._results.values()}
154+
raise TypeError(f"Graph results must be Vars, not {seen_types - {Var}}.")
156155
if self._arguments is not None and any(
157156
not isinstance(var, Var) for var in self._arguments
158157
):
159-
raise TypeError(
160-
f"Graph results must be Vars, not {set(type(obj) for obj in self._arguments) - {Var}}."
161-
)
158+
seen_types = {type(obj) for obj in self._arguments}
159+
raise TypeError(f"Build outputs must be Vars, not {seen_types - {Var}}.")
162160

163161
def with_name(self, name: str) -> "Graph":
164162
"""Return a Graph with its name set to ``name``."""

src/spox/_public.py

+4-6
Original file line numberDiff line numberDiff line change
@@ -98,13 +98,11 @@ def build(inputs: Dict[str, Var], outputs: Dict[str, Var]) -> onnx.ModelProto:
9898
>>> model = build({'a': a, 'b': b, 'c': c}, {'r': q})
9999
"""
100100
if not all(isinstance(var, Var) for var in inputs.values()):
101-
raise TypeError(
102-
f"Build inputs must be Vars, not {set(type(obj) for obj in inputs.values()) - {Var} }."
103-
)
101+
seen_types = {type(obj) for obj in inputs.values()}
102+
raise TypeError(f"Build inputs must be Vars, not {seen_types - {Var}}.")
104103
if not all(isinstance(var, Var) for var in outputs.values()):
105-
raise TypeError(
106-
f"Build outputs must be Vars, not {set(type(obj) for obj in outputs.values()) - {Var} }."
107-
)
104+
seen_types = {type(obj) for obj in outputs.values()}
105+
raise TypeError(f"Build outputs must be Vars, not {seen_types - {Var}}.")
108106
if not all(isinstance(var._op, Argument) for var in inputs.values()):
109107
raise TypeError(
110108
"Build inputs must be `Var`s constructed using the `spox.argument` function. "

0 commit comments

Comments
 (0)