A note on a certain differential #461
Replies: 2 comments 3 replies
-
Hey, thanks a lot for getting started on this. I have forked the notebook (https://gist.github.com/const-ae/ae5e26b255978a3b4eaec19f493282f8) and added the code for the I have also added code for the Rotation manifold and found that for random points on the tangent space, the differential that you suggest does not converge quadratically, whereas the version with |
Beta Was this translation helpful? Give feedback.
-
I found it! It was a little tricky and I confused myself with the (little older) notation in Manopt compared to Manifolds.jl and we should at some point rename two differentials (next breaking change). TLDR: We are both right. The thing is that you tried to use a differential for something it was not meant to be used for. And it is a chain rule. Long story: (Trying without math but with code, since math is not rendered here) There are two ways to specify a geodesic: The first is done with a start point The second way to specify a geodesic is by a start point But the differential of the geodesic with respect to the argument introduces a chain rule that leads to exactly the terms you specified (even “with time” is not that bad for a name). One could – still – use/extend the given coefficients with the t, since for DX of exp(M,p,X) we anyways plug in t=1 and then your new coefficient agree with the old ones; I adapted the documentation and code accordingly, which does not break any of the calls (was called with t=1 only until now anyways) I already started the result as a PR in JuliaManifolds/Manopt.jl#114, where I also denoted the few small caveats still to be taken into account. I hope the explanations are understandable. |
Beta Was this translation helpful? Give feedback.
-
As a continuation from a discussion on slack I started writing down an easy function @const-ae started with. Here is the derivation on why we have an original additional
t
in the derivative - the notebook is still missing the originaldiff_check
. Feel free to comment on this state.🎈 note-constantin.jl — Pluto.jl.pdf
(somehow .jl files are not allowed as files here so here it is as a gist https://gist.github.com/kellertuer/3e17cfce6b0e6d40d3d8e7d14651323c )
Beta Was this translation helpful? Give feedback.
All reactions