mvafin commented 10 months ago

Context

OpenVINO component responsible for support of PyTorch models is called as PyTorch Frontend (PT FE). PT FE converts a model represented as TorchScript model to a model in OpenVINO opset.

What needs to be done?

Implement conversion rule and/or transformation to support the new operation.
Implement operation tests in tests/layer_tests/pytorch_tests. Please consider different data types, but keep reasonable number of test cases
Enable the model in model_hub_tests: https://github.com/openvinotoolkit/openvino/blob/6a50ea3c74e19c2a1e011a43775d998c570de818/tests/model_hub_tests/torch_tests/hf_transformers_models#L102

Example Pull Requests

17999

Resources

This will enable the google/fnet-base model from Hugging Face. You can use the following script to reproduce the issue:

from transformers import AutoTokenizer, AutoModel
from openvino import convert_model

tokenizer = AutoTokenizer.from_pretrained("google/fnet-base")
model = AutoModel.from_pretrained("google/fnet-base", torchscript=True)
text = "Replace me by any text you'd like."
encoded_input = tokenizer(text, return_tensors='pt', padding='max_length', truncation=True, max_length=512)
inputs = dict(encoded_input)

m = convert_model(model, example_input=inputs)

Contact points

@openvinotoolkit/openvino-pytorch-frontend-maintainers

Ticket

TBD

siddhant-0707 commented 10 months ago

.take PR #21705

github-actions[bot] commented 10 months ago

Thank you for looking into this issue! Please let us know if you have any questions or require any help.

github-actions[bot] commented 9 months ago

Thanks for being interested in this issue. It looks like this ticket is already assigned to a contributor. Please communicate with the assigned contributor to confirm the status of the issue.

p-wysocki commented 9 months ago

I am happy to announce that we have created a channel dedicated to Good First Issues support on our Intel DevHub Discord server! Join it to receive support, engage in discussions, ask questions and talk to OpenVINO developers.

mvafin commented 7 months ago

@siddhant-0707 Hi, do you have plans to finish PR, it will be removed soon, due to inactivity?

mlukasze commented 6 months ago

awaiting response from contributor. further actions based on his decision.

openvinotoolkit / openvino

[Good First Issue]: Support `aten::fft_fftn` -> `aten::real` pattern #21665