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 tf.math.argmin for torch, jax, and paddle backends #28342

Merged
merged 4 commits into from
Feb 20, 2024

Conversation

Sai-Suraj-27
Copy link
Contributor

@Sai-Suraj-27 Sai-Suraj-27 commented Feb 20, 2024

PR Description

Fixed tf.math.argmin for torch, jax, and paddle backends

image

Related Issue

Closes #28339
Closes #28340
Closes #28341

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

@Sai-Suraj-27 Sai-Suraj-27 changed the title fix: Fixed tf.math.argmin for 3 torch, jax, and paddle backends fix: Fixed tf.math.argmin for torch, jax, and paddle backends Feb 20, 2024
@ivy-leaves ivy-leaves added the TensorFlow Frontend Developing the TensorFlow Frontend, checklist triggered by commenting add_frontend_checklist label Feb 20, 2024
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.

lgtm
thanks for your continuous contributions @Sai-Suraj-27 :)

@Ishticode Ishticode merged commit 503c29c into Transpile-AI:main Feb 20, 2024
121 of 145 checks passed
Kacper-W-Kozdon pushed a commit to Kacper-W-Kozdon/ivy that referenced this pull request Feb 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
TensorFlow Frontend Developing the TensorFlow Frontend, checklist triggered by commenting add_frontend_checklist
Projects
None yet
3 participants