ClassicWoWCommunity / cata-classic-bugs

This is a place for Cataclysm & Wrath of the Lich King Classic bug reports and research
64 stars 6 forks source link

[BUG] Pet delay #1464

Open kollariss opened 2 months ago

kollariss commented 2 months ago

**### 1. Current Behavior

1.1. Description**

Pets are not using their abilities on CD, they introduce artificial lag due to pet AI i presume. Currently it is somewhere between 150ms-1s. Therefore pet classes (and warlocks in particular) lose quite substantive dps.

#### 1.2. How to Reproduce

       1. Be a pet class
       2. Use a pet
       3. Watch how pet is not using abilities that are ready to be used and on autocast

#### 1.3 Source Material

  1. Imp https://classic.warcraftlogs.com/reports/Nmn2HxdZp19QMLtP#fight=last&type=damage-done&source=9&start=63213&end=371432

Firebolt casts: 126 Time: 308s Cast per second: 2.444s

This is without using the macro to force cast the spell.

The minmax version: https://classic.warcraftlogs.com/reports/gH96GARwDXjcYVKZ#fight=8&type=damage-done&source=32&ability=-688 Time: 230s Casts: 140 Cast per second: 1.64s Expected: 1.5s

  1. Felhunter https://classic.warcraftlogs.com/reports/H3r9WVQ4F6qvT1a8#fight=last&type=casts&source=2&view=timeline Time: 2278 Casts: 317 Cast per second: 7.18s Expected: 6s

With macro https://classic.warcraftlogs.com/reports/368w7XLKTkrhCAcN#fight=34&type=damage-done&source=23&ability=-691 Time: 206s Casts: 33 Cast per second: 6.24s Expected: 6s

Doomguards: https://classic.warcraftlogs.com/reports/368w7XLKTkrhCAcN#fight=34&type=damage-done&source=80 no macro helps here as it is a guardian Time: 65s Cast: 2.621s Actual casts: 20; 3.25s/doombolt Expected casts: 24,8 (ie 24)

**### 2. Expected Behavior

2.1. Description**

There should be no delay (the same way the pets work in retail now). I am aware that this bug has been in the game for a long while and has been reported in classic at least since Naxxramas in wrath. I am also aware that this has been a bug in retail cataclysm. The classic team has however made already many changes that targeted similar bugs that were not fixed during the time when the content was current (for instance ignite was fixed even though in cata munching was still supposed to exist thus making the class already the best dps in the game in phase 1). Therefore the argument that it is behaving the way it is supposed loses any substance. Pets are a major part of the game and this should be considered a core bug. The classes impacted the most are hunter and warlock and it can be clearly seen that their popularity during cata went significantly down (warlock is now the least played class in pve). The pets are now more integral part of the dps so the effects of this are felt more profoundly. I am fully aware that this is not an easy fix and i do not know if cata classic team has the resources to fix this. If it does not however, there are band aid fixes that can be done to compensate the classes that lose dmg to this and make the game more balanced. Because as it stands now, vast majority of players (that do not pet macros) lose significant dps due to this and it has the potential to significantly shake up the raid comps especially in tiers to come. I believe its grossly disrespectful to the classic community to just let this be for year(s) because it is hard to fix/backport whilst similar issues have been fixed in the past. Removing the need for the entire class communities to use the /use [@pettarget,exists]Shadow Bite/Firebolt/Bite etc... to mitigate the impact of this bug under every ability would be major QoL change.

Frankly i have low expectations in regards to what will be the effect of this report and i understand that people who will read this or work on this are not to be blamed for this issue being sidestepped for this long. However i do hope that after classic cataclysm being on the side track probably since its inception, now is the time for some changes to be made to make the game better and more enjoyable for the community.

Artemishowl commented 2 months ago

Valid issue, ticket created