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

Add numpy.real to Numpy frontend #10483

Merged
merged 14 commits into from
Feb 23, 2023
Merged

Conversation

Joboa
Copy link
Contributor

@Joboa Joboa commented Feb 11, 2023

Close #10478

@ivy-leaves ivy-leaves added the NumPy Frontend Developing the NumPy Frontend, checklist triggered by commenting add_frontend_checklist label Feb 11, 2023
@vaatsalya123
Copy link
Contributor

Hey @Joboa first of all thanks for contributing ,the PR implementation look's good, though few things require a small fix the PR introduced some lint failures which should be easy to fix could you go through them and fix the lint errors thanks!

@Joboa
Copy link
Contributor Author

Joboa commented Feb 21, 2023

@vaatsalya123 , the lint error still persists. I followed the documentation on how to fix it.

@vaatsalya123
Copy link
Contributor

vaatsalya123 commented Feb 22, 2023

Hey @Joboa you need to look through this once more you will understand how to fix the issue basically add spaces at the end of the file:
https://github.com/unifyai/ivy/pull/10483/files#diff-16e00a061bb209b8ff42d7e47a26da2517e96760201de4f6da1daa71100627d3R15

@vaatsalya123
Copy link
Contributor

Yep all look's good thanks!

@vaatsalya123 vaatsalya123 merged commit 84bb1b1 into ivy-llc:master Feb 23, 2023
vedpatwardhan pushed a commit to vedpatwardhan/ivy that referenced this pull request Feb 26, 2023
* Add imag function to Numpy frontend

* Fix imag function for Numpy frontend

* Add real function to Numpy frontend

* Revert "Add real function to Numpy frontend"

This reverts commit 9d1bf8f.

* Add real function to Numpy frontend

* fix real complex numbers function for Numpy frontend
vedpatwardhan pushed a commit to vedpatwardhan/ivy that referenced this pull request Feb 26, 2023
* Add imag function to Numpy frontend

* Fix imag function for Numpy frontend

* Add real function to Numpy frontend

* Revert "Add real function to Numpy frontend"

This reverts commit 9d1bf8f.

* Add real function to Numpy frontend

* fix real complex numbers function for Numpy frontend
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
NumPy Frontend Developing the NumPy Frontend, checklist triggered by commenting add_frontend_checklist
Projects
None yet
Development

Successfully merging this pull request may close these issues.

real
3 participants