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 more airdrop options #21107

Merged
merged 4 commits into from Oct 13, 2023
Merged

Add more airdrop options #21107

merged 4 commits into from Oct 13, 2023

Conversation

dnqbob
Copy link
Contributor

@dnqbob dnqbob commented Oct 11, 2023

new-airdrop

@PunkPun
Copy link
Member

PunkPun commented Oct 11, 2023

⚠️ this is violating physics and CnC lore in serious ways

@dnqbob
Copy link
Contributor Author

dnqbob commented Oct 11, 2023

How?

@dnqbob
Copy link
Contributor Author

dnqbob commented Oct 11, 2023

Your cargo plane land on the middle of the strip and actor spawn at start of the strip, I don't think that is what the lore suggest.

@PunkPun
Copy link
Member

PunkPun commented Oct 11, 2023

  1. it looks absurd when Inertia of the aircraft is violated in such an extreme way
  2. in Tiberian Dawn lore it's said the aircraft never stops. This is what I quickly found from the wiki

The large cargo plane does not land at the field. Instead it slowly flies in at an extremely low altitude and releases the vehicle from the aircrafts cargo bay as it passes over the gently inclined runway.

In openRA we have sped up the plane for balance, although it isn't slow in vanilla as well

@dnqbob
Copy link
Contributor Author

dnqbob commented Oct 11, 2023

it looks absurd when Inertia of the aircraft

It is the same for RA actually if you must say that, aircraft should not be stop all in a sudden.

And if it is the lore suggested, please just takes the third commit as showcase. Those option is still worthy on other airdrop case.

At least airdrop offset needs to be fixed.

@dnqbob dnqbob changed the title Nod airdrop visual fix Add more airdrop options Oct 11, 2023
@dnqbob
Copy link
Contributor Author

dnqbob commented Oct 11, 2023

Remove the wait, only fix the landing offset

@dnqbob
Copy link
Contributor Author

dnqbob commented Oct 11, 2023

d2k-airdrop

Also fix d2k airdrop. I don't see any wiki on D2k starport and I don't think carryall can be release while transport ship is still moving.
@PunkPun

Copy link
Member

@PunkPun PunkPun left a comment

Choose a reason for hiding this comment

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

LGTM, confirmed that in vanilla d2k they were at the ground for a prolonged period of time

@PunkPun PunkPun merged commit f5450cd into OpenRA:bleed Oct 13, 2023
3 checks passed
@PunkPun
Copy link
Member

PunkPun commented Oct 13, 2023

Changelog

@dnqbob dnqbob deleted the airdrop-land branch October 14, 2023 23:58
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.

None yet

2 participants