Merge branch '1.8-maintenance'

.travis.yml

@@ -13,7 +13,7 @@ script:
 
 after_success:
   - echo "secure travis:" "$TRAVIS_SECURE_ENV_VARS"
-  - sh -c 'if test "$TRAVIS_SECURE_ENV_VARS" = "true" -a "$TRAVIS_BRANCH" = "1.7-maintenance"; then echo "publish website"; ./scripts/ci/build_docs.sh; ./scripts/ci/add_deploy_key.sh; ./scripts/ci/deploy_website.sh $TRAVIS_BUILD_DIR/doc/build /tmp; fi'
+  - sh -c 'if test "$TRAVIS_SECURE_ENV_VARS" = "true" -a "$TRAVIS_BRANCH" = "1.8-maintenance"; then echo "publish website"; ./scripts/ci/build_docs.sh; ./scripts/ci/add_deploy_key.sh; ./scripts/ci/deploy_website.sh $TRAVIS_BUILD_DIR/doc/build /tmp; fi'
 
 notifications:
   on_success: always

CMakeLists.txt

@@ -28,7 +28,7 @@ mark_as_advanced(CMAKE_VERBOSE_MAKEFILE)
 
 # the next line is the ONLY place in the entire pdal system where
 # the version info is hard-coded
-set(PDAL_VERSION_STRING "1.8.0" CACHE STRING "PDAL version" FORCE)
+set(PDAL_VERSION_STRING "1.8.0" CACHE STRING "PDAL version")
 
 DISSECT_VERSION()
 GET_OS_INFO()
@@ -335,6 +335,7 @@ install(DIRECTORY ${PDAL_INCLUDE_DIR}/pdal
     FILES_MATCHING PATTERN "*.hpp"
     PATTERN "gitsha.h"
     PATTERN "pdal/private" EXCLUDE
+    PATTERN "pdal/util/private" EXCLUDE
     ${ZSTD_EXCLUDES}
     ${ZLIB_EXCLUDES}
     ${LZMA_EXCLUDES}

RELEASENOTES.txt

@@ -1,3 +1,125 @@
+================================================================================
+1.8.0
+================================================================================
+
+Changes of Note
+===============
+
+-  PointTableRef is now publicly accessible from PointView (#1926)
+-  Minimum CMake version is now 3.5
+-  ``filters.hexbin`` is now a built-in stage, rather than a plugin.
+   (#2001)
+-  Removed support for ``ght`` compression in ``writers.pgpointcloud``.
+   (#2148)
+-  On OSX, plugins are now installed with ID of ``@rpath`` rather than
+   ``@loader_path/../lib``
+
+Enhancements
+============
+
+-  Added ``denoise`` and ``reset`` options to ``pdal ground``. (#1579)
+-  ``readers.gdal`` now supports stream mode and provides the ``header``
+   option to map dimensions. It also supports fetching bounds without
+   reading the entire file. (#1819)
+-  ``readers.mbio`` added ``datatype`` option to support reading
+   sidescan data. (#1852)
+-  ``filters.stats`` was computing expensive kurtosis and skewness
+   statistics by default. These statistics are now available with the
+   ``advanced`` option. (#1878)
+-  Added backtrace support for alpine linux-based Docker containers.
+   (#1904)
+-  Added a ``condition`` option for ``filters.assign`` to limit
+   assignment. (#1956)
+-  Add access to artifact manager keys. (#2026)
+-  Added support for LAZ compression in ``writers.pgpointcloud`` (#2050)
+-  Replaced ``last`` option with ``returns`` to support more flexible
+   segmentation in ``filters.smrf`` and ``filters.pmf``. (#2053)
+-  ``writers.text`` now supports stream mode. (#2064)
+-  Added ``pdal tile`` kernel with streaming support to conveniently
+   tile data. (#2065)
+-  A KD-tree used in one filter will now be reused in subsequent filters
+   when possible. (#2123)
+-  ``writers.ply`` now has a ``precision`` option to specify output
+   precision. (#2144)
+-  ``filters.smrf`` and ``filters.pmf`` supports complete range syntax
+   for the ``ignore`` option. (#2157)
+-  ``filters.hexbin`` now supports stream mode. (#2170)
+-  ``readers.numpy`` now has the ``order`` option, which replaces the
+   previous ``x``, ``y`` and ``z`` options. It also supports structured
+   numpy arrays and maps values to the X, Y and Z dimensions
+   accordingly.
+-  All readers now support setting a spatial reference to override any
+   in the data with the ``spatialreference`` option.
+-  Add support for unicode filenames in pipelines on Windows platforms.
+-  Added NumpyReader::setArray() to support direct setting of a numpy
+   array into ``readers.numpy``.
+-  Added StreamPointTable::setNumPoints() and support in
+   Streamable::execute() allowing custom point tables to know the number
+   of points in each pass through the point table.
+-  Added SpatialReference::isProjected() to allow callers to determine
+   if a spatial reference specifies a projection. Also added
+   SpatialReference::identifyHorizontalEPSG() and
+   SpatialReference::identifyVerticalEPSG() to return an EPSG code from
+   a spatial reference if possible.
+-  Added support for reading BPF files stored remotely.
+
+New stages
+==========
+
+-  ``readers.rdb`` - Support reading RIEGL RDB data.
+-  ``readers.i3s`` - Support reading of web service-style Esri I3S point
+   clouds.
+-  ``readers.slpk`` - Support reading of file-based I3S-style point
+   clouds.
+-  ``writers.fbx`` - Experimental Unity engine (SDK required) support.
+   (#2127)
+-  ``filters.nndistance`` - Replaces ``filters.kdistance`` and adds
+   average distance support. (#2071)
+-  ``filters.dem`` - Filter data based on bounds relative to a raster
+   value. (#2090)
+-  ``filters.delaunay`` - Create a delauany triangulation of a point
+   cloud. (#1855)
+-  ``filters.info`` - Generate metadata about an input point set. Used
+   by ``pdal info``.
+
+Deprecated stages
+=================
+
+-  ``filters.kdistance`` - Replaced by ``filters.nndistance``.
+
+Bug fixes
+=========
+
+-  Fixed an incorrect error message suggesting there were multiple SRSs
+   in some cases when reading multiple inputs. (#2009)
+-  Fixed a problem in ``filters.reprojection`` in stream mode that would
+   improperly reproject when there were multiple input sources with
+   differing SRSs. (#2058)
+-  Fixed a problem in stream mode where a stage with no SRS would
+   override the active SRS during processing. (#2069)
+-  Fixed a problem in ``writers.gdal`` where output would be aggregated
+   if multiple inputs were provided. (#2074)
+-  The ``count`` option was not respected in stream mode. It now
+   properly limits the number of points read. (#2086)
+-  Fixed an off-by-one error that could create improper values in
+   ``writers.gdal``. Most data differences were small and not usually
+   problematic. (#2095)
+-  Multiple option values can be specified on the command line by
+   repeating the option assignment. (#2114)
+-  Added a missing initialization in ``filters.returns`` that could
+   cause more point views to be returned than requested. (#2115)
+-  Emit an error if the ``count`` option isn’t set for ``readers.faux``.
+   (#2128)
+-  PipelineManager::getStage() now returns a proper leaf node. (#2149)
+-  Fixed logic for ``filters.crop`` in streaming mode with multiple crop
+   areas that could return too few points. (#2198)
+-  Added the ``minimal`` option for ``readers.rxp`` that was documented
+   but not fully implemented. (#2225)
+-  Fixed an error in failing to read all points in ``readers.rxp``
+   exposed with a newer SDK. (#2226)
+-  Fixed an error in fetching floating point data from a PointContainer
+   when the value was NaN. (#2239)
+
 ================================================================================
 1.7.0
 ================================================================================

cmake/cpack.cmake

@@ -39,6 +39,7 @@ list(APPEND CPACK_SOURCE_IGNORE_FILES "/doc/presentations/")
 list(APPEND CPACK_SOURCE_IGNORE_FILES "/doc/_static/logo/dongle/")
 list(APPEND CPACK_SOURCE_IGNORE_FILES "/cmake/examples/")
 list(APPEND CPACK_SOURCE_IGNORE_FILES "pdal_features.hpp$")
+list(APPEND CPACK_SOURCE_IGNORE_FILES "package.sh")
 list(APPEND CPACK_SOURCE_IGNORE_FILES ".gz2")
 list(APPEND CPACK_SOURCE_IGNORE_FILES ".bz2")
 

cmake/macros.cmake

@@ -76,11 +76,14 @@ macro(PDAL_ADD_FREE_LIBRARY _name _library_type)
         ${PDAL_INCLUDE_DIR})
     pdal_target_compile_settings(${_name})
 
-    install(TARGETS ${_name}
-        EXPORT PDALTargets
-        RUNTIME DESTINATION ${PDAL_BIN_INSTALL_DIR}
-        LIBRARY DESTINATION ${PDAL_LIB_INSTALL_DIR}
-        ARCHIVE DESTINATION ${PDAL_LIB_INSTALL_DIR})
+    # Don't install static libraries - they're already built into libpdalXXX
+    if (NOT ${_library_type} STREQUAL "STATIC")
+        install(TARGETS ${_name}
+            EXPORT PDALTargets
+            RUNTIME DESTINATION ${PDAL_BIN_INSTALL_DIR}
+            LIBRARY DESTINATION ${PDAL_LIB_INSTALL_DIR}
+            ARCHIVE DESTINATION ${PDAL_LIB_INSTALL_DIR})
+    endif()
 endmacro(PDAL_ADD_FREE_LIBRARY)
 
 ###############################################################################

cmake/options.cmake

@@ -57,13 +57,10 @@ option(BUILD_PLUGIN_OPENSCENEGRAPH
 add_feature_info("OpenSceneGraph plugin" BUILD_PLUGIN_OPENSCENEGRAPH
     "read/write OpenSceneGraph objects")
 
-if(DEFINED ENV{ORACLE_HOME})
-    set(DEFINED_ORACLE_HOME TRUE)
-else(DEFINED ENV{ORACLE_HOME})
-    set(DEFINED_ORACLE_HOME FALSE)
-endif(DEFINED ENV{ORACLE_HOME})
 option(BUILD_PLUGIN_OCI
-    "Choose if OCI support should be built" ${DEFINED_ORACLE_HOME})
+    "Choose if OCI support should be built" FALSE)
+add_feature_info("Oracle OCI plugin" BUILD_PLUGIN_OCI
+    "Read/write point clould patches to Oracle")
 
 option(BUILD_PLUGIN_PCL "Choose if PCL support should be built" FALSE)
 add_feature_info("PCL plugin" BUILD_PLUGIN_PCL

filters/private/hexer/HexGrid.cpp

@@ -269,6 +269,7 @@ Hexagon *HexGrid::getHexagon(int x, int y)
     return hex_p;
 }
 
+/**
 // Walk the outside of the hexagons to make a path.  Hexagon sides are labeled:
 //
 //     __0_
@@ -277,7 +278,7 @@ Hexagon *HexGrid::getHexagon(int x, int y)
 //   \      /
 //  2 \____/ 4
 //      3
-//
+**/
 void HexGrid::findShapes()
 {
     if (m_pos_roots.empty())

filters/private/hexer/Hexagon.cpp

@@ -37,14 +37,14 @@
 namespace hexer
 {
 
-//
+/**
 //     __0_
 //  1 /    \ 5
 //   /      \
 //   \      /
 //  2 \____/ 4
 //      3
-//
+**/
 
 bool Hexagon::less(const Hexagon *h) const
 {