RuRo commented on pull request #18413: URL: https://github.com/apache/incubator-mxnet/pull/18413#issuecomment-647210147
**(Disclamer)** I didn't look too deeply into the reasons for these changes, so I may be wrong, but... **(Disclamer)** In my opinion, the distinction between the "unique/unreadable" parameter names and the "human-readable" parameter names is quite confusing. It's not obvious, where should you expect which kind of parameter name. For example, currently `.collect_params()` seems to return human-readable names as keys, but the parameter values themselves have "unique" `.name`s. Exceptions, errors and warnings (for example the "Parameter has not been initialized." error) currently seem to print the "unique" parameter name, which is really unhelpful for debugging purposes, since it doesn't tell you, where is the uninitialized parameter located. I am not very familiar with the inner workings of Gluon Parameters, but it really feels like there has to be a better way. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
