Skip to content
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] [ARM]: Implement CPU plugin just-in-time emitter for Erf operation #27499

Open
a-sidorova opened this issue Nov 11, 2024 · 11 comments · May be fixed by #28176
Open

[Good First Issue] [ARM]: Implement CPU plugin just-in-time emitter for Erf operation #27499

a-sidorova opened this issue Nov 11, 2024 · 11 comments · May be fixed by #28176
Assignees
Labels
category: CPU OpenVINO CPU plugin good first issue Good for newcomers platform: arm OpenVINO on ARM / ARM64

Comments

@a-sidorova
Copy link
Contributor

Context

JIT Emitters are part of code generation feature (a.k.a. tensor compiler) that automatically produces highly-efficient optimized fused subgraph binary code. Each emitter implements specific operation from low level OpenVINO dialect.

Prerequisites

Recommended to use ARM CPU based platform for development (e.g. Mac, Raspberry Pi etc). Emulators (e.g. QEMU) is still an option, but not that convenient, especially for final performance evaluation.

What needs to be done?

Before emitter implementation, please, modify tests to be sure that developed functionality is covered by test:

Tests

Tests are disabled in default build, so ensure to add -DENABLE_TESTS=ON into cmake command.

GoogleTest is used for testing. CPU functional test target is ov_cpu_func_tests. You can use GoogleTest filter:

./bin/[platform]/[build_type]/ov_cpu_func_tests --gtest_filter="*smoke*Activation*Erf*"

Example Pull Requests

Resources

Contact points

@a-sidorova, @dmitry-gorokhov

@a-sidorova a-sidorova added good first issue Good for newcomers category: CPU OpenVINO CPU plugin platform: arm OpenVINO on ARM / ARM64 labels Nov 11, 2024
@github-project-automation github-project-automation bot moved this to Contributors Needed in Good first issues Nov 11, 2024
Copy link
Contributor

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

@geeky33 geeky33 removed their assignment Nov 12, 2024
@geeky33
Copy link
Contributor

geeky33 commented Nov 13, 2024

.take

Copy link
Contributor

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

@geeky33 geeky33 removed their assignment Nov 14, 2024
@VectorNd
Copy link

.take

Copy link
Contributor

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

Copy link
Contributor

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.

@mlukasze
Copy link
Contributor

hey @VectorNd how things going? Do you need any help?

@shivam5522
Copy link
Contributor

Can I work on this issue? If no one is working on it.

@dmitry-gorokhov dmitry-gorokhov moved this from Assigned to Contributors Needed in Good first issues Dec 12, 2024
@dmitry-gorokhov
Copy link
Contributor

dmitry-gorokhov commented Dec 12, 2024

I moved the issue to "Contributors needed" state due to lack of activity. @shivam5522 feel free to take it :)

@shivam5522
Copy link
Contributor

.take

Copy link
Contributor

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

@mlukasze mlukasze moved this from Contributors Needed to Assigned in Good first issues Dec 12, 2024
@a-sidorova a-sidorova moved this from Assigned to In Review in Good first issues Dec 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: CPU OpenVINO CPU plugin good first issue Good for newcomers platform: arm OpenVINO on ARM / ARM64
Projects
Status: In Review
Development

Successfully merging a pull request may close this issue.

6 participants