celguar / mangosbot-bots

Bot AI Core from ike3 for cmangos/mangos classic/tbc/wotlk
https://github.com/cmangos/playerbots
84 stars 40 forks source link

Mage improvements #241

Closed kadeshar closed 1 year ago

kadeshar commented 1 year ago

obraz obraz

celguar commented 1 year ago

"and certain PvE situations". I don't see why we should limit bots like that

celguar commented 1 year ago

If you exclude blast wave that has alternative - frost nova, you need to make sure that frost nova is still there for fire mages to use when enemy is close. Still I don't agree we should exclude blast wave. If you took pvp specced fire mage to dungeon and don't want it to cast blast wave, use "ss [Blast Wave]" to disable it

celguar commented 1 year ago

Any thoughts?

mostlikely4r commented 1 year ago

Any thoughts?

Hey Celg, I asked for this since it was mentioned mage bots used blast wave the moment any mob gets close. The same happens with aoe fears which trigger from any player or mob gets within range.

I assumed this was incorrect for pve situations but ok for pvp. For pve I assumed an aoe trigger (multiple mobs near) would be better. Alas I don't know mages well enough to judge it either way.

@celguar is this change best expanded more or reverted?

kadeshar commented 1 year ago
  1. Pve fire mages dont even learn blast wave talent
  2. Real players feedback about blast wave in pve. Tbh i wanted to ask about bots and glyphs. They use it? obraz obraz
  3. Finally i think better use command "do Blast Wave" in "certain PvE situation" than put that in default pve rotation
  4. Frost nova is still in generic mage strategy
celguar commented 1 year ago

k it makes some sense