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

fix: Fixed passing of un-necessary device argument in ivy.blackman_window() #27897

Merged
merged 1 commit into from
Jan 11, 2024

Conversation

Sai-Suraj-27
Copy link
Contributor

PR Description

In the following function call, the argument device is passed,
https://github.com/unifyai/ivy/blob/06508027180ea29977b4cafd316d536247cb5664/ivy/data_classes/array/experimental/creation.py#L170-L172
From the actual definition of blackman_window there is no argument device
https://github.com/unifyai/ivy/blob/06508027180ea29977b4cafd316d536247cb5664/ivy/functional/ivy/experimental/creation.py#L684-L690

Related Issue

Closes #27896

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

@ivy-leaves ivy-leaves added Array API Conform to the Array API Standard, created by The Consortium for Python Data API Standards Ivy API Experimental Run CI for testing API experimental/New feature or request labels Jan 11, 2024
@KareemMAX KareemMAX merged commit 21df3f0 into Transpile-AI:main Jan 11, 2024
401 of 413 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Array API Conform to the Array API Standard, created by The Consortium for Python Data API Standards Ivy API Experimental Run CI for testing API experimental/New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unnecessary key-word argument device in ivy.blackman_window() function call
4 participants