Add implicit NumPy conversion for dpctl.tensor.usm_ndarray types#2131
Add implicit NumPy conversion for dpctl.tensor.usm_ndarray types#2131icfaust wants to merge 8 commits intoIntelPython:masterfrom
Conversation
|
@ndgrigorian I am unable to check the jenkins CI checks. Is there any way someone from your team can help me? |
I can help with that, but there are still concerns with this PR that need to be addressed, some of which I mentioned in the issue If this PR were modified to accept an environment variable to permit implicit conversion, that would be acceptable, but implicit conversion under all circumstances should not be allowed |
|
Implicit conversion means GPU resident array data is going to be copied to host without explicit user control. This introduces source of hard to pin-point performance bottlenecks. CuPy does not do it for a good reason. Computation on GPU using data-parallel algorithms is generally quite a lot faster. So CuPy promotes coercion of I hope this change does not get merged, it is a regression |
The functionality introduced in #1964 can be better optimized by moving code from
_copy_utilsto theusm_ndarrayitself. This will make seamless integration into other larger codebases like scikit-learn, where use ofasarrayis common. It attempts to solve #2129