VIU_WAVEVR is not defined with Wave XR SDK 4.x/5.x

[mhama]

Hi, I'm a bit confused when I import Wave XR SDK 4.x/5.x, because it looks like VIU_WAVEVR is not defined automatically.
My understanding is that to use WaveVR module, VIU_WAVEVR should be defined.

The condition to define VIU_WAVEVR is this:

ViveInputUtility-Unity/Assets/HTC.UnityPlugin/VRModule/Modules/Editor/WaveVRModuleEditor.cs

Lines 21 to 26 in 87340ea

	Add(new SymbolRequirement()
	{
	symbol = "VIU_WAVEVR",
	reqTypeNames = new string[] { "WaveVR" },
	reqFileNames = new string[] { "WaveVR.cs" },
	});

But, I can't find WaveVR type, nor WaveVR.cs in Wave XR SDK 4.x/5.x. And it seems like VIU_WAVEVR is not defined actually.
Though, it looks like I can still enable WaveVR module in VIUSetting.

Is it expected behaviour?

[chengnay]

Hi @mhama,
VIU_WAVEVR is for previous version which is imported from unitypackage,
Current Wave XR SDK 4.x/5.x will use different define.
Are you getting any error or issue?

[mhama]

VIU_WAVEVR is for previous version which is imported from unitypackage,
Current Wave XR SDK 4.x/5.x will use different define.
Are you getting any error or issue?

Hi, @chengnay ,
Thank you for the reply!

There's no error/issues. It's (mostly) ok if it's intentional.
I just wanted to split our app's code which depends on Wave SDK using VIU_WAVEVR and found this. Though the symbol maybe not designed to be used like this.

But there may be a bit of a problem about readability of the code.
I confused a bit when I read the code of WaveVRModule.cs .

• About property isWaveVRPluginDetected
  • I think that the functionality of this property seems to be a bit different from the name. I thought that this property returns true when Wave SDK is detected, whatever version it is.
    

    ViveInputUtility-Unity/Assets/HTC.UnityPlugin/VRModule/Modules/WaveVRModule.cs

    Lines 22 to 27 in 87340ea

    	public static readonly bool isWaveVRPluginDetected =
    	#if VIU_WAVEVR
    	true;
    	#else
    	false;
    	#endif

  • If this property is only for older versions of SDK, the name of the property is better to represent it.

Though, it's just my opinion and what's really important is that it works. So please take it with a grain of salt.
Thanks,

[chengnay]

Hi @mhama,

Thanks for your comments.
For latest Wave SDK using UnityXR, we change it to WAVEXR.
Just curious why you will need to split your code if you are using VIU?
I think spliting code should be already done from VIU side.
Maybe you are using some feature that VIU did not provide, such as passthrough.

[mhama]

Hi @chengnay, thanks for your reply,

For latest Wave SDK using UnityXR, we change it to WAVEXR.
Just curious why you will need to split your code if you are using VIU?
I think spliting code should be already done from VIU side.
Maybe you are using some feature that VIU did not provide, such as passthrough.

Yes, we wanted to use video pass-through functionality of WaveVR SDK.

[chengnay]

Hi @mhama,
You can add your custom defined symbols to differentiate between Oculus and Wave.
Unity Editor>Project Settings>Player>Other Settings(Script Compilation)