1hubert / genshin-dialogue-autoskip

A simple python script to automatically skip dialogue in Genshin Impact
MIT License
61 stars 23 forks source link

Merge controller branch into main #35

Closed Livour closed 9 months ago

Livour commented 9 months ago

Add the functionality of the controller branch into the main branch ,It's like 20 commits behind.

I could implement this myself but It says that the Controller branch's readme that it is still experimental (Not sure if Readme is up to date). If it's really is still experimental, what other tests/features does it need?

and btw, Is it supposed to work automatically when switching between keyboard/gamepad or do you need to update the .env?

1hubert commented 9 months ago

Hey, Livour. I appreciate your concern, but I'd rather keep the gamepad functionality somewhere else than the main script, for three reasons:

When xdenotte added Xbox gamepad support here, he said "there may be bugs, especially since I don’t know if DS4 works since because I don’t have it". That's the issue with gamepads. That's why I created a separate branch and labeled it as "Experimental" (might not be the semantically right word, but basically I meant "expect bugs").

Today, I've updated the gamepad branch to basically the newest version of the main branch, but exclusively for gamepads. The ahead/behind commit count is not a meaningful measure, as those branches are not planned to be merged.

and btw, Is it supposed to work automatically when switching between keyboard/gamepad or do you need to update the .env?

The gamepad branch is supposed to switch automatically between different gamepads, it doesn't support keyboard. The .env works in the exact same way as in the main branch; it stores screen dimensions.

I hope you understand <3

Have a nice day!