[pdal-commits] [PDAL/PDAL] d85ae7: Add number_of_returns option to FauxReader

[GitHub]

  Branch: refs/heads/point-context-schema
  Home:   https://github.com/PDAL/PDAL
  Commit: d85ae7a4b14b7b2e99e917a6c67a611e0f58b1cc
      https://github.com/PDAL/PDAL/commit/d85ae7a4b14b7b2e99e917a6c67a611e0f58b1cc
  Author: Pete Gadomski <pete.gadomski at gmail.com>
  Date:   2014-08-25 (Mon, 25 Aug 2014)

  Changed paths:
    M include/pdal/drivers/faux/Reader.hpp
    M src/drivers/faux/Reader.cpp
    M test/unit/drivers/faux/FauxReaderTest.cpp

  Log Message:
  -----------
  Add number_of_returns option to FauxReader

This option tells the FauxReader to write ReturnNumber and
NumberOfReturns to the buffer. NumberOfReturns will be the value passed
in via `number_of_returns` option. ReturnNumber will increase
monotonically from 1 to `number_of_returns`, then reset to 1 and repeat
as necessary.

This patch enables easy testing of high return counts.


  Commit: afc87495b64ca2a1ef9e6c204e465f1bdc60d595
      https://github.com/PDAL/PDAL/commit/afc87495b64ca2a1ef9e6c204e465f1bdc60d595
  Author: Pete Gadomski <pete.gadomski at gmail.com>
  Date:   2014-08-26 (Tue, 26 Aug 2014)

  Changed paths:
    M doc/stages/drivers.las.writer.txt
    M include/pdal/drivers/las/Writer.hpp
    M src/drivers/las/Writer.cpp
    M test/unit/drivers/las/LasWriterTest.cpp

  Log Message:
  -----------
  Skip, don't fail, when writing high return num las

This patch adds a new option to the las writer,
`discard_high_return_numbers`. This is a boolean option that defaults to
false. If this value is true, the las writer will check the return
number and return counts on incoming points. If the return number is
above the LAS maximum of five returns, then the point is dropped.
Additionally, the numberOfReturns is clipped to five.

This is to support data streams that might have more than five returns
for a given pulse.


  Commit: 38481b13ec990088e49c7e696883d385f0bf9b57
      https://github.com/PDAL/PDAL/commit/38481b13ec990088e49c7e696883d385f0bf9b57
  Author: Howard Butler <howard at hobu.co>
  Date:   2014-08-26 (Tue, 26 Aug 2014)

  Changed paths:
    M doc/stages/drivers.las.writer.txt
    M include/pdal/drivers/faux/Reader.hpp
    M include/pdal/drivers/las/Writer.hpp
    M src/drivers/faux/Reader.cpp
    M src/drivers/las/Writer.cpp
    M test/unit/drivers/faux/FauxReaderTest.cpp
    M test/unit/drivers/las/LasWriterTest.cpp

  Log Message:
  -----------
  Merge pull request #413 from gadomski/las-skip-high-return-numbers

Optionally skip high return numbers in the las writer


Compare: https://github.com/PDAL/PDAL/compare/84d1f4fffd0b...38481b13ec99