Skip to content

Commit 3f7fa26

Browse files
Add configurable dense and ordered point cloud parameters
Signed-off-by: Aleksander Kamiński <[email protected]>
1 parent 9a3d0a4 commit 3f7fa26

File tree

2 files changed

+2
-10
lines changed

2 files changed

+2
-10
lines changed

Code/Source/Lidar/LidarRaycaster.cpp

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -114,8 +114,7 @@ namespace RGL
114114
}
115115

116116
m_graph.ConfigureFieldNodes(m_rglRaycastResults.m_fields.data(), m_rglRaycastResults.m_fields.size());
117-
118-
m_graph.SetIsCompactEnabled(ShouldEnableCompact());
117+
m_graph.SetIsCompactEnabled(!m_returnNonHits);
119118
}
120119

121120
AZ::Outcome<ROS2::RaycastResults, const char*> LidarRaycaster::PerformRaycast(const AZ::Transform& lidarTransform)
@@ -220,7 +219,7 @@ namespace RGL
220219
UpdateNonHitValues();
221220

222221
// We need to configure if points should be compacted to minimize the CPU operations when retrieving raycast results.
223-
m_graph.SetIsCompactEnabled(ShouldEnableCompact());
222+
m_graph.SetIsCompactEnabled(!returnNonHits);
224223
}
225224

226225
AZStd::optional<size_t> LidarRaycaster::GetRglResultsSize(
@@ -284,9 +283,4 @@ namespace RGL
284283

285284
return resultsSize;
286285
}
287-
288-
bool LidarRaycaster::ShouldEnableCompact() const
289-
{
290-
return !m_raycastResults->IsFieldPresent<ROS2::RaycastResultFlags::Range>() && !m_returnNonHits;
291-
}
292286
} // namespace RGL

Code/Source/Lidar/LidarRaycaster.h

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,5 @@ namespace RGL
6363
AZStd::optional<ROS2::RaycastResults> m_raycastResults;
6464

6565
PipelineGraph m_graph;
66-
67-
[[nodiscard]] bool ShouldEnableCompact() const;
6866
};
6967
} // namespace RGL

0 commit comments

Comments
 (0)