Skip to content

fix(draw): Fix wheel rotation math#1507

Merged
xezon merged 2 commits into
TheSuperHackers:mainfrom
xezon:xezon/fix-wheels-spin-math
Sep 10, 2025
Merged

fix(draw): Fix wheel rotation math#1507
xezon merged 2 commits into
TheSuperHackers:mainfrom
xezon:xezon/fix-wheels-spin-math

Conversation

@xezon

@xezon xezon commented Aug 30, 2025

Copy link
Copy Markdown

This change just fixes the wheel rotation math. Is unlikely to be user facing but could be if a unit would drive forward for very long distances. Essentially the wheel rotation will keep accumulating and if the float number grows larger enough, then the float imprecision eventually would make the wheels no longer spin.

TODO

@xezon xezon added Minor Severity: Minor < Major < Critical < Blocker Gen Relates to Generals ZH Relates to Zero Hour Fix Is fixing something, but is not user facing labels Aug 30, 2025

@Mauller Mauller left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good

@xezon xezon force-pushed the xezon/fix-wheels-spin-math branch from f0c34ac to d4fe0b9 Compare September 10, 2025 09:41
@xezon

xezon commented Sep 10, 2025

Copy link
Copy Markdown
Author

Replicated in Generals without conflicts.

Tested in Generals and looked ok. Wheels are spinning and power sliding works.

@xezon xezon merged commit 15933f7 into TheSuperHackers:main Sep 10, 2025
19 checks passed
@xezon xezon deleted the xezon/fix-wheels-spin-math branch September 10, 2025 09:58
fbraz3 pushed a commit to fbraz3/GeneralsX that referenced this pull request Nov 10, 2025
fbraz3 pushed a commit to fbraz3/GeneralsX that referenced this pull request Feb 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Fix Is fixing something, but is not user facing Gen Relates to Generals Minor Severity: Minor < Major < Critical < Blocker ZH Relates to Zero Hour

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants