APE5: Coordinates Subpackage Plan#6
Conversation
|
I will start a discussion here since this doesn't relate to the API. It would be good if this proposal states how this package relates to a WCS package/functionality. Can you also clarify what the role of The APE says that In general how does this relates to the |
|
@nden - I see your point that As for And when I meant non-celestial coordinates, I did not mean the full WCS sorts of options - rather, other forms of spatial coordinates that are not celestial per se, but that one might want to hook into the trasnformation graph. The most critical example here is ITRF/ITRS, the reference system for points on the surface of the earth, as that's necessary for getting from ICRS to AltAz. But that's not a "celestial" system by definition. But to reiterate, I do not mean that this system will eventually be expanded to include dispersion/temporal coordinates as it's firmly in WCS territory. Does that all makes sense? If so, I can try to clear some of this up in the APE text itself. |
|
@eteq This makes sense. Thanks for the clarification. |
|
No objections have been raised for the merging of the APE, so going ahead and merging it! The API may require one final iteration before it is merged. |
APE5: Coordinates Subpackage Plan
Address several points raised by @mhvk
This APE provides an outline of a proposal for how to move forward with the coordinates subpackage. It is an attempt to compromise between the various suggested approaches in a way that I hope will be acceptable to all parties, as well as a plan for how it might be implemented.
I think it also will make it easier for multiple people to work on
coordinatesat the same time. This has been a problem in the past, where a single person (in most cases, me) would roadblock parallel work because they were making changes that would have a variety of important consequences for other efforts.Note that this APE is tied closely with astropy/astropy-api#12 - that outlines the actual proposed API, without which this APE is rather abstract.