-
Notifications
You must be signed in to change notification settings - Fork 12
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
[Feature Request] Sunshine support #4
Comments
Should be released tomorrow or maybe even today in the evening :) |
The logic used in all those forks is faulty, when you reconnect a physical (EDIT: even with Sunshine virtual controller the behaviour is the same) controller, the program will just keep detecting new devices, e.g.: Normal behaviour: 2024-09-02.17-54-07.mp4Behaviour with the faulty code: 2024-09-02.17-53-04.mp4I will investigate and try making it work properly but it might take longer than previously anticipated. |
A temporary solution that I will be adding to the newest beta (available here) is a 5-second detection cooldown which makes the program ignore the virtual devices that appear after the real one, however there are 2 tradeoffs:
This solution is temporary and should be ultimately replaced by code that actually detects whether or not the detected device should be added to the controllers list. |
Having tried the code on my laptop, I found out that I cannot replicate the behaviour from #4 (comment), therefore I've decided to add that code as the default Moonlight support and kept #4 (comment) as an optional, Advanced Support. If there are no bugs reported related this feature this week, I'll close the issue. |
Advanced mode working perfectly here! "Simple mode" gives me 8 controllers issue as expected in the documentation |
Working great with a hacked Nintendo Switch and motion control. Simple mode gave me 2 extra virtual controllers. |
Hello,
I saw in other recent DS4Windows forks that Sunshine support is one of the most important commit. Maybe it's worth to add some checkbox, where users can enable optional Sunshine support like they have in forks below:
Shev0@2dbac2b
BotAlejandro@09913e7
ragestaker21@9abf11c
galax20002@a718f3a
Related to:
Ryochan7#3152
The text was updated successfully, but these errors were encountered: