kisungyou / Riemann

Learning with Data on Riemannian Manifolds
https://kisungyou.com/Riemann
Other
10 stars 2 forks source link

Add centre-of-mass function to package? #1

Closed edwardlavender closed 1 year ago

edwardlavender commented 1 year ago

This is a great package! This is a feature request - if it fits within the scope of the package - it it possible to add a function to calculate the centre of mass of a set of longitude/latitudinal coordinates (with associated weights)? I think this would complement the existing riem.mean() and riem.median() functions that are based on minimising the geodesic distances between points. Thanks for sharing this resource!

kisungyou commented 1 year ago

It sounds a good idea to have this functionality but I think it's not suitable to implement this within Riemann package since I would like to focus more on providing a generic set of methodologies for manifold-valued data. You can still write your own function for your use though. Let me know how you think.

edwardlavender commented 1 year ago

That makes sense. Thank you for the response!