Skip to content

Commit

Permalink
can now read slpk as well as from http, created tests for bounding in…
Browse files Browse the repository at this point in the history
… local and remote files with autzen and small autzen
  • Loading branch information
kylemann16 committed Sep 19, 2018
1 parent 509cf19 commit c1a6bab
Show file tree
Hide file tree
Showing 7 changed files with 379 additions and 234 deletions.
4 changes: 2 additions & 2 deletions pdal/StageFactory.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -68,10 +68,10 @@ std::string StageFactory::inferReaderDriver(const std::string& filename)
search);
});

if (filename.find(".slpk") != std::string::npos)
ext = ".i3s";
if (protocol != protocols.end())
ext = "." + *protocol;
else if (filename.find(".slpk") != std::string::npos)

This comment has been minimized.

Copy link
@connormanning

connormanning Sep 19, 2018

Contributor

This shouldn't be here, rather the slpk extension should be mapped to the I3sReader in StageExtensions.

This comment has been minimized.

Copy link
@abellgithub

abellgithub Sep 19, 2018

Contributor

There's FileUtils::extension().

ext = ".i3s";
else
ext = FileUtils::extension(filename);
// Strip off '.' and make lowercase.
Expand Down
9 changes: 9 additions & 0 deletions plugins/i3s/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -12,14 +12,23 @@ PDAL_ADD_PLUGIN(reader_libname reader i3s
FILES
io/i3sReader.cpp
io/i3sReceiver.cpp
io/SlpkExtractor.cpp
LINK_WITH
${PDAL_JSONCPP_LIB_NAME}
${WINSOCK_LIBRARY}
)

target_link_libraries(${reader_libname} PRIVATE
${GDAL_LIBRARY}
${ZLIB_LIBRARY}
lepcc
)

include(${PDAL_CMAKE_DIR}/zlib.cmake)
if (ZLIB_FOUND)
target_compile_definitions(${reader_libname} PRIVATE -DARBITER_ZLIB)
endif()

target_include_directories(${reader_libname} PRIVATE
${PDAL_JSONCPP_INCLUDE_DIR}
${PDAL_VENDOR_DIR}
Expand Down

0 comments on commit c1a6bab

Please sign in to comment.