search

AttacqueSuperior / Engine

A customized fork for the modification Attacque Supérior of the open-source implementation of the Command & Conquer: Red Alert engine using .NET/Mono and OpenGL. Runs on Windows, Linux and Mac OS X.
http://attsup.swr-productions.com
GNU General Public License v3.0
16 stars 3 forks source link

From AIDeployNotifier to DeployNotifier #99

Closed dnqbob closed 3 years ago

dnqbob commented 3 years ago

Hi here,

Upon using the AIDeployNotifier/AIDeployHelper, I think this function can be normalized. There is a function missing that from Tiberium Sun, which is player can toggle the deployment of deployable vehicles by using attack order. If we make a fake weapon by using AIDeployHelper which works on player, we can solve most of the problem.

In conclusion, the new deployment control trait, maybe call it "DeployNotifier", can work for create AI micromanage at bot condition as well as for some units need deployment before attacks.

GraionDilach commented 3 years ago

Fair point. I can see this working with dropping the explicit IsBot checks for an implicit dependence on GrantConditionOnBotOwner.

GraionDilach commented 3 years ago

Fixed with https://github.com/AttacqueSuperior/Engine/commit/0322681f1c90f738b109817f6bfe99c8b664dfaa