Atlantiss / NetherwingBugtracker

Official bugtracker for the Netherwing and Karazhan (2.4.3) realms.
52 stars 36 forks source link

[Hunter] [Spell] Hunter Macros bugs #7069

Closed eri1997 closed 1 year ago

eri1997 commented 4 years ago

Description:

It is also important that the bugs that are already reported and I'll list below are as equally important to get fixed as any of them will cause the macro to reset or not do what its supposed to and become unreliable again :

If you need more information on how macro rotations shot order goes , Cheeky's hunter dps spreadsheet has a shot rotation section that is very detailed with timers to help with that.

Current behaviour: 3:2 is unreliable and will even do different shot rotations on different tests without you even changing anything because of different bugs.

Expected behaviour: 3:2 doing the correct rotation and hunter macros starting with a auto shot. No gcds lost for no reason too

Server Revision: latest

NatsumeLily commented 4 years ago

If you need more information on how macro rotations shot order goes , Cheeky's hunter dps spreadsheet has a shot rotation section that is very detailed with timers to help with that.

This is a bug that needs fixing but I would highly highly recommend not relying on cheeky's spreadsheet. It's wrong in many places. One of the places it's wrong in this context would be rotations. It's 1:1.5 rotation is incorrect, it adds in a extra steady/auto that isn't needed.

https://wow.allakhazam.com/wiki/shot_rotation_(WoW) is a better source on rotations.

eri1997 commented 4 years ago

Mostly focused on 3:2 rotation since that's the hardest to get right and the most important as well , the spreadsheet is a lot more helpful to show how the rotation works since it shows timers and the shots hitting. That link is also useful information but not as specific toward timers . I pointed out the spreadsheet to have a reference on how shots rotation should go off.

I'd recommend smb more experienced than me takes another look at 1:1.5 after the bugs on this one are fixed as I think the timers will change with the autoshot at the start added and I don't know as much for that macro since its not used by bm spec.

TheMoneyHater commented 4 years ago

Yes this needs fixing. https://streamable.com/6lg3jf https://streamable.com/am4cjh u shouldn't be getting these full gcd. Ofc the autoshot can push it forward but here it is taking much longer for it to do so then on retail. Its like its removing the next one in the que if it have been pushed forward with a autoshot but still using the gcd.

EDIT: Everything is working fine except the steadyshoot at 0:38, 0:48(https://streamable.com/bnfb27) It seems to be somthing with that the timer is in the middle of the autoshoot and don't know how to priorizing the incoming autoshoot or not.

This is the proof: https://youtu.be/cObqUH-zKwg As you can clearly see if u slow it down he never gets a full gcd between the steadyshoots.

Wolffenstein commented 4 years ago

Please retest in rev 3420.

TheMoneyHater commented 4 years ago

@Wolffenstein What exactly did u change? It seems to be working worse now. Those wrongly/fake gcds are a huge issue to the gameplay since 3:2 rotation is not playable with this being the case. Steadyshot and then arcaneshot should be priorited.

Wolffenstein commented 4 years ago

Another round of changes, what's the verdict now?

eri1997 commented 4 years ago

Updated the bugreport with the new fixes . 3:2 macro now is much better.

eri1997 commented 4 years ago

@Wolffenstein Not sure what has changed in Karazhan but macro is broken again after the first auto - steady it keeps doing 2 steady 1 auto all the time without haste effects with a 3.0 wep . Didn't do that when we were testing it on the ptr :( I'm assuming smth gcd related has changed but just a guess, hard to test without training dummies on karazhan

brewie21988 commented 4 years ago

@Wolffenstein can confirm @eri1997 comments I'm seeing the same thing with 3:2 Macro that was working on the PTR.

eri1997 commented 4 years ago

This is how the 3:2 macro works atm in Karazhan with a 3.0 wep : https://streamable.com/uv33vl This is how it worked in ptr much better version (wolf's testing) : https://streamable.com/xay3bu Idk what change broke it sometime between ptr and karazhan release but the macro is shit and useless atm with a 3.0 speed wep. Tried a few slightly faster weps and even though they were also broken they managed a slightly closer steady/auto % to the macro version which is completely the opposite of what should happen.

raethkcj commented 4 years ago

I lined up the two videos to start the rotation at the same frame: https://drive.google.com/file/d/18c4KaeWohZYHoF_CJTQ1gNIM_KnRQEQX/view?usp=sharing

After watching it through frame by frame, it looks like the current bug on Karazhan is that the GCD for a second Steady Shot can begin while the first Steady Shot is still casting. In the top picture below, note that there are 0.2 seconds left on the Steady Shot cast, but the GCD for the next cast has already begun. (This occurs ~1 second into the linked video) image

Because of these early GCDs, the hunter is able to queue Steady Shot a couple hundred milliseconds before they should be able to. This means that a 4th Steady Shot is queued during the 3:2 rotation when normally it might not be able to.

raethkcj commented 4 years ago

After talking with another hunter and doing some tests, my theory is it's a side effect of the way shot queuing during auto windup is implemented.

In order to allow hunters to cast Steady Shot during Auto windup, you need a 500ms window where early GCD is allowed. This window exists for all casts on all classes instead of just Auto Shot.

You can reproduce this with any cast time ability followed by instant cast; for example, Starfire into Moonfire. If you press Moonfire during the last 500ms of the cast, the GCD begins immediately even though Moonfire isn't cast until 300ms later.

I don't know whether this early-GCD behavior is blizzlike.

Wolffenstein commented 1 year ago

Fixed in rev 3869.