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 layer conv1d_transpose for paddle backend #20899

Conversation

yopknopixx
Copy link
Contributor

This PR adds conv1d_transpose layer to the paddle backend and closes #20896

@ivy-leaves ivy-leaves added the PaddlePaddle Backend Developing the Paddle Paddle Backend. label Jul 27, 2023
@MahmoudAshraf97 MahmoudAshraf97 removed their assignment Aug 6, 2023
Copy link
Contributor

@Daniel4078 Daniel4078 left a comment

Choose a reason for hiding this comment

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

Please don't forget that you need to create a test for this new function you implement. Thanks

@yopknopixx
Copy link
Contributor Author

Hey @Daniel4078! I'm unsure of how to create a test for this since this is a backend function.

@Daniel4078
Copy link
Contributor

Hey @Daniel4078! I'm unsure of how to create a test for this since this is a backend function.

Sorry, I though you are adding a completely new function. In this case, there is already a test for the function here (https://github.com/unifyai/ivy/blob/a592ae91c38c0a311849bebf1930215f22b364a6/ivy_tests/test_ivy/test_functional/test_nn/test_layers.py#L694).

@yopknopixx
Copy link
Contributor Author

That was my understanding. Is this ready to be merged?

@Daniel4078 Daniel4078 merged commit 425aba1 into ivy-llc:master Aug 8, 2023
260 of 266 checks passed
Sarvesh-Kesharwani pushed a commit to Sarvesh-Kesharwani/ivy_Sark42 that referenced this pull request Aug 10, 2023
@yopknopixx yopknopixx deleted the add_layer_paddle_backend_conv1d_transpose branch September 13, 2023 03:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PaddlePaddle Backend Developing the Paddle Paddle Backend.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

conv1d_transpose
4 participants