Gorgonia backend does not handle model with several root nodes about onnx-go HOT 4 CLOSED

Good catch!

The model has multiple "root" nodes which is not handled yet by the backend. This is an extract of the model rendered with netron:

Two things to do:

• setting a more explicit error message
• implement the feature

I will split this into two issues.

from onnx-go.

I am changing the title of this issue; I have created a "project" to follow the implementation steps to make the SSD model working.

Note: I have analyzed the model, and it will require a lot of effort to make it work with gorgonia.

from onnx-go.

There is a work-in-progress with the branch issue-48 but it needs some more investigation.

The actual design of "gorgonnx" does not handle operators with multiple output.

the signature of the apply method will probably require some changes:

type operator interface {
        // apply analyse the graph to find the children of the node
        // then extract its gorgonia.Node references
        // and assign the result of the operation to the node n
        apply(g *Graph, n *Node) error
        // init the operator with name and attributes as carried by the onnx.Operator
        init(o onnx.Operation) error
}

from onnx-go.

PR #115 fixes this error, but it introduces a breaking change.
The OperationCarrier interface needed to be changed to handle nodes with multiple outputs.

This will require to change any backend implementation.
As far as I know, the only backend implemented so far is Gorgonia (via gorgonnx), and it has been patched.

from onnx-go.