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

Change: USA War Factory door time from 3250 to 3300 ms #787

Merged
merged 1 commit into from
Jul 30, 2022

Conversation

xezon
Copy link
Collaborator

@xezon xezon commented Jul 29, 2022

Relates to

This change fixes the slow door close time of USA War Factory.

Object DoorOpeningTime DoorWaitOpenTime DoorCloseTime
Original USA Warfactory 3250 3000 4000
Patched USA Warfactory (this) 3300 3000 3300

With an update clock of 66.666 ms we can calculate that actual door time will be 3250 Modulus 66.66666 = 50, leading in 3250 + 50 = 3300 ms. Setting Open door time to 3300 makes it explicit and easier to comprehend. Closing door change is inconsequential to gameplay, as it does not affect any unit interactions.

This change is cosmetic.

Original

usa_wf_original.mp4

Patched

usa_wf_patch.mp4

@xezon xezon added Design Is a matter of game design Minor Severity: Minor < Major < Critical < Blocker labels Jul 29, 2022
@xezon xezon force-pushed the door-fix-america-wf branch from 9d66a79 to 2250b0a Compare July 30, 2022 14:33
@xezon xezon merged commit 1d4cd64 into main Jul 30, 2022
@xezon xezon deleted the door-fix-america-wf branch July 30, 2022 14:34
@ImTimK ImTimK added the USA Affects USA faction label Nov 6, 2022
@xezon xezon changed the title Change USA War Factory door time from 3250 to 3300 ms Change: USA War Factory door time from 3250 to 3300 ms Mar 19, 2023
@commy2 commy2 mentioned this pull request Aug 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Design Is a matter of game design Minor Severity: Minor < Major < Critical < Blocker USA Affects USA faction
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants