You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tvm.apache.org by "vvchernov (via GitHub)" <gi...@apache.org> on 2023/07/24 11:00:52 UTC

[GitHub] [tvm] vvchernov commented on pull request #15386: [Relay] Fix an adaptive_max_pool1d operator conversion bug

vvchernov commented on PR #15386:
URL: https://github.com/apache/tvm/pull/15386#issuecomment-1647692428

   From one hand LGTM, from another one it confuses me due to fix for specific op was done on very high level (and CI test was constructed for specific op not for all cases). It looks like it has been assumed that outputs exist at any `ret_name in ret_names`. I need time to analyze it by my-self or explanation what is wrong with the initial assumption.


-- 
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: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org