-
Notifications
You must be signed in to change notification settings - Fork 240
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
Fix projection to the space of closed curves #1511
Comments
Hi @ninamiolane , I am getting the orange points when using your code snippet and not the green one that you obtained, Code: `import matplotlib.pyplot as plt curve = np.array([[ 1.88070983e-02, 8.36082045e-02], projection = np.array([[ 0.0188071 , 0.0836082 ], from geomstats.geometry.discrete_curves import R2,ClosedDiscreteCurves plt.scatter(curve[:,0], curve[:,1]) plt.savefig("out.png") plt.scatter(closed_curve[:,0], closed_curve[:,1]) plt.savefig("out3.png") plt.scatter(projection[:,0],projection[:,1]) plt.savefig("out2.png") print(closed_curve)` |
Interesting! I wonder if I made a mistake copy-pasting this snippet :/ In any case, you get the bug too: since the orange points do not form the closed version of the open blue curve. Would you know where it is coming from? |
Hello @ninamiolane, |
Hi @EBIluishauptmann which version of Geomstats are you using? Two recommendations:
I have not fully understood why the bug happens, but I have noticed in practice that the two bullet points above helped. Send us your code snippet if you have a reproducible error, with the version you use, so that we can try to fix this? |
Where should the content be modified?
https://github.com/geomstats/geomstats/blob/master/geomstats/geometry/discrete_curves.py
What should be modified?
The following code:
for the curve of 200 sampling points in 2D defined below, does not give a closed curve, but rather:
Curve:
Closed curve:
Additional information
curve:
closed_curve:
The text was updated successfully, but these errors were encountered: