RuRo commented on pull request #11434: URL: https://github.com/apache/incubator-mxnet/pull/11434#issuecomment-633206491
> However, name like `ignore_reinit` doesn't seem obvious to me that it suppresses warnings. I'd be more glad to recommend accepting this if we could come up with a more intuitive name. I think, the option was called `ignore_reinit` for symmetry with the existing `force_reinit` option. Perhaps something like `skip_reinit` can better convey, what the proposed option does? The reasoning would be that by default it is ambiguous, what to do with already initialized params (hence the warning). Specifying either `force_reinit` or `skip_reinit` would then explicitly specify, what behavior the user wants and so the warning would no longer be required. ---------------------------------------------------------------- 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]
