Skip to content

Commit

Permalink
Fix gimbal on BDB LR79 (#1947)
Browse files Browse the repository at this point in the history
Don't delete a non-existent "second" ModuleGimbal, it ends up deleting the first. Also don't delete the non-existent ModuleEnginesFX.

BDB has 2 engines mapped to LR79: juno_engines3d and thorEngine. thorEngine has built-in verniers, so it has 2 ModuleGimbals instead of 1. The RO config strips out the verniers and their gimbal setting. The RO config for juno_engines3d tries to strip out the same verniers, but there aren't any. I don't speak ModuleManager, but it would seem `!MODULE[ModuleGimbal],1` is intended to match "the second one", but that "second of one item" is that one item.
lpgagnon authored and pap1723 committed Dec 6, 2018
1 parent 46bf0aa commit 59adc4d
Showing 1 changed file with 0 additions and 6 deletions.
Original file line number Diff line number Diff line change
@@ -59,12 +59,6 @@
%useGimbalResponseSpeed = true
%gimbalResponseSpeed = 16
}
!MODULE[ModuleEnginesFX]
{
}
!MODULE[ModuleGimbal],1
{
}
}

// Removing non-relevant configs and changing title

0 comments on commit 59adc4d

Please sign in to comment.