Geospatial component for heightmap & DEMs (gazebosim#267)

Signed-off-by: Jenn Nguyen <[email protected]> Co-authored-by: Steve Peters <[email protected]> Co-authored-by: Ian Chen <[email protected]>
scpeters · Jan 19, 2022 · 28eb6c7 · 28eb6c7
1 parent be3aa32
commit 28eb6c7
Show file tree

Hide file tree

Showing 17 changed files with 248 additions and 101 deletions.
diff --git a/.github/ci/packages.apt b/.github/ci/packages.apt
@@ -3,6 +3,7 @@ libavdevice-dev
 libavformat-dev
 libavutil-dev
 libfreeimage-dev
+libgdal-dev
 libgts-dev
 libignition-cmake2-dev
 libignition-math7-dev

diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -48,7 +48,7 @@ message(STATUS "\n\n-- ====== Finding Dependencies ======")
 
 #--------------------------------------
 # Find ignition-math
-ign_find_package(ignition-math7 REQUIRED_BY graphics events)
+ign_find_package(ignition-math7 REQUIRED_BY geospatial graphics events)
 set(IGN_MATH_VER ${ignition-math7_VERSION_MAJOR})
 
 #--------------------------------------
@@ -94,6 +94,13 @@ ign_find_package(
   REQUIRED_BY graphics
   PRIVATE_FOR graphics)
 
+#------------------------------------
+# Find GDAL
+ign_find_package(GDAL VERSION 3.0
+  PKGCONFIG gdal
+  PRIVATE_FOR geospatial
+  REQUIRED_BY geospatial)
+
 #------------------------------------
 # Find libswscale
 ign_find_package(SWSCALE REQUIRED_BY av PRETTY libswscale)
@@ -125,7 +132,7 @@ configure_file("${PROJECT_SOURCE_DIR}/cppcheck.suppress.in"
                ${PROJECT_BINARY_DIR}/cppcheck.suppress)
 
 ign_configure_build(QUIT_IF_BUILD_ERRORS
-  COMPONENTS av events graphics profiler)
+  COMPONENTS av events geospatial graphics profiler)
 
 #============================================================================
 # Create package information

diff --git a/Migration.md b/Migration.md
@@ -5,6 +5,20 @@ Deprecated code produces compile-time warnings. These warning serve as
 notification to users that their code should be upgraded. The next major
 release will remove the deprecated code.
 
+## Ignition Common 4.X to 5.X
+
+### Additions
+
+1. **geospatial** component that loads heightmap images and DEMs
+    + Depends on the ign-common's `graphics` component and the `gdal` library
+
+### Modifications
+
+1. `HeightmapData.hh` and `ImageHeightmap.hh` have been moved out of the
+`graphics` component and into the new `geospatial` component
+    + To use the heightmap features, users must add the `geospatial` component
+      to the `find_package` call
+
 ## Ignition Common 3.X to 4.X
 
 ### Modifications
@@ -72,4 +86,3 @@ release will remove the deprecated code.
 1. **ignition-cmake**
     + Ignition-math now has a build dependency on ignition-cmake, which
       allows cmake scripts to be shared across all the ignition packages.
-
diff --git a/geospatial/include/ignition/common/CMakeLists.txt b/geospatial/include/ignition/common/CMakeLists.txt
@@ -0,0 +1,2 @@
+
+ign_install_all_headers(COMPONENT geospatial)
diff --git a/graphics/include/ignition/common/Dem.hh → geospatial/include/ignition/common/Dem.hh b/graphics/include/ignition/common/Dem.hh → geospatial/include/ignition/common/Dem.hh
@@ -18,18 +18,17 @@
 #define IGNITION_COMMON_DEM_HH_
 
 #include <memory>
+#include <string>
+#include <vector>
+
 #include <ignition/math/Vector3.hh>
 #include <ignition/math/Angle.hh>
 
+#include <ignition/common/HeightmapData.hh>
 #include <ignition/common/graphics/Export.hh>
 
 #include <ignition/utils/ImplPtr.hh>
 
-#ifdef HAVE_GDAL
-# include <string>
-# include <vector>
-
-# include <ignition/common/HeightmapData.hh>
 
 namespace ignition
 {
@@ -53,7 +52,8 @@ namespace ignition
       /// \brief Get the elevation of a terrain's point in meters.
       /// \param[in] _x X coordinate of the terrain.
       /// \param[in] _y Y coordinate of the terrain.
-      /// \return Terrain's elevation at (x,y) in meters.
+      /// \return Terrain's elevation at (x,y) in meters or infinity if illegal
+      /// coordinates were provided.
       public: double Elevation(double _x, double _y);
 
       /// \brief Get the terrain's minimum elevation in meters.
@@ -68,7 +68,8 @@ namespace ignition
       /// origin in WGS84.
       /// \param[out] _latitude Georeferenced latitude.
       /// \param[out] _longitude Georeferenced longitude.
-      public: void GeoReferenceOrigin(ignition::math::Angle &_latitude,
+      /// \return True if able to retrieve origin coordinates. False otherwise.
+      public: bool GeoReferenceOrigin(ignition::math::Angle &_latitude,
                   ignition::math::Angle &_longitude) const;
 
       /// \brief Get the terrain's height. Due to the Ogre constrains, this
@@ -120,7 +121,8 @@ namespace ignition
       /// \param[in] _y Y coordinate of the terrain.
       /// \param[out] _latitude Georeferenced latitude.
       /// \param[out] _longitude Georeferenced longitude.
-      private: void GeoReference(double _x, double _y,
+      /// \return True if able to retrieve coordinates. False otherwise.
+      private: bool GeoReference(double _x, double _y,
                                  ignition::math::Angle &_latitude,
                                  ignition::math::Angle &_longitude) const;
 
@@ -130,11 +132,13 @@ namespace ignition
       /// \return 0 when the operation succeeds to open a file.
       private: int LoadData();
 
+      // Documentation inherited.
+      public: std::string Filename() const;
+
       /// internal
       /// \brief Pointer to the private data.
       IGN_UTILS_IMPL_PTR(dataPtr)
     };
   }
 }
 #endif
-#endif
diff --git a/.../include/ignition/common/HeightmapData.hh → .../include/ignition/common/HeightmapData.hh b/.../include/ignition/common/HeightmapData.hh → .../include/ignition/common/HeightmapData.hh
@@ -17,6 +17,7 @@
 #ifndef IGNITION_COMMON_HEIGHTMAPDATA_HH_
 #define IGNITION_COMMON_HEIGHTMAPDATA_HH_
 
+#include <string>
 #include <vector>
 #include <ignition/math/Vector3.hh>
 #include <ignition/common/graphics/Export.hh>
@@ -57,6 +58,17 @@ namespace ignition
       /// \brief Get the maximum terrain's elevation.
       /// \return The maximum terrain's elevation.
       public: virtual float MaxElevation() const = 0;
+
+      /// \brief Get the min terrain's elevation.
+      /// \return The min terrain's elevation.
+      public: virtual float MinElevation() const
+              {
+                return 0.0f;
+              }
+
+      /// \brief Get the full filename of loaded heightmap image/dem
+      /// \return The filename used to load the heightmap image/dem
+      public: virtual std::string Filename() const = 0;
     };
   }
 }

diff --git a/...include/ignition/common/ImageHeightmap.hh → ...include/ignition/common/ImageHeightmap.hh b/...include/ignition/common/ImageHeightmap.hh → ...include/ignition/common/ImageHeightmap.hh
@@ -49,8 +49,7 @@ namespace ignition
           const ignition::math::Vector3d &_scale, bool _flipY,
           std::vector<float> &_heights);
 
-      /// \brief Get the full filename of the image
-      /// \return The filename used to load the image
+      // Documentation inherited.
       public: std::string Filename() const;
 
       // Documentation inherited.

diff --git a/geospatial/src/CMakeLists.txt b/geospatial/src/CMakeLists.txt
@@ -0,0 +1,22 @@
+
+ign_get_libsources_and_unittests(sources gtest_sources)
+
+ign_add_component(geospatial
+  SOURCES ${sources}
+  DEPENDS_ON_COMPONENTS graphics
+  GET_TARGET_NAME geospatial_target)
+
+target_link_libraries(${geospatial_target}
+  PUBLIC
+    ${PROJECT_LIBRARY_TARGET_NAME}-graphics
+    ignition-math${IGN_MATH_VER}::ignition-math${IGN_MATH_VER}
+    ignition-utils${IGN_UTILS_VER}::ignition-utils${IGN_UTILS_VER}
+  PRIVATE
+    ${GDAL_LIBRARY})
+
+target_include_directories(${geospatial_target}
+  PRIVATE
+    ${GDAL_INCLUDE_DIR})
+
+ign_build_tests(TYPE UNIT SOURCES ${gtest_sources}
+  LIB_DEPS ${geospatial_target})
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1,2 @@

		ign_install_all_headers(COMPONENT geospatial)