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

Fix: Optimize drop location and spacing of USA Paradrops #2056

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

xezon
Copy link
Collaborator

@xezon xezon commented Jul 1, 2023

This change optimizes the drop location and spacing of the USA Paradrops. The soldiers now fall into a line and no longer charge towards to same position. The spacing is tightened consistently and they fall as close to the drop center as possible.

Optimized for fix TheAssemblyArmada/Thyme#752.

Original

Paradrop 1 spacing is not tight. Soldier fall too far behind.

generals.2023-07-01.17-19-12-33.mp4

Paradrop 3 is poorly executed. Depending on drop angle, planes will return a second time.

generals.2023-07-01.17-19-35-26.mp4

Patched (+Thyme 752)

Paradrop 1 is nice.

game.2023-07-01.17-07-31-31.mp4

Paradrop 3 is nice.

game.2023-07-01.17-08-04-79.mp4

@xezon xezon added Bug Something is not working right Minor Severity: Minor < Major < Critical < Blocker USA Affects USA faction labels Jul 1, 2023
@commy2
Copy link
Collaborator

commy2 commented Jul 1, 2023

I notice that each plane drops 6 rangers, then 4 mds next to each other. Maybe we can mix it up a bit?

Instead of

Payload = AmericaInfantryRanger 6
Payload = AmericaInfantryMissileDefender 4

have

Payload = AmericaInfantryRanger 3
Payload = AmericaInfantryMissileDefender 2
Payload = AmericaInfantryRanger 3
Payload = AmericaInfantryMissileDefender 2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something is not working right Minor Severity: Minor < Major < Critical < Blocker USA Affects USA faction
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants