dnqbob

> Start an AI vs AI match and join as specta The shroud/fog doesn't work for spectator but it works for unit of AI player.

> It is mainly because TS aircraft does not have vision. AttackAircraft cannot attack actor that is under shroud for AI. My bad, it should be "AttackAircraft cannot attack actor...

Pal, you can click  To turn your PR as draft PR instead of closing and reopening.


I was thinking, why not add parent actor to `Actor` field? Then we can directly use parent actor in `Actor` from all source without caring about initializing it in each...

Another reason of applying a `HasParent` trait or store the parent inside Actor's field is `SquadManager` will need those when use protection squad to defend base from spawner actor like...

SquadManager can solve this problem by using following steps: 1. Firstly, check if attacker is dead or null at present. 2. Then, if attacker is not in world, it may...

> Squads shouldn't protect vs a carryall when it is the unit it dropped off that is attacking Besides, it is also unecessary to make transport becomes parent of initialized...

I think removing "ProjectedCells" and use MPos directly is a better idea, which save a convertion to PPos.

I would suggest to use https://github.com/MustaphaTR/OpenRA/blob/rv-engine/OpenRA.Mods.AS/Traits/SharedCargo.cs https://github.com/MustaphaTR/OpenRA/blob/rv-engine/OpenRA.Mods.AS/Traits/SharedPassenger.cs https://github.com/MustaphaTR/OpenRA/blob/rv-engine/OpenRA.Mods.AS/Traits/Player/SharedCargoManager.cs and related activities for this. We also have a bot module on load unit to tunnel. https://github.com/MustaphaTR/OpenRA/blob/rv-engine/OpenRA.Mods.AS/Traits/BotModules/SharedCargoBotModule.cs