Ziktofel / Archipelago-SC2-data

Map and mod files for Archipelago-SC2
MIT License
1 stars 13 forks source link

Adding Infested Marines and Infested Bunkers (and a little more) #209

Closed EnvyDragon closed 1 month ago

EnvyDragon commented 1 month ago

Generator PR here: https://github.com/Ziktofel/Archipelago/pull/248

New items to unlock:

Supporting units (ie. not items to be found in the world, but either unlocked or produced by the above):

Triggers:

Ziktofel commented 1 month ago

Snímek obrazovky pořízený 2024-07-24 21-29-50 Snímek obrazovky pořízený 2024-07-24 21-29-32 Bug: Infested SCV gets the attack speed upgrade from normal SCV (Hostile Environment Adaptation) but not its HP or displayed icon

Ziktofel commented 1 month ago

Normal SCVs autocast repairs. Infested don't Snímek obrazovky pořízený 2024-07-24 21-32-11

Ziktofel commented 1 month ago

Script Error: Snímek obrazovky pořízený 2024-07-24 21-35-14 Triggers when Infested Bunker completes Spawn Infested Trooper and rally point is not set

Ziktofel commented 1 month ago

Snímek obrazovky pořízený 2024-07-24 21-46-02 Missing Actor AP_SIInfestedMarine_Land

Ziktofel commented 1 month ago

Snímek obrazovky pořízený 2024-07-24 21-47-51 Wild Mutation (Shield name/color needs to be copied from other Zerg units like lings)

Ziktofel commented 1 month ago

Snímek obrazovky pořízený 2024-07-24 22-02-29 Snímek obrazovky pořízený 2024-07-24 22-02-18 Add a requirement for Rooted for display (the same way as Tank use). Also do this for Regenerative Plating icon

EnvyDragon commented 1 month ago

Script Error: Snímek obrazovky pořízený 2024-07-24 21-35-14 Triggers when Infested Bunker completes Spawn Infested Trooper and rally point is not set

This error is harmless and is accounted for in the triggers. Unfortunately the sc2 editor doesn't appear to have a try/catch method, so if I use the only function in existence that can get the target point of a rally ability, it throws an error if there's no rally point. I'll take a second look, but the script error doesn't show up in regular gameplay and the trigger is designed to accommodate a missing point even if the editor finds it upsetting.

EnvyDragon commented 1 month ago

Add a requirement for Rooted for display (the same way as Tank use). Also do this for Regenerative Plating icon

What you're seeing is the vanilla behavior. The icon is present-but-disabled when uprooted so it's clearer to players that it won't spawn if it's mobile. I can put the same visual effect on Regenerative Plating though, if it helps.

Ziktofel commented 1 month ago

Script Error: Snímek obrazovky pořízený 2024-07-24 21-35-14 Triggers when Infested Bunker completes Spawn Infested Trooper and rally point is not set

This error is harmless and is accounted for in the triggers. Unfortunately the sc2 editor doesn't appear to have a try/catch method, so if I use the only function in existence that can get the target point of a rally ability, it throws an error if there's no rally point. I'll take a second look, but the script error doesn't show up in regular gameplay and the trigger is designed to accommodate a missing point even if the editor finds it upsetting.

Add a check if the point exists and if not, use Skip Remaining Actions

Ziktofel commented 1 month ago

Add a requirement for Rooted for display (the same way as Tank use). Also do this for Regenerative Plating icon

What you're seeing is the vanilla behavior. The icon is present-but-disabled when uprooted so it's clearer to players that it won't spawn if it's mobile. I can put the same visual effect on Regenerative Plating though, if it helps.

Use the same visual effect + like tanks do, use the requirement that tells the player that the infested bunker needs to be rooted. It's used for Tanks/Liberators/Vikings where a trait works only in one mode

EnvyDragon commented 1 month ago

Add a requirement for Rooted for display (the same way as Tank use). Also do this for Regenerative Plating icon

What you're seeing is the vanilla behavior. The icon is present-but-disabled when uprooted so it's clearer to players that it won't spawn if it's mobile. I can put the same visual effect on Regenerative Plating though, if it helps.

Use the same visual effect + like tanks do, use the requirement that tells the player that the infested bunker needs to be rooted. It's used for Tanks/Liberators/Vikings where a trait works only in one mode

Not sure what you mean or what you're suggesting, so I just did this: image

EnvyDragon commented 1 month ago

Script Error: Snímek obrazovky pořízený 2024-07-24 21-35-14 Triggers when Infested Bunker completes Spawn Infested Trooper and rally point is not set

This error is harmless and is accounted for in the triggers. Unfortunately the sc2 editor doesn't appear to have a try/catch method, so if I use the only function in existence that can get the target point of a rally ability, it throws an error if there's no rally point. I'll take a second look, but the script error doesn't show up in regular gameplay and the trigger is designed to accommodate a missing point even if the editor finds it upsetting.

Add a check if the point exists and if not, use Skip Remaining Actions

Skip Remaining Actions isn't an option because it needs to do a bunch of other actions, like spawning the actual unit. I found a workaround though, the error warning is now gone.

Ziktofel commented 1 month ago

Add a requirement for Rooted for display (the same way as Tank use). Also do this for Regenerative Plating icon

What you're seeing is the vanilla behavior. The icon is present-but-disabled when uprooted so it's clearer to players that it won't spawn if it's mobile. I can put the same visual effect on Regenerative Plating though, if it helps.

Use the same visual effect + like tanks do, use the requirement that tells the player that the infested bunker needs to be rooted. It's used for Tanks/Liberators/Vikings where a trait works only in one mode

Not sure what you mean or what you're suggesting, so I just did this: image

Snímek obrazovky pořízený 2024-07-25 18-52-35 Add this (it's a requirement that leads to an upgrade that's never researched). All transformers in AP use this

Ziktofel commented 1 month ago

Snímek obrazovky pořízený 2024-07-25 18-57-00 Snímek obrazovky pořízený 2024-07-25 18-56-02 Infested SCVs are affected by Terran Advanced Construction upgrade but the player isn't notified about that

EnvyDragon commented 1 month ago

Snímek obrazovky pořízený 2024-07-25 18-52-35 Add this (it's a requirement that leads to an upgrade that's never researched). All transformers in AP use this

image Added! Will add the SCV upgrades then commit.

Ziktofel commented 1 month ago

Also if your Infested Barracks is uprooted, you can no longer build infested Bunkers (normally lifted off buildings do count for tech requirements) Snímek obrazovky pořízený 2024-07-25 19-17-01

EnvyDragon commented 1 month ago

image Applied SCV upgrades to infested SCVs

EnvyDragon commented 1 month ago

Uprooted Barracks now uses the same tech alias as the rooted version (pictured is the only infested barracks on the map) image