comaniac commented on pull request #5656: URL: https://github.com/apache/incubator-tvm/pull/5656#issuecomment-632944262
> @comaniac I just ran this test. The two fused layer test at https://github.com/apache/incubator-tvm/blob/master/tests/python/relay/test_pass_partition_graph.py#L932 works, but the full mobilenet test https://github.com/apache/incubator-tvm/blob/master/tests/python/relay/test_pass_partition_graph.py#L936 seems to stuck during compilation. Is this the error you are seeing? > > I remember this test worked no problem when @zhiics enabled compilation with big constants. I don't know when regression was introduced. Sorry my bad. The original `MergeComposite` pass will recover bind parameters so the composite functions are guaranteed to have the same number of inputs as before. However, composite functions generated by `pattern.partition` still accesses the metadata for constant nodes. That's why the input number doesn't match. I'll try to solve this problem. ---------------------------------------------------------------- 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]
