-
Notifications
You must be signed in to change notification settings - Fork 2.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Good First Issue]: Support aten::atan2 for pytorch models #20575
Comments
Hi, I am new to open-source contribution. Could you point me to a resource that tells me how to compile and contribute to this project? |
Hi, thank you for assigning the task. However, I am not sure how I would begin the implementation. |
Hi @stncil |
Hi @stncil, are you still working on this issue or can I return it to be picked up by other contributors? |
Hi, I won't be able to complete the task. Please return it to the open issues. Apologies for the delay in response. |
.take |
Thank you for looking into this issue! Please let us know if you have any questions or require any help. |
Hello @sydarb, are you still working on this issue? 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. |
.take |
Thank you for looking into this issue! Please let us know if you have any questions or require any help. |
Hi @p-wysocki @mvafin, I am facing a little issue while building openvino on my macbook (apple silicon). I have followed the apple silicon related guide so far (https://github.com/openvinotoolkit/openvino/blob/master/docs/dev/build_mac_arm.md). cmake -G "Ninja Multi-Config" -DENABLE_SYSTEM_PUGIXML=ON -DENABLE_SYSTEM_SNAPPY=ON -DENABLE_SYSTEM_PROTOBUF=ON .. The issue comes up with the above step when I am running cmake to fetch project dependencies and create build rules. After receiving a few warnings related to which functionality is turned off (expected), I get the following error message:
Is this a known issue while building with Apple Silicon? Or is there some other issue? Please let me know if you have seen this before. Would be very grateful! |
Hi @sahilpmehra you missed several steps with dependency installation. Specifically, you have missed:
Note, it can also be installed via |
Hi @sahilpmehra, are you still working on that issue? |
Hey @mvafin made the recommended changes please review once. |
.take |
Thank you for looking into this issue! Please let us know if you have any questions or require any help. |
hey @Increshi do you need any support? |
.take |
Thank you for looking into this issue! Please let us know if you have any questions or require any help. |
hi @mlukasze :) , i just finish working on it and open a pr, should i do something or just wait for review. |
no, reviewers received notifications, no we wait ;) |
@mvafin will be your contact point to solve those issues and answers questions. |
okay, thank you. |
hi @mvafin i just encountered some issues in the CI pytorch layer tests. in the test logs it reports errors about "no conversion rules for aten.atan2.default", but there is only aten::atan2 operator. i try to reproduce using the command to run pytest locally on my mac but it runs normally. could you provide some hints about where could this "aten.atan2.default" come from or any ways to inspect it? thanks a lot! |
### Details: - add atan2 operator and unit tests ### Tickets: - [[Good First Issue]: Support aten::atan2 for pytorch models](#20575) --------- Co-authored-by: Michal Lukaszewski <[email protected]> Co-authored-by: Maxim Vafin <[email protected]>
### Details: - add atan2 operator and unit tests ### Tickets: - [[Good First Issue]: Support aten::atan2 for pytorch models](openvinotoolkit#20575) --------- Co-authored-by: Michal Lukaszewski <[email protected]> Co-authored-by: Maxim Vafin <[email protected]>
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?
tests/layer_tests/pytorch_tests
. Please consider different data types, but keep reasonable number of test casesExample Pull Requests
#18998
Resources
Contact points
@openvinotoolkit/openvino-pytorch-frontend-maintainers
Ticket
TBD
The text was updated successfully, but these errors were encountered: