Mage Focus Makes Tiny Pets Fix

[demonstar55]

Pet focuses suck. They're horribly implemented on live. There really shouldn't be any code deciding on their stats at all, they really define a new NPC on live, which is what we should be doing :P

Proof: http://mysticallightfighters.com/ima...er/sodpets.jpg that's based on data given to a dev.

[Akkadius]

Originally Posted by demonstar55

Pet focuses suck. They're horribly implemented on live. There really shouldn't be any code deciding on their stats at all, they really define a new NPC on live, which is what we should be doing :P

Proof: http://mysticallightfighters.com/ima...er/sodpets.jpg that's based on data given to a dev.

That's a pretty static way of implementing something though. Pros and cons to going that route.

[demonstar55]

Originally Posted by Akkadius

That's a pretty static way of implementing something though. Pros and cons to going that route.

If there isn't a matching NPC for that focus it should use the default pet. That's how it works on live. (BST pets aren't focused until their level 64 pet despite the fact there are pet focuses that should be applied to them, well there is a pet power 15 and 25 that works on pre-64, but there are better pet focus that should work on pre-64 but don't)

The auto-scaling in code should not exist in a live-like environment.

[Akkadius]

Originally Posted by demonstar55

If there isn't a matching NPC for that focus it should use the default pet. That's how it works on live. (BST pets aren't focused until their level 64 pet despite the fact there are pet focuses that should be applied to them, well there is a pet power 15 and 25 that works on pre-64, but there are better pet focus that should work on pre-64 but don't)

The auto-scaling in code should not exist in a live-like environment.

For sure I wasn't arguing against how live implements it, if that is how they do it obviously we would want to do something that matches it. But a ground-up approach, if I were working for SOE/DB and implementing something like that, doing a static entry for each pet focus is silly.

[demonstar55]

Originally Posted by Akkadius

For sure I wasn't arguing against how live implements it, if that is how they do it obviously we would want to do something that matches it. But a ground-up approach, if I were working for SOE/DB and implementing something like that, doing a static entry for each pet focus is silly.

Yes it is, which is probably why that BST pet oddity exists XD


But yeah, I got all the low level mag focus items now ... just need to parse out their everything -.-

[tigurius]

Looks like the present code is mostly emulating the really old trilogy rules where your pet gets a rank bump on focus. Thats fine for the lower level stuff which is where i'm playing now, but after reviewing that chart, it really needs a different approach like you said.

I haven't played live since Luclin so I have a hell of a lot of catching up all the rules since. I'd like to take a stab at some pet code later.