I know I did the same thing to a spells function also related to casting speed so just fixed that.

Hmm, still erroring on linux compile. Same error.

What the hell is this? LOL I think my own Brain just exploded.

LOL it returns the percentage of the amount casting speed will mod somebody (capped at 50%)

John Adams wrote:Hmm, still erroring on linux compile. Same error.

Sorry was in a hurry to commit the fix forgot to add the ( ) around the equation

thefoof wrote:LOL it returns the percentage of the amount casting speed will mod somebody (capped at 50%)

If you want the maximum to be 50% (0.5) you should use min() not max(), max will take the higher value of the two numbers so if you have 0.5 and 0.75 it will return 0.75 giving you a 75% boost

Proper fix is on Dev SVN

Jabantiz wrote:

thefoof wrote:LOL it returns the percentage of the amount casting speed will mod somebody (capped at 50%)

If you want the maximum to be 50% (0.5) you should use min() not max(), max will take the higher value of the two numbers so if you have 0.5 and 0.75 it will return 0.75 giving you a 75% boost

Reading the code again I think I miss read it, the return value will be multiplied against the base cast time, if that is the case you have it correct sorry for the confusion

Not to derail the AI thread but it would be nice if complex stuff like that were broken down into local vars that made more sense. Me looking at it, I have no idea what I'm looking at - though I am pretty good with math

And... // comments! Jab, what are you teaching your apprentice?

And finally (again) I must ask, Rules! When at all possible, consider using Rules if you can imagine yourself as an Admin wanting to tweak an offset/modifier without having to recompile the code.

Other than that, the mind-numbingly complex return looks good!

Looks like the other one needs to be fixed too.

About to do a commit of my instance code, I can add the fix in there unless some one beats me to it

EDIT: Fix is on Dev SVN