Skip to content

Commit

Permalink
Fix typos (onnx#5944)
Browse files Browse the repository at this point in the history
Fix typos in the comments and documentation.

Signed-off-by: Fangjun Kuang <[email protected]>
Co-authored-by: Justin Chu <[email protected]>
  • Loading branch information
csukuangfj and justinchuby authored Feb 29, 2024
1 parent 09d668f commit 7ed9dcb
Show file tree
Hide file tree
Showing 6 changed files with 7 additions and 7 deletions.
4 changes: 2 additions & 2 deletions docs/docsgen/source/intro/python.md
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ to build the graph among the {ref}`l-onnx-make-function`:
(an operator type), its inputs and outputs
- `make_graph`: a function to create an ONNX graph with
the objects created by the two previous functions
- `make_model`: a last function with merges the graph and
- `make_model`: a last function which merges the graph and
additional metadata

All along the creation, we need to give a name to every input,
Expand Down Expand Up @@ -1500,7 +1500,7 @@ object of type `ModelProto`. But it is not. According to
[Protobuf 4, changes](https://developers.google.com/protocol-buffers/docs/news/2022-05-06),
this is no longer possible after version 4 and it is safer to assume the
only way to get a hold on the content is to serialize the model
into bytes, give it the C function, then deserialize it.
into bytes, give it to the C function, then deserialize it.
Functions like `check_model` or
`shape_inference` are calling `SerializeToString` then
`ParseFromString` before checking the model with a C code.
Expand Down
2 changes: 1 addition & 1 deletion onnx/onnx-ml.proto
Original file line number Diff line number Diff line change
Expand Up @@ -212,7 +212,7 @@ message NodeProto {
repeated string output = 2; // namespace Value

// An optional identifier for this node in a graph.
// This field MAY be absent in ths version of the IR.
// This field MAY be absent in this version of the IR.
optional string name = 3; // namespace Node

// The symbolic identifier of the Operator to execute.
Expand Down
2 changes: 1 addition & 1 deletion onnx/onnx-ml.proto3
Original file line number Diff line number Diff line change
Expand Up @@ -212,7 +212,7 @@ message NodeProto {
repeated string output = 2; // namespace Value

// An optional identifier for this node in a graph.
// This field MAY be absent in ths version of the IR.
// This field MAY be absent in this version of the IR.
string name = 3; // namespace Node

// The symbolic identifier of the Operator to execute.
Expand Down
2 changes: 1 addition & 1 deletion onnx/onnx.in.proto
Original file line number Diff line number Diff line change
Expand Up @@ -209,7 +209,7 @@ message NodeProto {
repeated string output = 2; // namespace Value

// An optional identifier for this node in a graph.
// This field MAY be absent in ths version of the IR.
// This field MAY be absent in this version of the IR.
optional string name = 3; // namespace Node

// The symbolic identifier of the Operator to execute.
Expand Down
2 changes: 1 addition & 1 deletion onnx/onnx.proto
Original file line number Diff line number Diff line change
Expand Up @@ -210,7 +210,7 @@ message NodeProto {
repeated string output = 2; // namespace Value

// An optional identifier for this node in a graph.
// This field MAY be absent in ths version of the IR.
// This field MAY be absent in this version of the IR.
optional string name = 3; // namespace Node

// The symbolic identifier of the Operator to execute.
Expand Down
2 changes: 1 addition & 1 deletion onnx/onnx.proto3
Original file line number Diff line number Diff line change
Expand Up @@ -210,7 +210,7 @@ message NodeProto {
repeated string output = 2; // namespace Value

// An optional identifier for this node in a graph.
// This field MAY be absent in ths version of the IR.
// This field MAY be absent in this version of the IR.
string name = 3; // namespace Node

// The symbolic identifier of the Operator to execute.
Expand Down

0 comments on commit 7ed9dcb

Please sign in to comment.