kevinthesun edited a comment on pull request #6449: URL: https://github.com/apache/incubator-tvm/pull/6449#issuecomment-693007551
@t-vi For now IMO it doesn't bring obvious benefit of adding an extra common api to just wrap ```try ... except``` for ```_infer_value```, considering the lack of generalization among different dynamic op handlings. Also we shouldn't just limit to pytorch frontend for this topic since in tf frontend we have a lot of such patterns. Probably you can also check tf frontend to see what kind of general logic we can have for such pattern and discuss with community about the potential improvement. After that we can have a more complete solution for this. Sounds like a plan? In general I feel this topic is a larger one to a common functionality of tvm frontend parser. In this PR probably we can more focus on pytorch specific features, and have a separate discussion about how to better handle dynamic ops in frontend. ---------------------------------------------------------------- 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: us...@infra.apache.org