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

3d rotations #12

Open
danieleghisi opened this issue Oct 1, 2020 · 2 comments
Open

3d rotations #12

danieleghisi opened this issue Oct 1, 2020 · 2 comments

Comments

@danieleghisi
Copy link

Hello HOAlibrary crew,

I am in the process of linking the HOA library to an offline audio processing library I am writing, and in doing so I noticed that you had no implementation for 3d rotations. I have set up in a couple of days a barebone implementation, based on yaw/pitch/roll; there's a little convention mismatch with the axis of my library, but I will handle it outside. One could certainly imagine a direct quaternion input (as a matter of fact, that's probably almost a one liner, given that my current implementation uses Eigen's quaternionToEuler conversions).

Not sure if the 3d rotation was already in your pipeline. Also, I'm no expert in this 3d rotational geometry, euler angles, quaternions, etc. So what I did is definitely error prone, and should be perhaps checked by someone more skilled than I am. Another limitation is that only orders up to 21 are supported (in practice this is not a limitation, of course, but in theory it is...), since I am relying on the Ry90 matrices Frank Zotter provided (and they exist up to the 21st order). That being said, for my application it seems to work nicely, although I'm sure that you would have implemented it ten times better than I did.

In any case, if that was not in your pipeline and you are interested to merge it, let me know and I can perhaps make a pull request at some point.

Thanks for your amazing work!
Daniele

@pierreguillot
Copy link
Member

Hello Daniele,

In fact, the library is no more maintained. Eliott and I are both working in other places and we don't have enough time to work on it. When we were working on the library, we planned to add the 3D rotation even with order limitation so, indeed, it would be great to integrate this feature. But even if you do the best pull request in the world, I'm afraid that I won't be able to merge it before a while because I've already so many pull requests to merge on other projects :( I could discuss with the CICM what would be the best strategy for these repositories that are not maintained and keep you informed.

Cheers,
Pierre

@danieleghisi
Copy link
Author

I understand. Too bad it is not maintained any longer...

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

No branches or pull requests

2 participants