Hi Sigil,

I was away for a while. Thank you for your help.

I figure out the problem is that, after switching from the current attackBehaviour (take cover and shoot) to a new attackBehaviour (which doesnt have the “FindCoverPoint” action, the Cover Occupant will be set to null. I think this is due to in the “FindCoverPoint”, when Stop, it Vacate the spot. Hence, this spot become available for other AI to take.

Can I comment out this Vacate(ai); in the Stop function? would it affect anything?

Thank you.

Hi Sigil,

I comment out the “Vacate(ai);” in the Stop function. And modify the FindBestCoverPoint as below. What I’m trying to do is to have the AI move to a new cover point (if there is one). The code below correctly vacate and occupy the AIObjective. However, the AI never moves to a new cover. What would be the problem with my code? Thank you for your help.

private void FindBestCoverPoint(AI ai)
    {
        if (!CoverVariable.IsValid || !CoverVariable.IsVariable)
            return;
        //We don't actually have to have an enemy.  We can choose cover points that are simply near ourselves.
        Vector3 tEnemyPosition = ai.Kinematic.Position;
        if (EnemyVariable.IsValid && EnemyVariable.IsVariable)
        {
            //using a MoveLookTarget lets us automatically convert between gameObject, Vector3, etc., when getting
            //a position from AI memory
            MoveLookTarget tTarget = MoveLookTarget.GetTargetFromVariable(ai.WorkingMemory, EnemyVariable.VariableName, ai.Motor.CloseEnoughDistance);
            if (tTarget.IsValid)
                tEnemyPosition = tTarget.Position;
        }
        //Cover points should be marked with the "cover" tactical aspect
        _coverSensor.Sense("cover", RAINSensor.MatchType.ALL);
	//**********Modified**********
	AIObjective previousCoverSpot = null;
	Vector3 previousCoverAspect = ai.Kinematic.Position;;
	//**********Modified**********
        //Default cover point is our own position
        float bestCost = float.MaxValue;
        Vector3 tCoverPosition = ai.Kinematic.Position;
        for (int i = 0; i < _coverSensor.Matches.Count; i++)
        {
            TacticalAspect tCandidateAspect = _coverSensor.Matches[i] as TacticalAspect;
            if (tCandidateAspect == null)
                continue;
            //Cover points are AIObjectives, which are objects that allow AI to "occupy" them
            AIObjective tObjective = tCandidateAspect.Entity.Form.GetComponent<AIObjective>();
            //Skip occupied cover points
	   //**********Modified**********
	   if((tObjective != null) && (tObjective.IsOccupied) && (tObjective.Occupant == ai.Body)){
		previousCoverSpot = tObjective;
		previousCoverAspect = tCandidateAspect.Position;
		continue;
	   }
	   if ((tObjective == null) || (tObjective.IsOccupied && tObjective.Occupant != ai.Body))
                continue;
	//**********Modified**********
            //Our cost function gives 50% more weight to points near the enemy
            //But then also adds in the AI distance to the point
            float tCost = 1.5f * Vector3.Distance(tEnemyPosition, tCandidateAspect.Position) + Vector3.Distance(ai.Kinematic.Position, tCandidateAspect.Position);
            if (tCost < bestCost)
            {
                _currentCoverPoint = tObjective;
                tCoverPosition = tCandidateAspect.Position;
                bestCost = tCost;
            }
        }
        //If we found a cover point, then occupy it
	//**********Modified**********
        if (_currentCoverPoint != null){
		if(previousCoverSpot != null)
			previousCoverSpot.Vacate(ai.Body);
            _currentCoverPoint.Occupy(ai.Body);
	}else if(previousCoverSpot != null){
	    tCoverPosition = previousCoverAspect;
	    previousCoverSpot.Occupy(ai.Body);
	}
	//**********Modified**********
        //Set the cover position in AI memory
        ai.WorkingMemory.SetItem<Vector3>(CoverVariable.VariableName, tCoverPosition);
    }

• This reply was modified 1 year, 2 months ago by  James.AVN.