Hzfengsy commented on PR #17497: URL: https://github.com/apache/tvm/pull/17497#issuecomment-2445909111
Thanks for the contribution! First, it's not a typical **bug**. `(10)` is just an integer rather than a list based on the Python syntax, while `[10]` and `(10, )` are list and tuples. Here, we request a list as input, so the error is expected (based on the current implementation) Second, it should be a valuable sugar for end users. But please: 1. consider supporting not only integers but also prim expr, symbolic shapes etc. 2. please add tests in the parser test file -- 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]
