Skip to content

Commit

Permalink
Fix controller detection for Pico Neo3 Link (#2192)
Browse files Browse the repository at this point in the history
* fix controller detection for Pico Neo3 Link, show HMD model in app to detect model without connection to server

* Revert change in CHANGELOG.md

* remove display of HMD model from app

Will move the displaying of additional information to a different branch.

* reformat code

* reformat code
  • Loading branch information
HoLo85 authored and zmerp committed Jul 2, 2024
1 parent ef9f2ce commit 4f44b9f
Show file tree
Hide file tree
Showing 4 changed files with 13 additions and 4 deletions.
3 changes: 3 additions & 0 deletions alvr/client_core/src/platform/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@ pub enum Platform {
QuestPro,
QuestUnknown,
PicoNeo3,
PicoNeo3Link,
Pico4,
Focus3,
XRElite,
Expand All @@ -38,6 +39,7 @@ impl Display for Platform {
Platform::QuestPro => "Quest Pro",
Platform::QuestUnknown => "Quest (unknown)",
Platform::PicoNeo3 => "Pico Neo 3",
Platform::PicoNeo3Link => "Pico Neo 3 Link",
Platform::Pico4 => "Pico 4",
Platform::Focus3 => "VIVE Focus 3",
Platform::XRElite => "VIVE XR Elite",
Expand Down Expand Up @@ -69,6 +71,7 @@ pub fn platform() -> Platform {
("Oculus", _, "seacliff") => Platform::QuestPro,
("Oculus", _, _) => Platform::QuestUnknown,
("Pico", "Pico Neo 3", _) => Platform::PicoNeo3,
("Pico", "Pico Neo3 Link", _) => Platform::PicoNeo3Link,
("Pico", _, _) => Platform::Pico4,
("HTC", "VIVE Focus 3", _) => Platform::Focus3,
("HTC", "VIVE XR Series", _) => Platform::XRElite,
Expand Down
2 changes: 1 addition & 1 deletion alvr/client_openxr/src/interaction.rs
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,7 @@ pub fn initialize_interaction(
| Platform::Quest3
| Platform::QuestPro
| Platform::QuestUnknown => QUEST_CONTROLLER_PROFILE_PATH, // todo: create new controller profile for quest pro and 3
Platform::PicoNeo3 => PICO_NEO3_CONTROLLER_PROFILE_PATH,
Platform::PicoNeo3 | Platform::PicoNeo3Link => PICO_NEO3_CONTROLLER_PROFILE_PATH,
Platform::Pico4 => PICO4_CONTROLLER_PROFILE_PATH,
Platform::Focus3 | Platform::XRElite | Platform::ViveUnknown => {
FOCUS3_CONTROLLER_PROFILE_PATH
Expand Down
2 changes: 1 addition & 1 deletion alvr/client_openxr/src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -126,7 +126,7 @@ pub fn entry_point() {
| Platform::Quest3
| Platform::QuestPro
| Platform::QuestUnknown => "_quest",
Platform::PicoNeo3 | Platform::Pico4 => "_pico",
Platform::PicoNeo3 | Platform::PicoNeo3Link | Platform::Pico4 => "_pico",
Platform::Yvr => "_yvr",
Platform::Lynx => "_lynx",
_ => "",
Expand Down
10 changes: 8 additions & 2 deletions alvr/client_openxr/src/stream.rs
Original file line number Diff line number Diff line change
Expand Up @@ -102,7 +102,11 @@ impl StreamContext {
{
alvr_client_core::try_get_permission("com.oculus.permission.EYE_TRACKING")
}
if config.combined_eye_gaze && matches!(platform, Platform::Pico4 | Platform::PicoNeo3)
if config.combined_eye_gaze
&& matches!(
platform,
Platform::Pico4 | Platform::PicoNeo3 | Platform::PicoNeo3Link
)
{
alvr_client_core::try_get_permission("com.picovr.permission.EYE_TRACKING")
}
Expand Down Expand Up @@ -189,7 +193,9 @@ impl StreamContext {
],
config.foveated_encoding_config.clone(),
platform != Platform::Lynx
&& !((platform == Platform::Pico4 || platform == Platform::PicoNeo3)
&& !((platform == Platform::Pico4
|| platform == Platform::PicoNeo3
|| platform == Platform::PicoNeo3Link)
&& config.encoder_config.enable_hdr),
!config.encoder_config.enable_hdr,
config.encoder_config.encoding_gamma,
Expand Down

0 comments on commit 4f44b9f

Please sign in to comment.