This repository was archived by the owner on Aug 6, 2024. It is now read-only.
This repository was archived by the owner on Aug 6, 2024. It is now read-only.
better embedding of mass and potential #24
Description
Description
Currently the mass and potential are embedded in a SkyCoord. This does not work very well as any operation that returns a new instance, like reshape means that the attributes need to be reassigned (separately reshaped).
Possible fixes:
- Subclass SkyCoord, with hooks to properly maintain these attributes
- embed in a table (SkyCoord is a valid mixin) ... but can tables be stacked depth-wise?
- Xarray? but how does that deal with SkyCoords?
Checklist
- Check out our contributing guidelines.
- Check out our code of conduct.
- Search the GitHub repository to see if a similar issue has already been posted. If a similar issue is closed, have a quick look to see if you are satisfied by the resolution. If not please go ahead and open an issue!