openvino icon indicating copy to clipboard operation
openvino copied to clipboard

Added torchdynamo backend entry point in package setup

Open ynimmaga opened this issue 1 year ago • 4 comments

With this change, users don't have to add "import openvino.torch" in their applications. Providing torch.compile(model, backend="openvino") automatically imports the necessary modules. This simplifies the user experience and enables PT2 based benchmarking OOB without any changes. .

More information can be found here: https://github.com/pytorch/pytorch/pull/124536

JIRA: CVS-139370

ynimmaga avatar Apr 23 '24 03:04 ynimmaga

import openvino.torch

should we deprecated this approach then? CC @jiwaszki what are the recommendations how to deprecate such approach?

What if OpenVINO is installed as archive? where we don't have setup.py involved to installation process.

ilya-lavrenov avatar Apr 23 '24 07:04 ilya-lavrenov

build_jenkins

mvafin avatar May 02 '24 10:05 mvafin

@ynimmaga Could you also update docs?

mvafin avatar May 02 '24 10:05 mvafin

import openvino.torch

should we deprecated this approach then? CC @jiwaszki what are the recommendations how to deprecate such approach?

What if OpenVINO is installed as archive? where we don't have setup.py involved to installation process.

In this case old approach can be used, right?

mvafin avatar May 02 '24 10:05 mvafin

build_jenkins

suryasidd avatar May 09 '24 07:05 suryasidd

build_jenkins

ilya-lavrenov avatar May 09 '24 09:05 ilya-lavrenov

build_jenkins

suryasidd avatar May 13 '24 15:05 suryasidd