Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fixed Unreal Lidar sensor incorrectly setting start position for the traces #2143

Merged
merged 7 commits into from
Oct 8, 2019
Merged

Fixed Unreal Lidar sensor incorrectly setting start position for the traces #2143

Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
63b4294
Fixed Unreal Lidar sensor incorrectly setting start position for the …
WouterJansen Aug 19, 2019
dd8c327
Merge branch 'master' of https://github.com/microsoft/AirSim
WouterJansen Sep 18, 2019
2cd28cf
Fixed python client QuaternionR class having bug to correctly invert …
WouterJansen Sep 18, 2019
b8d076f
Update types.py
WouterJansen Sep 18, 2019
c92b76b
Revert "Update types.py"
WouterJansen Oct 3, 2019
02b2a82
Revert "Fixed python client QuaternionR class having bug to correctly…
WouterJansen Oct 3, 2019
49cbfd1
Merge remote-tracking branch 'upstream/master'
WouterJansen Oct 3, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion Unreal/Plugins/AirSim/Source/UnrealSensors/UnrealLidarSensor.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -113,7 +113,7 @@ bool UnrealLidarSensor::shootLaser(const msr::airlib::Pose& lidar_pose, const ms
const msr::airlib::LidarSimpleParams params, Vector3r &point, int &segmentationID)
{
// start position
Vector3r start = lidar_pose.position + vehicle_pose.position;
Vector3r start = VectorMath::add(lidar_pose, vehicle_pose).position;

// We need to compose rotations here rather than rotate a vector by a quaternion
// Hence using coordOrientationAdd(..) rather than rotateQuaternion(..)
Expand Down