Replies: 9 comments
-
Hi @ilska , you are right. This part is actually similar to this DMP variation (Eq. 3). We could document this more clearly or we could introduce a parameter to change it. I am not entirely sure at the moment, but I think this version fixes the scaling issue mentioned in this paper in Fig. 1. |
Beta Was this translation helpful? Give feedback.
-
Hi @AlexanderFabisch, thanks for your response! That is what I undestood, that the term K(g-yo) theta of equation 1 in the paper you link is added to fix the scaling issues around g = y0. So I tried to find a term related to it when integrating, in the function dmp_step_euler and I could not understand why there was nothing, and I thought that I may have missed something :) |
Beta Was this translation helpful? Give feedback.
-
I'm a bit confused. Eq. 2.3 of Ijspeert et al. is the forcing term (Eq. 3 of Pastor et al.). Eq. 1 of Pastor et al. is the transformation system. Anyway, I'll check what the effects of the scaling in the forcing term are. |
Beta Was this translation helpful? Give feedback.
-
Here is what happens when you scale a DMP with And this is the result including the scaling by It's not exactly the same version as in Pastor et al., but it seems to solve the problem. Code
import numpy as np
import matplotlib.pyplot as plt
from movement_primitives.dmp import DMP
|
Beta Was this translation helpful? Give feedback.
-
Ok, thanks! It definitely solves the problem yes. I will try to understand how the current implementation deals with it without adding any term related to it in the transformation system as in Eq 1 in Pastor et al. Thank you for your time and your super quick answers :) |
Beta Was this translation helpful? Give feedback.
-
@ilska We are thinking about implementing the term proposed by Pastor et al. in the transformation system. Did you find out more about it? How does it change the scaling? |
Beta Was this translation helpful? Give feedback.
-
Hi! I'm afraid I haven't had free time to look at it. Most likely, by the end of the month, I'll be able to return back to this. I will write you back with whatever I find :) |
Beta Was this translation helpful? Give feedback.
-
I was working on this issue for the last days. The term |
Beta Was this translation helpful? Give feedback.
-
Great!! I will play with it as soon as I have some time :) |
Beta Was this translation helpful? Give feedback.
-
Hi,
According to the code, the _dmp.py has been implemented following A.J. Ijspeert, J. Nakanishi, H. Hoffmann, P. Pastor, S. Schaal:
Dynamical Movement Primitives: Learning Attractor Models for Motor Behaviors (2013), Neural Computation 25(2), pp. 328-373, doi:
10.1162/NECO_a_00393, https://ieeexplore.ieee.org/document/6797340
I am going through the code and I have doubts about equation 2.3 in the paper. During the calculation of the forcing term, I could not find the (g-y0) term in the imlementation, and I have not been able to find it in the function dmp_step_euler. I would have expected to have a (g-y0) term in there.
I was hoping you could help me understand this better. Any assistance you could provide would be greatly appreciated.
Beta Was this translation helpful? Give feedback.
All reactions