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

[OpenXR] Remove DoF filtering #1619

Merged
merged 1 commit into from
Nov 8, 2024
Merged

[OpenXR] Remove DoF filtering #1619

merged 1 commit into from
Nov 8, 2024

Conversation

svillar
Copy link
Member

@svillar svillar commented Nov 5, 2024

We were filtering by DoF in order not to suggest some profiles for devices not supporting 6DoF controllers. On the one hand, this was only needed for HVR so we better handle it as an exception. And on the other hand, there is really no need to filter out profiles in general as the runtime will select the most appropiate one.

@svillar svillar force-pushed the remove_dof_mapping branch 2 times, most recently from 5794cd8 to 4099cf2 Compare November 6, 2024 15:44
We were filtering by DoF in order not to suggest some profiles for
devices not supporting 6DoF controllers. On the one hand, this was
only needed for HVR so we better handle it as an exception. And on
the other hand, there is really no need to filter out profiles in
general as the runtime will select the most appropiate one.
Copy link
Member

@javifernandez javifernandez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks good, thanks.

@svillar svillar merged commit 0e328ec into main Nov 8, 2024
22 checks passed
@svillar svillar deleted the remove_dof_mapping branch November 8, 2024 11:09
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.

2 participants