Ability Rework Progress

[Telen]

Any word on the aoe radius bug? Sorcs and BW are abusing the hell out of it.
This is a big issue.

[Azarael]

I looked into it last night but I didn't see anything obviously wrong. The radii are read from the tooltip and the internal functions for distance work in feet.

[Telen]

I looked into it last night but I didn't see anything obviously wrong. The radii are read from the tooltip and the internal functions for distance work in feet.

Test them in game. They are wrong. Its more than 10 feet on some now i've tested some more.

EDIT: No it is 10 feet on all now i've done more testing.

Seems likely the client already has 10 feet as a base for aoe. This is probably due to the range patch for when melee couldn't hit moving targets.

[noisestorm]

Maybe the formula you use has a diameter in it instead of a radius one, or you missed a (.../4) somewhere in between? sometimes you dont find the tree in the forest (lulz)

[Azarael]

The tooltips use radius, in the form "Within x ft of the target". I've never seen a tooltip that uses a diameter. I'm looking into it some more now.

[noisestorm]

Yeah but you somewhere have to use some sort of circle-calculation around your actual target? or how do you handle that?

[Azarael]

It's done via spawning a temporary object at the player's location and calculating the distance of each nearby target from that object's position.

The distance calculation is FUBAR though, there's a strange linear interpolation in here whose purpose is unknown to me, but the result is that the distance calculation comes out totally wrong.

[noisestorm]

Cant you just put a circle around your target with fixed parameters (like the 10 feet or w/e) and set up a query for targets in its parameters and apply the Class of your LoS Check to them?
And by 'just' i mean, that i have no clue about your actual code, but hope i can give you some perspectives on how to solve the problem

[herrmarek]

SM Blurring shock hits me, not opponent

[Azarael]

I didn't write the vector magnitude calculation functions, but this lerp must have been added for a reason so I'm not gonna change it without knowing why.

Issue with damaging procs is already noted in the General section.