leandron commented on PR #10941: URL: https://github.com/apache/tvm/pull/10941#issuecomment-1096581885
> The recommended approach for bundling the relay format with the parameters/weights wouldn't be solved in this PR, right? You're correct. This would need to be agreed within the TVM community in an RFC. > As recommended by @areusch I could add a warning like `Parameters in the module will be initialized with ones.`. Any ideas for a different message? I think this message is OK given it is the Relay frontend and it has limited scope in terms of published models. However, I would suggest, for the sake of having a better way to provide Relay in future (not only in tvmc, but for TVM in general), to start that discussion above (regarding a Relay format that includes parameters) in the community forum - https://discuss.tvm.apache.org - what do you think? -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
