[GitHub] [tvm] anwang2009 opened a new pull request #9489: Add default for split op

[GitBox <gi...@apache.org>]

anwang2009 opened a new pull request #9489:
URL: https://github.com/apache/tvm/pull/9489


   When the attr.split length is 1 or less, we should fall back to default split behavior -- dividing evenly over the axes, because the split importer drops the first item in the attr.split list. The shape func for split assumes at least one item in the split indices. 


-- 
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

[GitHub] [tvm] mbrookhart commented on pull request #9489: Add default for split op

[GitBox <gi...@apache.org>]

mbrookhart commented on pull request #9489:
URL: https://github.com/apache/tvm/pull/9489#issuecomment-965684708


   Can you add a test that hits this case? Otherwise LGTM


-- 
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

[GitHub] [tvm] junrushao1994 merged pull request #9489: Add default for split op

[GitBox <gi...@apache.org>]

junrushao1994 merged pull request #9489:
URL: https://github.com/apache/tvm/pull/9489


   


-- 
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