Modify actions for sequential calibration

[willfordrockwell]

Hello.

I spent 2 days with wrong actions at sequential mode calibration on the G25 shifter.

When I read, "Please move the gear shifter to sequentially shift up", I watched a drawing on the shifter and pulled the lever toward me.

When I read, "Please move the gear shifter to sequentially shift down", I watched a drawing on the shifter and pushed the lever away from me.

But calibration meant opposite action.

Could you please rename commands at sequential calibration from
"Please move the gear shifter to sequentially shift UP/DOWN"
to "Please move the shifter lever AWAY FROM YOU/TOWARD YOU"?

Best wishes.

[dmadison]

Hello! Thanks for creating an issue. I think this is a good suggestion. I'll make that change and push a new release.

I watched a drawing on the shifter and pulled the lever toward me.

Could you clarify this? On my own shifter there is a small drawing on the selection knob which has a '+' sign with an arrow facing away from you, and a '-' sign with an arrow facing towards you. Is your shifter different?

[dmadison]

Resolved by PR #22. This will be included in release 2.0.1.

[willfordrockwell]

On my own shifter there is a small drawing on the selection knob which has a '+' sign with an arrow facing away from you, and a '-' sign with an arrow facing towards you. Is your shifter different?

Yes, my shifter is different:

[dmadison]

Huh. It seems like they must have produced two different versions since I can find photos of both online. How odd.

I re-checked the photos of my own shifter and I was wrong, it also has the '+' towards the user:

That's probably the "correct" way to do it, since a sequential shift lever in a real car would also be mounted like that so G-forces under braking / acceleration would help you shift.

But that does mean that the library's implementation is wrong and should be corrected. Thanks for bringing this to my attention. I'll try to find some time to make the necessary changes.