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: Ivy Failing Test: paddle - elementwise.logical_and #28405

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

MuhammadNizamani
Copy link
Contributor

@MuhammadNizamani MuhammadNizamani commented Feb 23, 2024

PR Description

locally passing on 200 examples
logical_and

Related Issue

Closes #28404

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

Copy link
Contributor

@Ishticode Ishticode left a comment

Choose a reason for hiding this comment

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

Hi @MuhammadNizamani
Thanks for the continued contributions 😊
As the functions seems to have an explicit handling for complex under if paddle.is_complex why are we excluding complex from supported types? and if this is not being handled properly then we should either try to fix the clause or remove it entirely and remove from supported types. In current state it doesn't make sense to me 🤔

@MuhammadNizamani
Copy link
Contributor Author

@Ishticode I am so sorry I overlooked it. I am looking into it.

@Ishticode
Copy link
Contributor

any updates @MuhammadNizamani :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fix Ivy Failing Test: paddle - elementwise.logical_and
3 participants