Skip to content

[Good First Issue]: Support request for aten::pop and aten::reverse #29630

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

Open
geeky33 opened this issue Mar 22, 2025 · 15 comments · May be fixed by #30101
Open

[Good First Issue]: Support request for aten::pop and aten::reverse #29630

geeky33 opened this issue Mar 22, 2025 · 15 comments · May be fixed by #30101
Assignees
Labels
good first issue Good for newcomers no_stale Do not mark as stale

Comments

@geeky33
Copy link
Contributor

geeky33 commented Mar 22, 2025

Context

Context
Operations list to be supported

aten::pop 
aten::reverse

What needs to be done?

Enable the support system for list of operations in Pytorch Frontend. (PT FE)

Example Pull Requests

No response

Resources

Contact points

@mvafin

Ticket

No response

@geeky33 geeky33 added good first issue Good for newcomers no_stale Do not mark as stale labels Mar 22, 2025
@github-project-automation github-project-automation bot moved this to Contributors Needed in Good first issues Mar 22, 2025
Copy link
Contributor

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

@hsiangwang0129
Copy link

Hi @geeky33 ,

I am interested in working on this issue as part of my preparation for GSoC 2025. I would like to take ownership of this task.

I am currently planning to apply for OpenVINO’s GSoC program and believe this issue would be a great opportunity to get more familiar with the codebase.

Could you please assign this issue to me?

Thank you, and looking forward to your feedback!

@johnrhimawan
Copy link

johnrhimawan commented Mar 22, 2025

Hi @mvafin ! I'd love to have the opportunity to work on this for GSoC 2025. Would it be possible to assign this task for me?

I took a quick look at the code base and from what I understand I think what needs to be done is to add the aten::pop and aten::reverse operations by adding the functionality via pop.cpp and reverse.cpp files here (and to write the relevant tests).

I went through previous issues and I found this PR I could use for reference for my starting point

Looking forward to your feedback!

@johnrhimawan
Copy link

.take

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.

@labyrinth-ssr
Copy link

.take

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.

@rohan-bhxtia
Copy link

Hi, I would like to work on this issue as part of my GSoC prerequisite task.

I have experience with Python and couple of it's libraries. I'm currently diving deeper into OpenVINO and PyTorch internals. I'm also committed to learning fast and collaborating closely with the community.

Please assign me this issue, and I'll start discussing the approach with the developers and refer to the contribution guide right away.

Thanks!

@rohan-bhxtia
Copy link

.take

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 mlukasze moved this from Contributors Needed to Assigned in Good first issues Mar 24, 2025
@p-wysocki
Copy link
Contributor

Reopening the issue due to inactivity.

@p-wysocki p-wysocki moved this from Assigned to Contributors Needed in Good first issues Apr 4, 2025
@geeky33
Copy link
Contributor Author

geeky33 commented Apr 5, 2025

.take

Copy link
Contributor

github-actions bot commented Apr 5, 2025

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

@p-wysocki p-wysocki moved this from Contributors Needed to Assigned in Good first issues Apr 6, 2025
@p-wysocki
Copy link
Contributor

Hello @geeky33, are you still working on that task?

@geeky33
Copy link
Contributor Author

geeky33 commented Apr 13, 2025

@p-wysocki yes sir kindly check my PR
thank you.

@mlukasze mlukasze linked a pull request Apr 14, 2025 that will close this issue
@mlukasze mlukasze moved this from Assigned to In Review in Good first issues Apr 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers no_stale Do not mark as stale
Projects
Status: In Review
Development

Successfully merging a pull request may close this issue.

6 participants