sony / nnabla

Neural Network Libraries
https://nnabla.org/
Apache License 2.0
2.73k stars 334 forks source link

fix bug of network expander for duplicated name #1239

Closed TomonobuTsujikawa closed 11 months ago

TomonobuTsujikawa commented 11 months ago

This PR tends to fix bug of network expander, which might produce duplicated variable name.

For cpp expander, the repeat block will be expanded as follows:

convolution_{repeat_start}_1/conv/b  --> convolution_3_1/conv/b

But python expander will expand it as follows:

convolution_{repeat_start}_1/conv/b  --> convolution_3[1]/conv/b

This caused problems with reading the wrong parameters or not being able to read parameters.