diff --git a/src/MissionManager/FWLandingPattern.FactMetaData.json b/src/MissionManager/FWLandingPattern.FactMetaData.json index 8d5a21f0022332061542c2e67f99cfa00ef6cc7d..ca77c066c2ef877f4e729492601a63872e09bd9b 100644 --- a/src/MissionManager/FWLandingPattern.FactMetaData.json +++ b/src/MissionManager/FWLandingPattern.FactMetaData.json @@ -1,6 +1,6 @@ [ { - "name": "Landing dist", + "name": "LandingDistance", "shortDescription": "Distance between landing and loiter points.", "type": "double", "units": "m", @@ -9,7 +9,7 @@ "defaultValue": 300.0 }, { - "name": "Landing heading", + "name": "LandingHeading", "shortDescription": "Heading from loiter point to land point.", "type": "double", "units": "deg", @@ -19,7 +19,7 @@ "defaultValue": 270.0 }, { - "name": "Loiter altitude", + "name": "LoiterAltitude", "shortDescription": "Aircraft will proceed to the loiter point and loiter until it reaches this altitude. Once altitude is reached the aircraft will proceed to land.", "type": "double", "units": "m", @@ -27,7 +27,7 @@ "defaultValue": 40.0 }, { - "name": "Loiter radius", + "name": "LoiterRadius", "shortDescription": "Loiter radius.", "type": "double", "decimalPlaces": 1, @@ -36,7 +36,7 @@ "defaultValue": 75.0 }, { - "name": "Landing altitude", + "name": "LandingAltitude", "shortDescription": "Altitude for landing point.", "type": "double", "units": "m", @@ -44,7 +44,7 @@ "defaultValue": 0.0 }, { - "name": "Descent rate", + "name": "DescentRate", "shortDescription": "Descent rate between landing and loiter altitude.", "type": "double", "units": "%", diff --git a/src/MissionManager/FixedWingLandingComplexItem.cc b/src/MissionManager/FixedWingLandingComplexItem.cc index 03e69b8e9932ea6d12b277a664a4cf5797d9d152..e81cb512abc96865179a319bcc15cbdc32363326 100644 --- a/src/MissionManager/FixedWingLandingComplexItem.cc +++ b/src/MissionManager/FixedWingLandingComplexItem.cc @@ -20,12 +20,12 @@ QGC_LOGGING_CATEGORY(FixedWingLandingComplexItemLog, "FixedWingLandingComplexIte const char* FixedWingLandingComplexItem::jsonComplexItemTypeValue = "fwLandingPattern"; -const char* FixedWingLandingComplexItem::_loiterToLandDistanceName = "Landing dist"; -const char* FixedWingLandingComplexItem::_landingHeadingName = "Landing heading"; -const char* FixedWingLandingComplexItem::_loiterAltitudeName = "Loiter altitude"; -const char* FixedWingLandingComplexItem::_loiterRadiusName = "Loiter radius"; -const char* FixedWingLandingComplexItem::_landingAltitudeName = "Landing altitude"; -const char* FixedWingLandingComplexItem::_fallRateName = "Descent rate"; +const char* FixedWingLandingComplexItem::loiterToLandDistanceName = "LandingDistance"; +const char* FixedWingLandingComplexItem::landingHeadingName = "LandingHeading"; +const char* FixedWingLandingComplexItem::loiterAltitudeName = "LoiterAltitude"; +const char* FixedWingLandingComplexItem::loiterRadiusName = "LoiterRadius"; +const char* FixedWingLandingComplexItem::landingAltitudeName = "LandingAltitude"; +const char* FixedWingLandingComplexItem::fallRateName = "DescentRate"; const char* FixedWingLandingComplexItem::_jsonLoiterCoordinateKey = "loiterCoordinate"; const char* FixedWingLandingComplexItem::_jsonLoiterRadiusKey = "loiterRadius"; @@ -35,44 +35,25 @@ const char* FixedWingLandingComplexItem::_jsonLandingCoordinateKey = "lan const char* FixedWingLandingComplexItem::_jsonLandingAltitudeRelativeKey = "landAltitudeRelative"; const char* FixedWingLandingComplexItem::_jsonFallRateKey = "fallRate"; -QMap FixedWingLandingComplexItem::_metaDataMap; - FixedWingLandingComplexItem::FixedWingLandingComplexItem(Vehicle* vehicle, QObject* parent) - : ComplexMissionItem(vehicle, parent) - , _sequenceNumber(0) - , _dirty(false) - , _landingCoordSet(false) - , _ignoreRecalcSignals(false) - , _landingDistanceFact (0, _loiterToLandDistanceName, FactMetaData::valueTypeDouble) - , _loiterAltitudeFact (0, _loiterAltitudeName, FactMetaData::valueTypeDouble) - , _loiterRadiusFact (0, _loiterRadiusName, FactMetaData::valueTypeDouble) - , _landingHeadingFact (0, _landingHeadingName, FactMetaData::valueTypeDouble) - , _landingAltitudeFact (0, _landingAltitudeName, FactMetaData::valueTypeDouble) - , _fallRateFact (0, _fallRateName, FactMetaData::valueTypeDouble) - , _loiterClockwise(true) - , _loiterAltitudeRelative(true) - , _landingAltitudeRelative(true) + : ComplexMissionItem (vehicle, parent) + , _sequenceNumber (0) + , _dirty (false) + , _landingCoordSet (false) + , _ignoreRecalcSignals (false) + , _metaDataMap (FactMetaData::createMapFromJsonFile(QStringLiteral(":/json/FWLandingPattern.FactMetaData.json"), this)) + , _landingDistanceFact (_metaDataMap[loiterToLandDistanceName]) + , _loiterAltitudeFact (_metaDataMap[loiterAltitudeName]) + , _loiterRadiusFact (_metaDataMap[loiterRadiusName]) + , _landingHeadingFact (_metaDataMap[landingHeadingName]) + , _landingAltitudeFact (_metaDataMap[landingAltitudeName]) + , _fallRateFact (_metaDataMap[fallRateName]) + , _loiterClockwise (true) + , _loiterAltitudeRelative (true) + , _landingAltitudeRelative (true) { _editorQml = "qrc:/qml/FWLandingPatternEditor.qml"; - if (_metaDataMap.isEmpty()) { - _metaDataMap = FactMetaData::createMapFromJsonFile(QStringLiteral(":/json/FWLandingPattern.FactMetaData.json"), NULL /* metaDataParent */); - } - - _landingDistanceFact.setMetaData (_metaDataMap[_loiterToLandDistanceName]); - _loiterAltitudeFact.setMetaData (_metaDataMap[_loiterAltitudeName]); - _loiterRadiusFact.setMetaData (_metaDataMap[_loiterRadiusName]); - _landingHeadingFact.setMetaData (_metaDataMap[_landingHeadingName]); - _landingAltitudeFact.setMetaData (_metaDataMap[_landingAltitudeName]); - _fallRateFact.setMetaData (_metaDataMap[_fallRateName]); - - _landingDistanceFact.setRawValue (_landingDistanceFact.rawDefaultValue()); - _loiterAltitudeFact.setRawValue (_loiterAltitudeFact.rawDefaultValue()); - _loiterRadiusFact.setRawValue (_loiterRadiusFact.rawDefaultValue()); - _landingHeadingFact.setRawValue (_landingHeadingFact.rawDefaultValue()); - _landingAltitudeFact.setRawValue (_landingAltitudeFact.rawDefaultValue()); - _fallRateFact.setRawValue (_fallRateFact.rawDefaultValue()); - connect(&_loiterAltitudeFact, &Fact::valueChanged, this, &FixedWingLandingComplexItem::_updateLoiterCoodinateAltitudeFromFact); connect(&_landingAltitudeFact, &Fact::valueChanged, this, &FixedWingLandingComplexItem::_updateLandingCoodinateAltitudeFromFact); diff --git a/src/MissionManager/FixedWingLandingComplexItem.h b/src/MissionManager/FixedWingLandingComplexItem.h index 296fa70f6bc7ed5d2f0702d2611bcdaf1b6d0bd7..9d04c053af843586181049b071bdd2317d2d71e8 100644 --- a/src/MissionManager/FixedWingLandingComplexItem.h +++ b/src/MissionManager/FixedWingLandingComplexItem.h @@ -92,6 +92,13 @@ public: static const char* jsonComplexItemTypeValue; + static const char* loiterToLandDistanceName; + static const char* loiterAltitudeName; + static const char* loiterRadiusName; + static const char* landingHeadingName; + static const char* landingAltitudeName; + static const char* fallRateName; + signals: void loiterCoordinateChanged (QGeoCoordinate coordinate); void loiterTangentCoordinateChanged (QGeoCoordinate coordinate); @@ -122,6 +129,8 @@ private: bool _landingCoordSet; bool _ignoreRecalcSignals; + QMap _metaDataMap; + Fact _landingDistanceFact; Fact _loiterAltitudeFact; Fact _loiterRadiusFact; @@ -133,15 +142,6 @@ private: bool _loiterAltitudeRelative; bool _landingAltitudeRelative; - static QMap _metaDataMap; - - static const char* _loiterToLandDistanceName; - static const char* _loiterAltitudeName; - static const char* _loiterRadiusName; - static const char* _landingHeadingName; - static const char* _landingAltitudeName; - static const char* _fallRateName; - static const char* _jsonLoiterCoordinateKey; static const char* _jsonLoiterRadiusKey; static const char* _jsonLoiterClockwiseKey; diff --git a/src/PlanView/FWLandingPatternEditor.qml b/src/PlanView/FWLandingPatternEditor.qml index 6ce790a4f33a2c36b3ddc09031ba42ae6c7d41b3..b703e9a63fc1cf02a1617871734e52dc7895259b 100644 --- a/src/PlanView/FWLandingPatternEditor.qml +++ b/src/PlanView/FWLandingPatternEditor.qml @@ -52,6 +52,7 @@ Rectangle { anchors.left: parent.left anchors.right: parent.right factList: [ missionItem.loiterAltitude, missionItem.loiterRadius ] + factLabels: [ qsTr("Altitude"), qsTr("Radius") ] } Item { width: 1; height: _spacer } @@ -82,14 +83,14 @@ Rectangle { anchors.right: parent.right columns: 2 - QGCLabel { text: missionItem.landingHeading.name } + QGCLabel { text: qsTr("Heading") } FactTextField { Layout.fillWidth: true fact: missionItem.landingHeading } - QGCLabel { text: missionItem.landingAltitude.name } + QGCLabel { text: qsTr("Altitude") } FactTextField { Layout.fillWidth: true @@ -98,7 +99,7 @@ Rectangle { QGCRadioButton { id: useLandingDistance - text: missionItem.landingDistance.name + text: qsTr("Landing dist") checked: !useFallRate.checked onClicked: { useFallRate.checked = false @@ -115,7 +116,7 @@ Rectangle { QGCRadioButton { id: useFallRate - text: missionItem.fallRate.name + text: qsTr("Descent rate") checked: !useLandingDistance.checked onClicked: { useLandingDistance.checked = false @@ -160,19 +161,12 @@ Rectangle { Column { id: editorColumnNeedLandingPoint anchors.margins: _margin + anchors.top: parent.top anchors.left: parent.left anchors.right: parent.right visible: !missionItem.landingCoordSet spacing: ScreenTools.defaultFontPixelHeight - QGCLabel { - anchors.left: parent.left - anchors.right: parent.right - wrapMode: Text.WordWrap - font.pointSize: ScreenTools.smallFontPointSize - text: qsTr("WIP (NOT FOR REAL FLIGHT!)") - } - QGCLabel { anchors.left: parent.left anchors.right: parent.right