Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix controller detection for Pico Neo3 Link #2192

Merged
merged 6 commits into from
Jun 15, 2024

Conversation

HoLo85
Copy link
Contributor

@HoLo85 HoLo85 commented Jun 15, 2024

Fixes the detection of the controllers for the Pico Neo3 Link headset. With the introduction of the Pico 4 the HMD was detected as this model which broke the detection of the controller.

HoLo85 and others added 2 commits June 15, 2024 03:43
Fix controller detection for Pico Neo3 Link
CHANGELOG.md Outdated Show resolved Hide resolved
alvr/client_core/src/connection.rs Outdated Show resolved Hide resolved
HoLo85 added 2 commits June 15, 2024 12:48
Will move the displaying of additional information to a different branch.
@The-personified-devil
Copy link
Collaborator

You'll need to autoformat the changed code to fix the check

@HoLo85
Copy link
Contributor Author

HoLo85 commented Jun 15, 2024

You'll need to autoformat the changed code to fix the check

Are those suggestions by rustfmt and are all listed changes marked as error? I think my changes are already formatted for a good readability.

@The-personified-devil
Copy link
Collaborator

You should format everything according to rustfmt

@Vixea
Copy link
Collaborator

Vixea commented Jun 15, 2024

You need to run cargo fmt we don't care if you think you're code is readable to you. You must follow rustfmt's rules.

@Meister1593 Meister1593 self-requested a review June 15, 2024 14:32
@HoLo85 HoLo85 requested a review from Vixea June 15, 2024 14:43
@The-personified-devil The-personified-devil merged commit b9d2503 into alvr-org:master Jun 15, 2024
8 checks passed
zmerp pushed a commit that referenced this pull request Jul 2, 2024
* 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
zmerp pushed a commit that referenced this pull request Jul 2, 2024
* 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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants