RadeonML icon indicating copy to clipboard operation
RadeonML copied to clipboard

Multiple Outputs, get Number and get Names

Open Armin234 opened this issue 4 years ago • 7 comments

Hi,

my model has multiple outputs, and it would be fine if there are functions for get the number of outputs and there names. Like it is already implemented for the inputs.

Armin234 avatar Jun 19 '20 08:06 Armin234

Hi, @Armin234 , we'll introduce such feature in the next update.

JohnBravo08am avatar Jun 19 '20 12:06 JohnBravo08am

The last graph node is considered to be an output node by default. However any other network node can be marked as output by a client. That's why the API has the setting function only. The library just "doesn't know" which node outputs do you need.

dtarakanov1 avatar Jun 22 '20 13:06 dtarakanov1

Unlike output nodes, the input nodes are fixed in a graph (TF Placeholder ops / ONNX initializers / RadeonML RML_PLACEHOLDER_OP), so Get function for them makes large sense.

dtarakanov1 avatar Jun 22 '20 14:06 dtarakanov1

@dtarakanov1
when i design a network there could be more then "one" last graph node. When I need additional layer as an output, I design the network with an additional last graph node. So the possibility to setting every layer as an output is a nice feature, but not really needed. But getting the Informations of the "designed" outputs, from my point of view is more useful, specifically when the application should be able to work with different models.

Armin234 avatar Jun 23 '20 14:06 Armin234

Hi @Armin234 ! So, you propose to consider all nodes that have input edges but don't have output edges as output by default, am I right ?

achernigin1987 avatar Jun 25 '20 06:06 achernigin1987

Hi @achernigin1987 ! Yes, from my point of view, every nodes output edges which is not connected to a input edges from another node should be a output by default.

Armin234 avatar Jun 25 '20 07:06 Armin234

@Armin234 It makes sense. We'll add it in the following release

achernigin1987 avatar Jun 25 '20 09:06 achernigin1987