-
Notifications
You must be signed in to change notification settings - Fork 15
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
Long term support of jaxlie? #15
Comments
Hi Michael, thanks again for the nice words! There are no new features planned, but I'm planning to support That said,
Overall I think the specific decisions we made for these things were correct given my personal priorities, but I can see other folks having different ones! Footnotes
|
Thank you for answering this, this is all helpful to know.
As for feature requests, if we go forward with this as a dependency at some point I would consider a PR that allows for utilities for using different euler angle conventions. However, this is easy enough to do with the package as is. Thanks again! |
Hi sorry for the late reply, this got buried by holiday things... Yeah, I don't have plans to implement #7. This would be a breaking API change, the ergonomics of having several floating arrays now feels worse to me, and I don't think the hypothetical + likely small runtime benefits would be worth the effort. If you have features you want I'm happy to review PRs! Euler angles are frowned upon in most of the circles I find myself in (because of singularities, the illusion of interpretability, too many ordering conventions, etc), but I do think incorporating more support for them makes sense for this package. 😛 |
Great, thanks for the update and no problem on the delay. The help is much appreciated! |
Hello! Me again. Again, this is a great library and thank you for responding to pull requests so promptly. I was wondering though its plans for long-term support, as I’m debating if I should keep this as a dependency in my library or write something home-cooked.
The text was updated successfully, but these errors were encountered: