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 zeros_like and ones_like methods to ArrayLikeFactory subclasses #121

Closed
wants to merge 4 commits into from

Conversation

GiulioRomualdi
Copy link
Member

@GiulioRomualdi GiulioRomualdi commented Jan 9, 2025

⚠️ This PR is on top of #120

This pull request includes several improvements and additions to the CasadiLike, JaxLike, NumpyLike, and TorchLike classes and their respective tests. This PR adds new methods for creating zero and one matrices and updates the corresponding tests.

Key changes:

Enhancements to CasadiLike, JaxLike, NumpyLike, and TorchLike classes:

  • Added zeros_like and ones_like static methods to create zero and one matrices based on the input matrix type. (src/adam/casadi/casadi_like.py, src/adam/jax/jax_like.py, src/adam/numpy/numpy_like.py, src/adam/pytorch/torch_like.py) [1] [2] [3] [4]

Updates to SpatialMath class:

  • Added zeros_like and ones_like abstract methods to the ArrayLikeFactory abstract class. (src/adam/core/spatial_math.py)

Test additions:

  • Added new test functions to validate the behavior of zeros_like and ones_like methods, as well as ellipsis handling in CasadiLike, JaxLike, NumpyLike, and TorchLike classes. (tests/test_casadi.py, tests/test_jax.py, tests/test_numpy.py, tests/test_pytorch.py) [1] [2] [3] [4]

📚 Documentation preview 📚: https://adam-docs--121.org.readthedocs.build/en/121/

@GiulioRomualdi GiulioRomualdi self-assigned this Jan 9, 2025
@GiulioRomualdi GiulioRomualdi requested a review from Giulero January 9, 2025 12:10
@GiulioRomualdi
Copy link
Member Author

I will close this PR for the time being. We can open it as soon as we start again with this activity

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

Successfully merging this pull request may close these issues.

1 participant