MixedRealityToolkit / MixedRealityToolkit-Unity

This repository holds the third generation of the Mixed Reality Toolkit for Unity. The latest version of the MRTK can be found here.
BSD 3-Clause "New" or "Revised" License
411 stars 111 forks source link

Error retrieving configuration when starting WindowsPhraseRecognitionSubsystem (MRTK3) #326

Closed IssueSyncBot closed 1 year ago

IssueSyncBot commented 1 year ago

Original issue opened by:

@datglove datglove


Describe the bug

Encounter an error-- "Configuration could not be retrieved for Microsoft.MixedReality.Toolkit.Speech.Windows.WindowsPhraseRecognitionSubsystem+WindowsPhraseRecognitionProvider." --when MRTKLifecycleManager attempts to start the MRTK Windows PhraseRecognition Subsystem.

To reproduce

Steps to reproduce the behavior:

  1. (In Editor) enable "MRTK Windows PhraseRecognition Subsystem" in Project Settings > MRTK3
  2. Open Unity scene containing the default MRTK XR Rig prefab
  3. Play
  4. See above error in Console immediately after "[MRTKLifecycleManager] Starting Microsoft.MixedReality.Toolkit.Speech.Windows.WindowsPhraseRecognitionSubsystem"

Expected behavior

PhraseRecognition Subsystem starts with no errors.

Your setup

Target platform

Additional context

Error does not occur after downgrading to com.microsoft.mrtk.windowsspeech 3.0.0-pre.12 .

I believe the problem is in WindowsPhraseRecognitionSubsystem.cs: In WindowsPhraseRecognitionProvider.Start , changing the second type argument in: config = XRSubsystemHelpers.GetConfiguration<WindowsPhraseRecognitionSubsystemConfig, WindowsPhraseRecognitionProvider>(); to: config = XRSubsystemHelpers.GetConfiguration<WindowsPhraseRecognitionSubsystemConfig, WindowsPhraseRecognitionSubsystem>(); appears to fix the error and is consistent with other XRSubsystemHelpers.GetConfiguration calls e.g. in WindowsSpeechRecognitionProvider and the MRTK Hand Aggregator Subsystem.


ISSUE MIGRATION

Issue migrated from: https://github.com/microsoft/MixedRealityToolkit-Unity/issues/11366

IssueSyncBot commented 1 year ago

Original comment by:

@MaxWang-MS MaxWang-MS


Thanks for filing the issue @datglove! I just noticed this as well and it will be fixed in #11348

IssueSyncBot commented 1 year ago

Original comment by:

@MaxWang-MS MaxWang-MS


Fixed via #11348.