Navigation Menu

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

Integrate with Manifolds.jl #244

Closed
dehann opened this issue Feb 20, 2020 · 11 comments · Fixed by #436
Closed

Integrate with Manifolds.jl #244

dehann opened this issue Feb 20, 2020 · 11 comments · Fixed by #436

Comments

@dehann
Copy link
Member Author

dehann commented Feb 20, 2020

Retroactively influences JuliaRobotics/DistributedFactorGraphs.jl#147

@dehann
Copy link
Member Author

dehann commented Feb 20, 2020

@Affie, if I understand your earlier comment right -- Optim.jl currently has support for Sphere or Euclidean and other Manifolds.jl but not SE(n) (Special Euclidean) yet?

@Affie
Copy link
Member

Affie commented Feb 20, 2020

Optim.jl currently has support for Sphere or Euclidean and other Manifolds.jl but not SE(n) (Special Euclidean) yet?

Optim.jl does not support Manifolds.jl yet as far as I can tell.

SE(n)

Thanks, I missed SE in Manifolds.jl, I didn't think to look in decorators.

@Affie
Copy link
Member

Affie commented Feb 20, 2020

@kellertuer
Copy link

Cool, that you already have this in your plans!

Some simple integration to Optim.jl will happen at some time, for example a gradient descent is quite straight forward.
(Biased side note): For more advanced algorithms doing Optimization on Manifold, see Manopt.jl, which will also switch to be based on Manifolds.jl as soon as there is a public release.

@dehann
Copy link
Member Author

dehann commented May 13, 2020

@dehann
Copy link
Member Author

dehann commented Jun 19, 2020

@dehann
Copy link
Member Author

dehann commented Oct 19, 2020

xref #341

@dehann
Copy link
Member Author

dehann commented Nov 21, 2020

xref, need to rebuild densities starting from JuliaRobotics/ApproxManifoldProducts.jl#41

@dehann
Copy link
Member Author

dehann commented Mar 29, 2021

v0.15.0 closes this issue (from RoME.jl perspective). Major future work remains deeper inside and will be tracked at JuliaRobotics/ApproxManifoldProducts.jl#41

@dehann dehann closed this as completed Mar 29, 2021
@dehann
Copy link
Member Author

dehann commented Mar 29, 2021

PS, the local manifolds, SE2E2_Manifold and BearingRange_Manifold are not yet consolidated with the standard Manifolds.jl interface format. That work still remains to be done. The main delivery here is that all variables and surrounding code from RoME IIF AMP not expect and use <:Manifold.

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

Successfully merging a pull request may close this issue.

3 participants