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

Trivial euclidean manifold E(n) ? (feature request) #108

Closed
markusgft opened this issue Nov 5, 2019 · 4 comments · Fixed by #109
Closed

Trivial euclidean manifold E(n) ? (feature request) #108

markusgft opened this issue Nov 5, 2019 · 4 comments · Fixed by #109
Assignees
Labels
enhancement New feature or request

Comments

@markusgft
Copy link

Dear all,

I'm an application which takes an "manif" manifold as a template parameter. This works nicely, since the basic interface, accessors and methods are naturally the same for all manifolds.

However, it fails as soon as I want to operate in Euclidean space. What is missing, is essentially a manif-implementation of the must fundamental n-dimensional Euclidean manifold E(n), in which case anything within manif should fall back to the natural Eigen-types and methods.

Is there any plan to add such a feature?

Best, and thanks in advance!

@joansola
Copy link
Collaborator

joansola commented Nov 5, 2019

Hi Markus,

We've been discussing for a while the topics you are raising. For the R^n manifolds, or E(n), they should be easy to implement since they are mostly trivial.

We'll discuss soon about this possibility. We'll keep you posted here.

@markusgft
Copy link
Author

Hi @joansola , thanks for your quick reply!
I really appreciate "manif" and would highly welcome the extended functionalities!
All the best!

@artivis
Copy link
Owner

artivis commented Nov 12, 2019

@markusgft thanks for your interest in manif!
Implementing the E(n) manifolds has been on my todo-list for a while now but I did not have a use for it outside of the infamous composite manifolds (#84). Given your interest in those I will start looking at the implementation asap.
Best.

@artivis artivis self-assigned this Nov 12, 2019
@artivis artivis added the enhancement New feature or request label Nov 12, 2019
@markusgft
Copy link
Author

Dear @artivis, thanks a lot for taking the action, I'm looking forward to the feature.

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

Successfully merging a pull request may close this issue.

3 participants