fly/flightgear
1
0
Fork 0
Code Activity

Submodels: Fix initial position offset

Browse source 
Fixes https://sourceforge.net/p/flightgear/codetickets/2521/

For submodels with rotation offsets, the rotation was applied to the
translation offset.
This is incorrect: translation offsets should be in the aircraft local
frame, and thus are not affected by submodel rotation offsets.
This commit is contained in:
Colin Geniet 2021-01-17 21:35:48 +01:00 committed by James Turner
parent 02814ebe35
commit 4f4a81d12b
1 changed files with 3 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
src/AIModel/submodel.cxx
View file

@ -712,9 +712,9 @@ void FGSubmodelMgr::loadSubmodels()
SGVec3d FGSubmodelMgr::getCartOffsetPos(submodel* sm) const
{
// Transform to the right coordinate frame, configuration is done in
// the x-forward, y-right, z-up coordinates (feet), computation
// in the simulation usual body x-forward, y-right, z-down coordinates
// (meters) )
// either x-backward, y-right, z-up coordinates (meter),
// or (deprecated) x-forward, y-right, z-up coordinates (feet).
// computation in the simulation usual body x-forward, y-right, z-down coordinates (meters)
SGVec3d offset;
if (sm->offsets_in_meter) {
offset = SGVec3d(-sm->x_offset->get_value(),
@ -736,11 +736,6 @@ SGVec3d FGSubmodelMgr::getCartOffsetPos(submodel* sm) const
IC.elevation,
IC.roll);
// postrotate by any pitch/heasing/roll offset
hlTrans *= SGQuatd::fromYawPitchRollDeg(sm->yaw_offset->get_value(),
sm->pitch_offset->get_value(),
0.0);
// The offset converted to the usual body fixed coordinate system
// rotated to the earth-fixed coordinates axis
SGVec3d off = hlTrans.backTransform(offset);