CLMM icon indicating copy to clipboard operation
CLMM copied to clipboard

Issue/635/Modify coordinate system implementation

Open marina-ricci opened this issue 7 months ago • 1 comments

This PR aims at resolving issues explained in issue #635.

Copied from the issue :

These are my suggestions :

  • [ ] add warning message when GalaxyCluster (or compute_tangential_xxx, see second point) object is initiated without specifying the coordinate system. Like "no coordinate system provided, taking euclidean as default"
  • [ ] put coordinate system as a keyword for compute_tangential_and_cross_components, from data_ops or as a GalaxyCluster method. Maybe not necessary to have it as a GalaxyCluster keyword because it may be confusing as it is not necessarily propagated.
  • [ ] create a simple notebook to highlight this.

marina-ricci avatar Jul 30 '24 18:07 marina-ricci