Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add target_type for target furthest enemy in spell casting AI #2352

Open
wants to merge 8 commits into
base: development
Choose a base branch
from

Conversation

balakethelock
Copy link
Contributor

@balakethelock balakethelock commented Dec 15, 2023

🍰 Pullrequest

  • Added new target_type to target the furthest enemy within spell range for Creature_Spells table ai
  • Implemented that new targetting type for the naxxramas mobs "Necro Stalker" and "Venom Stalker"

Proof

  • Ability timers of the mobs are based on sniffs
    image

  • Replay to show targetting behavior

2023-12-15.18-07-06.mp4

Issues

  • None

How2Test

  • Log into game with a number of characters
  • .npc summon 16453 (in the open world works)
  • Tank it in melee with a character, two characters stand in range and move around to see it consistently targets the furthest one within LOS and spell range (in the case of poison charge, 40 yards)

Todo / Checklist

  • Find out if the charge minimal range is 5 yard or 8 yards or something else. Needs data on both the naxxramas spiders and lava surgers from molten core
  • If found that they stop charging at 8 yards instead of 5 yards, need to implement a targetParam for changing minimum range to search for targets. EDIT: Solved with Condition

Changed spell timers (based on sniffs) and targeting (based on warcraft logs live replay)
mob name typo
@balakethelock balakethelock changed the title Target furthest script Add target_type for target furthest enemy in spell casting AI Dec 15, 2023
@0blu 0blu added SQL A issue / PR which references SQL code CPP A issue / PR which references CPP code labels Oct 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CPP A issue / PR which references CPP code SQL A issue / PR which references SQL code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants