ysh329 opened a new pull request, #18180: URL: https://github.com/apache/tvm/pull/18180
> The `resize2d` operator in Relax already covers the functionality, so there’s no need to introduce a dedicated upsampling op at the Relax level. 1. Model-wise, only YOLO v3 Tiny currently uses this op, and I haven’t seen any other models that require it. The op originates from Darknet; 2. although it becomes an ONNX `Upsample` op during conversion, the node name still ends with `/resize`. From ONNX version 10 onward, the `Upsample` op is no longer supported. 3. `resize2d` provides a superset of `upsample` (2-D) capabilities, and end-to-end model results are correct, so `resize2d` fully covers the need. 4. During ONNX-to-Relay conversion, the old frontend maps the op to either `upsampling` or `upsampling3d` based on the input rank. For the 3-D case, Relax could translate `upsampling3d` to `resize3d` (TOPI already has the resze3d implementation), but Relax currently lacks a `resize3d` operator. Therefore, if we ever need `upsampling3d` in the future, the better path would be to add the more capable `resize3d` operator at the Relax op level. -- 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]
