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

About DualShock 3 triggers in one of mappings #247

Closed
CookiePLMonster opened this issue Feb 10, 2019 · 7 comments
Closed

About DualShock 3 triggers in one of mappings #247

CookiePLMonster opened this issue Feb 10, 2019 · 7 comments

Comments

@CookiePLMonster
Copy link
Contributor

Hey,

while I realize this database is not supposed to overwrite stock SDL mappings, there is at least one which is plain wrong.

I'm talking about one of DS3 mappings, namely the one with GUID 050000004c0500006802000000800000. In SDL (and thus in this database too) its triggers are defined as:

lefttrigger:b6,righttrigger:b7

however, mapping this specific controller in Steam Link app gives a better mapping, where triggers are axes:

lefttrigger:a2,righttrigger:a5

What should be done about this? Other DualShock 3 mappings have correctly mapped triggers, this one is an outlier and my pad just happens to use that mapping on Raspbian.

@p-groarke
Copy link
Contributor

No worries, the best approach is to open a ticket on the SDL forums, or send them a pull request directly.

Like you said, we don't change their mappings. When they fix theirs, we'll get the update. Cheers o/

@p-groarke p-groarke reopened this Feb 10, 2019
@p-groarke
Copy link
Contributor

I'll leave this open. You can ping me once the fixes are in.

@CookiePLMonster
Copy link
Contributor Author

Sounds good! Do you know how do they even accept PR's, though? Looks like the git repository on GitHub is just a mirror, and an actual master repository is on hg, which I'm not familiar with.

@p-groarke
Copy link
Contributor

Yeah they use mercurial. Its pretty similar to git from what I remember. Other than that can't really help you, sorry. I think they have a dev guide though.

@CookiePLMonster
Copy link
Contributor Author

I have now submitted a patch for this mapping:
https://bugzilla.libsdl.org/show_bug.cgi?id=4504

Fingers crossed it gets answered in a timely manner!

@CookiePLMonster
Copy link
Contributor Author

Seems like the fix has been accepted upstream:

https://bugzilla.libsdl.org/show_bug.cgi?id=4504#c1
https://hg.libsdl.org/SDL/rev/3d2042ac4ab9

@p-groarke
Copy link
Contributor

Great! Once SDL 2.0.10 releases, I'll update our SDL mappings and the fix will roll in. Thx for going through the process :)

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

No branches or pull requests

2 participants