areusch commented on PR #10941: URL: https://github.com/apache/tvm/pull/10941#issuecomment-1104577966
hm. for (1), it seems like we could do something like: - add a parameter which you have to pass for Relay models indicating the inputs - accept params.npz and treat everything not in there as inputs - a combination of both for (2), that seems like an oversight on Model Library Format part. it seems like we should make tvmc validate the version numbers in relay models. Perhaps we should change Model Library Format? i suppose it's also a bit of a burden on users authoring relay to figure out the presently-supported version, and they probably always just mean "the current version." so maybe allowing some implicitness there makes sense. @leandron what are your thoughts? -- 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]
