ossama-othman / MaRC

MaRC - Map Reprojections and Conversions
GNU Lesser General Public License v2.1
1 stars 0 forks source link

ViewingGeometry longitude handling is confusing #49

Open ossama-othman opened 6 years ago

ossama-othman commented 6 years ago

The MaRC::ViewingGeometry class (formerly part of MaRC::PhotoImage) expects all supplied longitudes, such as the sub-observation and sub-solar longitudes, to be West longitudes, regardless of the body rotation direction. This is confusing since the typically followed convention is to use West longitudes for bodies with a prograde rotation, and East longitudes for those with retrograde rotation. Ideally, the supplied longitudes should follow that convention. Doing so would allow for removal of the body rotation checks in MaRC::ViewingGeometry implementation.

Convenience wrappers, such as MaRC::EastLongitude and MaRC::WestLongitude, could be provided to prevent the user from inadvertently supplying an East longitude when a West longitude was expected, or vice versa.