DM-15831: Remove unused afw/coord.h methods

include/lsst/afw/coord.h

@@ -28,6 +28,5 @@
 
 #include "lsst/afw/coord/Observatory.h"
 #include "lsst/afw/coord/Weather.h"
-#include "lsst/afw/coord/Coord.h"
 
 #endif  // LSST_AFW_COORD_H

include/lsst/afw/coord/Observatory.h

@@ -79,10 +79,6 @@ class Observatory final {
     /// get telescope elevation (meters above reference spheroid)
     double getElevation() const noexcept { return _elevation; }
 
-    /// get telescope longitude as a dd:mm:ss.s string (positive values are E of Greenwich)
-    std::string getLongitudeStr() const;
-    /// get telescope latitude as a dd:mm:ss.s string
-    std::string getLatitudeStr() const;
     /// get string representation
     std::string toString() const;
 

python/lsst/afw/coord/observatory.cc

@@ -36,7 +36,6 @@ PYBIND11_MODULE(observatory, mod) {
 
     /* Constructors */
     cls.def(py::init<lsst::geom::Angle const, lsst::geom::Angle const, double const>());
-    cls.def(py::init<std::string const, std::string const, double const>());
 
     /* Operators */
     cls.def("__eq__", [](Observatory const& self, Observatory const& other) { return self == other; },
@@ -50,8 +49,6 @@ PYBIND11_MODULE(observatory, mod) {
     cls.def("getLongitude", &Observatory::getLongitude);
     cls.def("getLatitude", &Observatory::getLatitude);
     cls.def("getElevation", &Observatory::getElevation);
-    cls.def("getLatitudeStr", &Observatory::getLatitudeStr);
-    cls.def("getLongitudeStr", &Observatory::getLongitudeStr);
     cls.def("setLongitude", &Observatory::setLongitude, "longitude"_a);
     cls.def("setLatitude", &Observatory::setLatitude, "latitude"_a);
     cls.def("setElevation", &Observatory::setElevation, "elevation"_a);

src/coord/Observatory.cc

@@ -28,7 +28,6 @@
 #include "boost/format.hpp"
 
 #include "lsst/geom/Angle.h"
-#include "lsst/afw/coord/Coord.h"
 #include "lsst/afw/coord/Observatory.h"
 
 namespace lsst {
@@ -39,11 +38,6 @@ Observatory::Observatory(lsst::geom::Angle const longitude, lsst::geom::Angle co
                          double const elevation)
         : _latitude(latitude), _longitude(longitude), _elevation(elevation) {}
 
-Observatory::Observatory(std::string const& longitude, std::string const& latitude, double const elevation)
-        : _latitude(dmsStringToAngle(latitude)),
-          _longitude(dmsStringToAngle(longitude)),
-          _elevation(elevation) {}
-
 Observatory::~Observatory() noexcept = default;
 
 Observatory::Observatory(Observatory const&) noexcept = default;
@@ -61,10 +55,6 @@ void Observatory::setLongitude(lsst::geom::Angle const longitude) { _longitude =
 
 void Observatory::setElevation(double const elevation) { _elevation = elevation; }
 
-std::string Observatory::getLongitudeStr() const { return angleToDmsString(_longitude); }
-
-std::string Observatory::getLatitudeStr() const { return angleToDmsString(_latitude); }
-
 std::string Observatory::toString() const {
     return (boost::format("%gW, %gN  %g") % getLatitude().asDegrees() % getLongitude().asDegrees() %
             getElevation())