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

sc2: Added liberator guerilla missiles to replace 360 degree sensors #144

Merged
merged 3 commits into from
Jun 17, 2024

Conversation

MatthewMarinets
Copy link
Contributor

@MatthewMarinets MatthewMarinets commented Jun 16, 2024

Replaced liberator 360 degree sensors upgrade with Guerilla Missiles upgrade:

Liberators in Fighter Mode apply a debuff to enemy units the hit, causing them to move and attack slower for 8 seconds.

I'm fine to play with the numbers a little bit. Currently it's 30% reduced move speed and 15% reduced attack speed (see AP_LiberatorGuerillaMissilesDebuff in BehaviorData.xml). The move speed is much stronger, but takes kiting to really utilize and that's hard to do with the liberators' slow damage point and short range; it also doesn't synergize with sieged liberators as much. The attack speed debuff is much more subtle, but provides some reason to mix in a few liberators with an AA mix to reduce incoming damage.

The visual effect could also be adjusted on request, but I'm going to leave it as-is until someone provides feedback / says how they'd like it changed. Currently it uses a scaled-down version of Fenix's Whirlwind effect animation.

image

image

video link:
https://discord.com/channels/731205301247803413/980554570075873300/1251799672234709032

@MatthewMarinets
Copy link
Contributor Author

MatthewMarinets commented Jun 16, 2024

Got it to grey out in defender mode:
image

@MatthewMarinets
Copy link
Contributor Author

image

@Ziktofel Ziktofel merged commit 7dfe0bc into Ziktofel:sc2-next Jun 17, 2024
1 check passed
@MatthewMarinets MatthewMarinets deleted the mm/liberator_aa_upgrade branch July 15, 2024 20:48
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.

2 participants