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

Add wheel rotation lock to Gimx configs. #709

Open
wants to merge 16 commits into
base: master
Choose a base branch
from

Conversation

crowbarmaster
Copy link

Hello and good day Matlo!

I want to start out with a thank you for the wonderful piece of software you wrote for the world to enjoy. I have been using GIMX for a few weeks now with a home-built adapter, and it works well!

However, nothing is ever perfect. I was pretty well used to drifting in GT6 under the 540* setting available with the button combo on my G27. So when I fired up GT7 for the first time to find out this was not an option, I panicked a bit. After having a look around at the source, I had found that your layout for haptic_tweaks spelled out a switch case for RANGE. Perfect, I now had a hard-coded version of GIMX for 540* lock.

I spent the couple days figuring out CodeBlocks and wxSmith as I desired more and more to share this. I had seen quite a few posts asking if this was possible, so I wanted to put some effort down to help ease this request. Gimx-Config now includes a separate control in the FFB Tweaks tab to control wheel rotation ceiling, mirrored logic to FFBTweaks. This creates a whole new node within the config to avoid conflicts with existing configs, structured like so:

      <wheel_settings>
        <device type="joystick" id="0" name="Logitech G27 Racing Wheel USB"/>
        <wheel_rotation degrees="540"/>
      </wheel_settings>

Please, have a look at my PR here, feel free to criticize my work as you feel necessary. C/C++ is not my primary language. Some technical details here, I added a whole new node to the XML configs to avoid causing issues with currently modified config files. I have done some basic debugging and everything seems to work as intended.

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.

1 participant