smartdevicelink / sdl_core

SmartDeviceLink In-Vehicle Software and Sample HMI
BSD 3-Clause "New" or "Revised" License
241 stars 244 forks source link

Multiple media and navigation apps activation during active embedded audio source+audio mixing supported #1918

Closed GetmanetsIrina closed 4 years ago

GetmanetsIrina commented 6 years ago

Occurrence: Always

Description

Multiple media and navigation apps activation during active embedded audio source+audio mixing supported

Preconditions

Steps to reproduce

Actual result

SDL does not set required HMILevel and audioStreamingState.

Expected result

  1. SDL -> navigation app_2 : OnHMIStatus (LIMITED, AUDIBLE)
  2. SDL -> HMI: SDL.ActivateApp (SUCCESS)
  3. SDL -> media app_1 : OnHMIStatus (FULL, AUDIBLE)

Environment

Attachments

Expected delivery

dboltovskyi commented 6 years ago

Priority was set to Medium. Issue is related to setting wrong HMI level and/or audioStreamingState for activated application in some specific cases. May affect correct permissions calculation for navi activated application.

HSavynetska commented 4 years ago

The issue is relevant on the current develop branch

Environment: Test script: https://github.com/smartdevicelink/sdl_atf_test_scripts/pull/2089 SDL build: https://github.com/smartdevicelink/sdl_core/tree/develop ( e137556 ) ATF: develop branch

E-SAITO-TMC commented 4 years ago

@joeygrover -san @theresalech -san This issue is a top priority for Toyota. Can the reproduction steps described actually reproduce this issue? Also, Does this issue occur in the latest version?

theresalech commented 4 years ago

Hi @E-SAITO-TMC, we have documented that this is priority for Toyota, and will include for Steering Committee review and vote when planning the contents for the Q4 Core release. This planning will take place in June. @HSavynetska, can you please advise if you are seeing this issue when using Core 6.1?

dboltovskyi commented 4 years ago

@theresalech Please notice issue is still actual for Core 6.1 ATF script and logs: 2020-06-02_20-52-08_1918.tar.gz

theresalech commented 4 years ago

Hi @dboltovskyi, thank you for the additional information. Will Luxoft be submitting a PR to resolve this issue?

dboltovskyi commented 4 years ago

@theresalech Please notice we're not planning to create the fix in scope of 6.2, though will try to do as much as possible

jacobkeeler commented 4 years ago

Closing with the merge of #3476