Skip to content

feat: add OpenVR service and autostart with SteamVR option#127

Merged
RedHawk989 merged 54 commits intoEyeTrackVR:v2.0-beta-feature-branchfrom
Fracas42:v2.0-beta-feature-branch
Dec 14, 2025
Merged

feat: add OpenVR service and autostart with SteamVR option#127
RedHawk989 merged 54 commits intoEyeTrackVR:v2.0-beta-feature-branchfrom
Fracas42:v2.0-beta-feature-branch

Conversation

@Fracas42
Copy link
Contributor

Description

Add OpenVR support and an option to start and stop the app with SteamVR

The autostart option can only be activated when SteamVR is running. Currently, to make it more visible, the checkbox is unchecked when the autostart option couldn't be enabled, and a warn message is displayed in the console.
Please suggest a better solution if you know one, I'm not familiar with PySimpleGUI.

Checklist

  • The pull request is done against the latest development branch

  • Only relevant files were touched

  • Code change compiles without warnings

  • The code change is tested and works with EyeTrackVR core ESP32 newest release

  • I accept the CLA.

@Fracas42 Fracas42 force-pushed the v2.0-beta-feature-branch branch 4 times, most recently from 0cbfe08 to ae074ea Compare December 14, 2024 00:46
@Fracas42 Fracas42 force-pushed the v2.0-beta-feature-branch branch from ae074ea to 549217c Compare December 14, 2024 01:23
@Fracas42
Copy link
Contributor Author

I added a popup that will appear if you try to enable Start with SteamVR while SteamVR is not running.
Right now, I'm getting the window after its creation in eyetrackvr then assigning it to the OpenVR Service class static instance for accessing it later.
I should probably get it another way, by storing it globally in eyetrackvr, or from the event listener args.

@RedHawk989
Copy link
Collaborator

This breaks MacOS and possibly Linux compatibility. Can you make sure OpenVR is only being imported and called on supported Windows systems? I pushed a fix for the installation but not the calls.

@Fracas42
Copy link
Contributor Author

Huh? Does it?
The package got .so and .dylib so it should work on linux and mac
Is it when packaged? Right now I only bundle the .dll in the exe (assuming pyinstaller was only used for the windows version)

@RedHawk989
Copy link
Collaborator

On Mac, it breaks primarily due to arm stuff (it blurts out a long error, with that being the ultimate cause from what I can tell). I don't have my Linux environment set up at the moment to test it fully.

@RedHawk989
Copy link
Collaborator

I can dig into it more later and fix the Mac-specific error myself, Linux will be a bit delayed, but I would like to confirm it works before merging.

@Fracas42
Copy link
Contributor Author

I'll try on linux, can you post the error from MacOS? I don't have a MacOs environment setup

@Fracas42
Copy link
Contributor Author

Fracas42 commented Dec 28, 2024

I reworked the error handling to avoid crashes and get better error messages.
I also tested on linux and was able to run the program fine non-bundled, I did not tested it with SteamVR as I cannot connect an headset to my linux environnement.

@Fracas42 Fracas42 force-pushed the v2.0-beta-feature-branch branch from a39a1d5 to 9e97667 Compare December 28, 2024 13:53
@Fracas42 Fracas42 force-pushed the v2.0-beta-feature-branch branch from 9e97667 to 36c8a4b Compare January 2, 2025 19:48
@Fracas42
Copy link
Contributor Author

Fracas42 commented Jan 6, 2025

Any updates for MacOS @RedHawk989 ?

@RedHawk989 RedHawk989 self-requested a review January 8, 2025 22:03
@RedHawk989
Copy link
Collaborator

Sorry for the wait. I fixed the support on my end by preventing the import if the detected system is mac os. If you could make sure I did not break anything, I will merge it.

Copilot AI and others added 28 commits December 7, 2025 09:28
… start

Co-authored-by: m-RNA <41933749+m-RNA@users.noreply.github.com>
Co-authored-by: m-RNA <41933749+m-RNA@users.noreply.github.com>
Co-authored-by: m-RNA <41933749+m-RNA@users.noreply.github.com>
Co-authored-by: m-RNA <41933749+m-RNA@users.noreply.github.com>
Co-authored-by: m-RNA <41933749+m-RNA@users.noreply.github.com>
Co-authored-by: m-RNA <41933749+m-RNA@users.noreply.github.com>
Co-authored-by: m-RNA <41933749+m-RNA@users.noreply.github.com>
Co-authored-by: m-RNA <41933749+m-RNA@users.noreply.github.com>
Co-authored-by: m-RNA <41933749+m-RNA@users.noreply.github.com>
Co-authored-by: m-RNA <41933749+m-RNA@users.noreply.github.com>
Co-authored-by: m-RNA <41933749+m-RNA@users.noreply.github.com>
Co-authored-by: m-RNA <41933749+m-RNA@users.noreply.github.com>
Co-authored-by: m-RNA <41933749+m-RNA@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@RedHawk989
Copy link
Collaborator

oh....... well git mistake but it works.... heh......

@RedHawk989 RedHawk989 merged commit b438a80 into EyeTrackVR:v2.0-beta-feature-branch Dec 14, 2025
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.

6 participants