Skip to content

[Bots] Prevent non-taunters from potentially fleeing from mob#4859

Merged
fryguy503 merged 1 commit intoEQEmu:masterfrom
nytmyr:bot-target-reflection
Apr 28, 2025
Merged

[Bots] Prevent non-taunters from potentially fleeing from mob#4859
fryguy503 merged 1 commit intoEQEmu:masterfrom
nytmyr:bot-target-reflection

Conversation

@nytmyr
Copy link
Copy Markdown
Contributor

@nytmyr nytmyr commented Apr 11, 2025

Description

  • Casters could endlessly flee a mob if they had TargetReflection and were too close.
  • Prevents all bots that aren't taunting from adjusting if they're too close and have target reflection. This is safer than limiting it to just casters and ranged bots to prevent situations where melee bots might not be able to outrun the mob to get to their melee range and continually flee.
  • WE'LL GET THIS RIGHT EVENTUALLY /sigh, missed it on last PR.

Type of change

  • Bug fix (non-breaking change which fixes an issue)

Testing

  • Need it be said? Casters and ranged won't flee when mob approaches them with TargetReflection

Clients tested: RoF2

Checklist

  • I have tested my changes
  • I have performed a self-review of my code. Ensuring variables, functions and methods are named in a human-readable way, comments are added only where naming of variables, functions and methods can't give enough context.
  • I own the changes of my code and take responsibility for the potential issues that occur

…lection

- Casters could endlessly flee a mob if they had target reflection and were too close.
- Prevents all bots that aren't taunting from adjusting if they're too close and have target reflection. This is safer than limiting it to just casters and ranged bots to prevent situations where melee bots might not be able to outrun the mob to get to their melee range and continually flee.
- WE'LL GET THIS RIGHT EVENTUALLY /sigh
@fryguy503 fryguy503 merged commit 7e7fb7b into EQEmu:master Apr 28, 2025
1 check passed
@Akkadius Akkadius mentioned this pull request May 15, 2025
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