Diego Ferigo

Results 370 comments of Diego Ferigo

Awesome, if ruff gains support of a static type checker, it'd be great exploiting it. We don't need many features, something quite basic could work at least in the beginning.

Awesome, thanks @flferretti for this PR. I'd expect the following consequences: - With static velocity representations, functions compiled passing different values are different for JAX. Only the section of the...

> The AD tests take a bit longer to complete. I guess that now AD needs to propagate gradients through all possible branches instead of just one. And yes, this...

Having more examples that are automatically tested in CI would be great (and in our case, after #187, readthedocs would fail to render if a notebook is not running). I...

This is the same security alert that triggered https://github.com/diegoferigo/phd-thesis/pull/6. Should we take advantage and also update the upload-artifact? I'd suggest to just pin the major release to `v4` for both.

Failures in CI seem unrelated to the changes of this PR. The [`osqp_test.py`](https://github.com/google/jaxopt/blob/4a50198711e53ed0c25f2be5394825092c2427db/tests/osqp_test.py), only test affected by the proposed changes, is passing.

Friendly ping @Algue-Rythme, any comment on this tiny PR?

The error is the following: > ERROR HTTPError: 400 Bad Request from https://test.pypi.org/legacy/ User 'traversaro' does not have two-factor authentication enabled. Please enable two-factor authentication before attempting to upload to...

Keep in mind that you might also need to update the deployment GH workflow as done in https://github.com/ami-iit/jaxsim/pull/174. The action needs extra permissions.

In any case, if it's not already there, having an error to report that the RPC command is wrong is advisable.