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

feat: add kaiser_window to pytorch frontend #26788

Merged
merged 4 commits into from
Oct 13, 2023

Conversation

Mainakdeb
Copy link
Contributor

@Mainakdeb Mainakdeb commented Oct 8, 2023

PR Description

add kaiser_window to pytorch frontend

Related Issue

Close #26783

Checklist

  • Did you add a function?
  • Did you add the tests?
  • Did you run your tests and are your tests passing?
  • Did pre-commit not fail on any check?
  • Did you follow the steps we provided?

Socials

https://twitter.com/MainakDeb19

@github-actions
Copy link
Contributor

github-actions bot commented Oct 8, 2023

Thank you for this PR, here is the CI results:


This pull request does not result in any additional test failures. Congratulations!

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

PR Compliance Checks Passed!

@ivy-leaves ivy-leaves added the PyTorch Frontend Developing the PyTorch Frontend, checklist triggered by commenting add_frontend_checklist label Oct 9, 2023
Copy link
Contributor

@Sam-Armstrong Sam-Armstrong left a comment

Choose a reason for hiding this comment

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

@Mainakdeb looks good, please could you address the comment I left :)

ivy/functional/frontends/torch/spectral_ops.py Outdated Show resolved Hide resolved
@Mainakdeb
Copy link
Contributor Author

@Mainakdeb looks good, please could you address the comment I left :)

Thanks @Sam-Armstrong, updated 🙂

@Sam-Armstrong
Copy link
Contributor

@Mainakdeb please could you also add it to the ivy method call within the frontend function and the test? 🙂

@Mainakdeb
Copy link
Contributor Author

@Sam-Armstrong , many thanks for the pointers🙂, updated test (../test_spectral_ops.py) and ivy method call (../spectral_ops.py)

Copy link
Contributor

@Sam-Armstrong Sam-Armstrong left a comment

Choose a reason for hiding this comment

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

lgtm, thanks for the contribution @Mainakdeb!

@Sam-Armstrong Sam-Armstrong merged commit 9c52b46 into ivy-llc:main Oct 13, 2023
p1utoze pushed a commit to p1utoze/ivy that referenced this pull request Oct 13, 2023
p1utoze pushed a commit to p1utoze/ivy that referenced this pull request Oct 13, 2023
druvdub pushed a commit to druvdub/ivy that referenced this pull request Oct 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PyTorch Frontend Developing the PyTorch Frontend, checklist triggered by commenting add_frontend_checklist
Projects
None yet
Development

Successfully merging this pull request may close these issues.

kaiser_window
3 participants