voidLitchi commented on PR #690: URL: https://github.com/apache/tvm-ffi/pull/690#issuecomment-5140482314
> Thanks @voidLitchi . We are increasingly moving towards upstream to bring up the convention which is more robust in terms of ABI version and use the optional plugin mainly for backward compact cases. > > Ideally the best places the DLPack exchange API location definition should be contributed to the upstream package(in this case torch_npu package as part of its dlpack module). > > the overall mechanism is torch_npu get autoloaded as a torch backend. It can bundle this variant of the C DLPack Exchange API in then override `torch.Tensor.__dlpack_c_exchange_api__` during its autoloading process Thanks for the suggestion. We discussed this with the Ascend community, and we agree that torch_npu is the more appropriate home for this capability. We will investigate the integration on the torch_npu side and work toward implementing it there. In the meantime, we would prefer to keep this PR open as a reference while we validate that approach and confirm that it fully covers the relevant downstream use cases. If that path proves sufficient, we can close this PR in favor of the torch_npu implementation. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
