diff options
author | Yana Chernysheva (GitHub) <59469418+ychernysheva@users.noreply.github.com> | 2020-08-10 17:27:26 +0300 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-08-10 10:27:26 -0400 |
commit | 6e5ef3dd422f8da25d224928979a8c7e59d54723 (patch) | |
tree | 86f71600cea8e6b6f89f6ea7013e749604726ad2 /src | |
parent | b46921b8b185b710f4ec5778b55c4b5055e9ca62 (diff) | |
download | sdl_core-6e5ef3dd422f8da25d224928979a8c7e59d54723.tar.gz |
SDL Core implementation of feature [0257] New vehicle data HandsOffSteering (#3394)
* Add changes to HMI_API
* Add changes related to MessageHelper
* Add the new parameter into the sdl_preloaded_pt and policy_table_interface_ext
files
The new parameter "handsOffSteering" was added into the
"DrivingCharacteristics-3" section of the sdl_preloaded_pt file.
Also the new parameter was added into the policy_table_interface_ext file
* Update the sdl_preloaded_pt.json file with the new schema item
Co-authored-by: Igor Gapchuk <igapchuck@luxoft.com>
Co-authored-by: YarikMamykin <ymamykin@gmail.com>
Co-authored-by: Andrii Kalinich <AKalinich@luxoft.com>
Diffstat (limited to 'src')
10 files changed, 76 insertions, 21 deletions
diff --git a/src/appMain/sdl_preloaded_pt.json b/src/appMain/sdl_preloaded_pt.json index 860c83d2fc..e07a5eeab0 100644 --- a/src/appMain/sdl_preloaded_pt.json +++ b/src/appMain/sdl_preloaded_pt.json @@ -507,7 +507,8 @@ "prndl", "rpm", "steeringWheelAngle", - "gearStatus" + "gearStatus", + "handsOffSteering" ] }, "OnVehicleData": { @@ -525,7 +526,8 @@ "prndl", "rpm", "steeringWheelAngle", - "gearStatus" + "gearStatus", + "handsOffSteering" ] }, "SubscribeVehicleData": { @@ -543,7 +545,8 @@ "prndl", "rpm", "steeringWheelAngle", - "gearStatus" + "gearStatus", + "handsOffSteering" ] }, "UnsubscribeVehicleData": { @@ -561,7 +564,8 @@ "prndl", "rpm", "steeringWheelAngle", - "gearStatus" + "gearStatus", + "handsOffSteering" ] } } @@ -4276,6 +4280,13 @@ "key": "OEM_REF_MY_KEY", "type": "Struct", "mandatory": false + }, + { + "name": "handsOffSteering", + "key": "OEM_REF_HANDS_OFF_STRNG", + "type": "Boolean", + "mandatory": false, + "since": "7.0" } ] } diff --git a/src/components/application_manager/include/application_manager/smart_object_keys.h b/src/components/application_manager/include/application_manager/smart_object_keys.h index fe7dbbdc26..c96c9ebe69 100644 --- a/src/components/application_manager/include/application_manager/smart_object_keys.h +++ b/src/components/application_manager/include/application_manager/smart_object_keys.h @@ -340,6 +340,7 @@ extern const char* priority; extern const char* engine_oil_life; extern const char* oem_custom_data_type; extern const char* window_status; +extern const char* hands_off_steering; // app services extern const char* app_service_manifest; diff --git a/src/components/application_manager/src/message_helper/message_helper.cc b/src/components/application_manager/src/message_helper/message_helper.cc index 969307ad68..83e93d9c80 100644 --- a/src/components/application_manager/src/message_helper/message_helper.cc +++ b/src/components/application_manager/src/message_helper/message_helper.cc @@ -245,7 +245,10 @@ std::pair<std::string, mobile_apis::VehicleDataType::eType> std::make_pair(strings::engine_oil_life, mobile_apis::VehicleDataType::VEHICLEDATA_ENGINEOILLIFE), std::make_pair(strings::window_status, - mobile_apis::VehicleDataType::VEHICLEDATA_WINDOWSTATUS)}; + mobile_apis::VehicleDataType::VEHICLEDATA_WINDOWSTATUS), + std::make_pair( + strings::hands_off_steering, + mobile_apis::VehicleDataType::VEHICLEDATA_HANDSOFFSTEERING)}; const VehicleData MessageHelper::vehicle_data_( kVehicleDataInitializer, diff --git a/src/components/application_manager/src/smart_object_keys.cc b/src/components/application_manager/src/smart_object_keys.cc index e193967f7e..f95476b7a6 100644 --- a/src/components/application_manager/src/smart_object_keys.cc +++ b/src/components/application_manager/src/smart_object_keys.cc @@ -307,6 +307,7 @@ const char* priority = "priority"; const char* engine_oil_life = "engineOilLife"; const char* oem_custom_data_type = "oemCustomDataType"; const char* window_status = "windowStatus"; +const char* hands_off_steering = "handsOffSteering"; // app services const char* app_service_manifest = "appServiceManifest"; diff --git a/src/components/interfaces/HMI_API.xml b/src/components/interfaces/HMI_API.xml index 06f472224b..116ab91060 100644 --- a/src/components/interfaces/HMI_API.xml +++ b/src/components/interfaces/HMI_API.xml @@ -1217,6 +1217,7 @@ <element name="VEHICLEDATA_OEM_CUSTOM_DATA" /> <element name="VEHICLEDATA_STABILITYCONTROLSSTATUS" /> <element name="VEHICLEDATA_WINDOWSTATUS"/> + <element name="VEHICLEDATA_HANDSOFFSTEERING"/> </enum> <enum name="WiperStatus"> @@ -6169,6 +6170,9 @@ <param name="windowStatus" type="Boolean" mandatory="false"> <description>See WindowStatus</description> </param> + <param name="handsOffSteering" type="Boolean" mandatory="false"> + <description>To indicate whether driver hands are off the steering wheel</description> + </param> <!-- Ford Specific Data Items --> <param name="eCallInfo" type="Boolean" mandatory="false"> @@ -6274,6 +6278,9 @@ <param name="windowStatus" type="Common.VehicleDataResult" mandatory="false"> <description>See WindowStatus</description> </param> + <param name="handsOffSteering" type="Common.VehicleDataResult" mandatory="false"> + <description>To indicate whether driver hands are off the steering wheel</description> + </param> <!-- Ford Specific Data Items --> <param name="eCallInfo" type="Common.VehicleDataResult" mandatory="false"> @@ -6382,6 +6389,9 @@ <param name="windowStatus" type="Boolean" mandatory="false"> <description>See WindowStatus</description> </param> + <param name="handsOffSteering" type="Boolean" mandatory="false"> + <description>To indicate whether driver hands are off the steering wheel</description> + </param> <!-- Ford Specific Data Items --> <param name="eCallInfo" type="Boolean" mandatory="false"> @@ -6487,6 +6497,9 @@ <param name="windowStatus" type="Common.VehicleDataResult" mandatory="false"> <description>See WindowStatus</description> </param> + <param name="handsOffSteering" type="Common.VehicleDataResult" mandatory="false"> + <description>To indicate whether driver hands are off the steering wheel</description> + </param> <!-- Ford Specific Data Items --> <param name="eCallInfo" type="Common.VehicleDataResult" mandatory="false"> @@ -6596,6 +6609,9 @@ <param name="windowStatus" type="Boolean" mandatory="false"> <description>See WindowStatus</description> </param> + <param name="handsOffSteering" type="Boolean" mandatory="false"> + <description>To indicate whether driver hands are off the steering wheel</description> + </param> <!-- Ford Specific Data Items --> <param name="eCallInfo" type="Boolean" mandatory="false"> @@ -6704,6 +6720,9 @@ <param name="windowStatus" type="Common.WindowStatus" array="true" minsize="0" maxsize="100" mandatory="false"> <description>See WindowStatus</description> </param> + <param name="handsOffSteering" type="Boolean" mandatory="false"> + <description>To indicate whether driver hands are off the steering wheel</description> + </param> <!-- Ford Specific Data Items --> <param name="eCallInfo" type="Common.ECallInfo" mandatory="false"> @@ -6813,6 +6832,9 @@ <param name="windowStatus" type="Common.WindowStatus" array="true" minsize="0" maxsize="100" mandatory="false"> <description>See WindowStatus</description> </param> + <param name="handsOffSteering" type="Boolean" mandatory="false"> + <description>To indicate whether driver hands are off the steering wheel</description> + </param> <!-- Ford Specific Data Items --> <param name="eCallInfo" type="Common.ECallInfo" mandatory="false"> diff --git a/src/components/policy/policy_external/include/policy/policy_table_interface_ext.xml b/src/components/policy/policy_external/include/policy/policy_table_interface_ext.xml index a723177c33..7446d3acae 100644 --- a/src/components/policy/policy_external/include/policy/policy_table_interface_ext.xml +++ b/src/components/policy/policy_external/include/policy/policy_table_interface_ext.xml @@ -59,6 +59,7 @@ <element name="tirePressureValue" /> <element name="tpms" /> <element name="cloudAppVehicleID" /> + <element name="handsOffSteering" /> </enum> <enum name="AppHMIType"> diff --git a/src/components/policy/policy_external/test/json/sdl_preloaded_pt_for_merge_initial.json b/src/components/policy/policy_external/test/json/sdl_preloaded_pt_for_merge_initial.json index dbef04e0b8..dec76dd96c 100644 --- a/src/components/policy/policy_external/test/json/sdl_preloaded_pt_for_merge_initial.json +++ b/src/components/policy/policy_external/test/json/sdl_preloaded_pt_for_merge_initial.json @@ -2056,7 +2056,8 @@ "myKey", "prndl", "rpm", - "steeringWheelAngle" + "steeringWheelAngle", + "handsOffSteering" ] }, "OnVehicleData" : @@ -2071,7 +2072,8 @@ "myKey", "prndl", "rpm", - "steeringWheelAngle" + "steeringWheelAngle", + "handsOffSteering" ] }, "SubscribeVehicleData" : @@ -2086,7 +2088,8 @@ "myKey", "prndl", "rpm", - "steeringWheelAngle" + "steeringWheelAngle", + "handsOffSteering" ] }, "UnsubscribeVehicleData" : @@ -2101,7 +2104,8 @@ "myKey", "prndl", "rpm", - "steeringWheelAngle" + "steeringWheelAngle", + "handsOffSteering" ] } }, diff --git a/src/components/policy/policy_external/test/json/sdl_preloaded_pt_for_merge_latest.json b/src/components/policy/policy_external/test/json/sdl_preloaded_pt_for_merge_latest.json index 3260558e70..85693ff515 100644 --- a/src/components/policy/policy_external/test/json/sdl_preloaded_pt_for_merge_latest.json +++ b/src/components/policy/policy_external/test/json/sdl_preloaded_pt_for_merge_latest.json @@ -2056,7 +2056,8 @@ "myKey", "prndl", "rpm", - "steeringWheelAngle" + "steeringWheelAngle", + "handsOffSteering" ] }, "OnVehicleData" : @@ -2071,7 +2072,8 @@ "myKey", "prndl", "rpm", - "steeringWheelAngle" + "steeringWheelAngle", + "handsOffSteering" ] }, "SubscribeVehicleData" : @@ -2086,7 +2088,8 @@ "myKey", "prndl", "rpm", - "steeringWheelAngle" + "steeringWheelAngle", + "handsOffSteering" ] }, "UnsubscribeVehicleData" : @@ -2101,7 +2104,8 @@ "myKey", "prndl", "rpm", - "steeringWheelAngle" + "steeringWheelAngle", + "handsOffSteering" ] } }, diff --git a/src/components/policy/policy_regular/test/json/sdl_preloaded_pt_for_merge_initial.json b/src/components/policy/policy_regular/test/json/sdl_preloaded_pt_for_merge_initial.json index 909fd2cb2f..3855ccd481 100644 --- a/src/components/policy/policy_regular/test/json/sdl_preloaded_pt_for_merge_initial.json +++ b/src/components/policy/policy_regular/test/json/sdl_preloaded_pt_for_merge_initial.json @@ -2056,7 +2056,8 @@ "myKey", "prndl", "rpm", - "steeringWheelAngle" + "steeringWheelAngle", + "handsOffSteering" ] }, "OnVehicleData" : @@ -2071,7 +2072,8 @@ "myKey", "prndl", "rpm", - "steeringWheelAngle" + "steeringWheelAngle", + "handsOffSteering" ] }, "SubscribeVehicleData" : @@ -2086,7 +2088,8 @@ "myKey", "prndl", "rpm", - "steeringWheelAngle" + "steeringWheelAngle", + "handsOffSteering" ] }, "UnsubscribeVehicleData" : @@ -2101,7 +2104,8 @@ "myKey", "prndl", "rpm", - "steeringWheelAngle" + "steeringWheelAngle", + "handsOffSteering" ] } }, diff --git a/src/components/policy/policy_regular/test/json/sdl_preloaded_pt_for_merge_latest.json b/src/components/policy/policy_regular/test/json/sdl_preloaded_pt_for_merge_latest.json index 3260558e70..85693ff515 100644 --- a/src/components/policy/policy_regular/test/json/sdl_preloaded_pt_for_merge_latest.json +++ b/src/components/policy/policy_regular/test/json/sdl_preloaded_pt_for_merge_latest.json @@ -2056,7 +2056,8 @@ "myKey", "prndl", "rpm", - "steeringWheelAngle" + "steeringWheelAngle", + "handsOffSteering" ] }, "OnVehicleData" : @@ -2071,7 +2072,8 @@ "myKey", "prndl", "rpm", - "steeringWheelAngle" + "steeringWheelAngle", + "handsOffSteering" ] }, "SubscribeVehicleData" : @@ -2086,7 +2088,8 @@ "myKey", "prndl", "rpm", - "steeringWheelAngle" + "steeringWheelAngle", + "handsOffSteering" ] }, "UnsubscribeVehicleData" : @@ -2101,7 +2104,8 @@ "myKey", "prndl", "rpm", - "steeringWheelAngle" + "steeringWheelAngle", + "handsOffSteering" ] } }, |