kimm240 commented on PR #18515: URL: https://github.com/apache/tvm/pull/18515#issuecomment-3645650266
I completely agree. However, implementing a full, general Data Flow Reconnection logic requires significant development time and extensive testing. Given that the current PR is functionally complete and stable, and it addresses two very common patterns (Clipping and ReLU), could we proceed with merging the current PR first? This would allow the community to gain the immediate performance benefit. I commit to delivering the generalized implementation as the next dedicated PR. And, I am also happy to discuss the generalized logic with you before implementing the code. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
