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.

Sign up for GitHub

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

Jump to bottom

[PT FE] Support aten::masked_select for pytorch models #26162

Merged
merged 6 commits into from
Aug 26, 2024
Merged

[PT FE] Support aten::masked_select for pytorch models #26162

merged 6 commits into from
Aug 26, 2024

Conversation

hub-bla
Copy link
Contributor

@hub-bla hub-bla commented Aug 21, 2024

Details:

  • support aten::masked_select operator

Tickets:

@hub-bla hub-bla requested a review from a team as a code owner August 21, 2024 13:34
@hub-bla hub-bla requested review from slyalin and PiotrKrzem August 21, 2024 13:34
@github-actions github-actions bot added the category: PyTorch FE OpenVINO PyTorch Frontend label Aug 21, 2024
@sys-openvino-ci sys-openvino-ci added the ExternalPR External contributor label Aug 21, 2024
@mvafin
Copy link
Contributor

mvafin commented Aug 22, 2024

build_jenkins

@mvafin mvafin self-assigned this Aug 22, 2024
@mvafin mvafin self-requested a review August 22, 2024 09:48
@ilya-lavrenov ilya-lavrenov added this to the 2024.4 milestone Aug 22, 2024
@mvafin
Copy link
Contributor

mvafin commented Aug 23, 2024

Thank you for the contribution, could you please resolve merge conflicts so we can merge you PR

@hub-bla
Copy link
Contributor Author

hub-bla commented Aug 23, 2024 •
edited
Loading

Done! @mvafin

@mvafin
Copy link
Contributor

mvafin commented Aug 23, 2024

build_jenkins

mvafin
mvafin requested changes Aug 23, 2024
View reviewed changes
Copy link
Contributor

@mvafin mvafin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Build failed. Please add includes for Transpose, NonZero and GatherND

@hub-bla hub-bla requested a review from mvafin August 23, 2024 13:29
@mvafin
Copy link
Contributor

mvafin commented Aug 23, 2024

build_jenkins

@mvafin
Copy link
Contributor

mvafin commented Aug 26, 2024

build_jenkins

@mvafin mvafin enabled auto-merge August 26, 2024 16:30
@mvafin mvafin added this pull request to the merge queue Aug 26, 2024
Merged via the queue into openvinotoolkit:master with commit 2660f85 Aug 26, 2024
141 checks passed
@hub-bla hub-bla mentioned this pull request Aug 27, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mvafin mvafin mvafin approved these changes

@slyalin slyalin Awaiting requested review from slyalin slyalin is a code owner automatically assigned from openvinotoolkit/openvino-pytorch-frontend-maintainers

@PiotrKrzem PiotrKrzem Awaiting requested review from PiotrKrzem PiotrKrzem is a code owner automatically assigned from openvinotoolkit/openvino-pytorch-frontend-maintainers

Assignees

@mvafin mvafin

Labels
category: PyTorch FE OpenVINO PyTorch Frontend ExternalPR External contributor
Projects
None yet
Milestone
2024.4
Development

Successfully merging this pull request may close these issues.
4 participants
@hub-bla @mvafin @ilya-lavrenov @sys-openvino-ci