Fix names of dimensions received from ept.json.

PDAL · Apr 2, 2021 · b3bb575 · b3bb575
1 parent fdee1a5
commit b3bb575
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/io/private/ept/EptInfo.cpp b/io/private/ept/EptInfo.cpp
@@ -151,6 +151,7 @@ void EptInfo::initialize()
         double scale = element.value("scale", 1.0);
         double offset = element.value("offset", 0);
 
+        name = Dimension::fixName(name);
         Dimension::Id id = m_remoteLayout.registerOrAssignFixedDim(name, type);
         m_dims[name] = DimType(id, type, scale, offset);
     }

diff --git a/io/private/ept/TileContents.hpp b/io/private/ept/TileContents.hpp
@@ -63,6 +63,10 @@ class TileContents
         { return m_overlap.m_key; }
     point_count_t nodeId() const
         { return m_overlap.m_nodeId; }
+    //ABELL - This is bad. We're assuming that the actual number of points we have matches
+    // what our index information told us. This may not be the case because of some
+    // issue. Downstream handling may depend on this being the actual number of points
+    // in the tile, rather than the number that were *supposed to be* in the tile.
     point_count_t size() const
         { return m_overlap.m_count; }
     const std::string& error() const