EQEmulator Forums - aggro radius

EQEmulator Forums (https://www.eqemulator.org/forums/index.php)

- Development::Development (https://www.eqemulator.org/forums/forumdisplay.php?f=590)

- - aggro radius (https://www.eqemulator.org/forums/showthread.php?t=28697)

gaeorn

06-26-2009 07:03 PM

aggro radius

I've been noticing that lull/harmony effects do not appear to have the expected effect. In particular, the monk phantom line of disciplines do not appear to reduce assist range. Looking through code, I found this:

Code:

void EntityList::AIYellForHelp(Mob* sender, Mob* attacker) {

        _ZP(EntityList_AIYellForHelp);

        if(!sender || !attacker)

                return;

        if (sender->GetPrimaryFaction() == 0 )

                return; // well, if we dont have a faction set, we're gonna be indiff to everybody

        

        LinkedListIterator<NPC*> iterator(npc_list);

        

        for(iterator.Reset(); iterator.MoreElements(); iterator.Advance()) {

                NPC* mob = iterator.GetData();

                if(!mob){

                        continue;

                }

                float r = mob->GetAssistRange();

                r = r * r;



                if(sender == NULL)

                        return;



                if (

                        mob != sender

                        && mob != attacker

//                      && !mob->IsCorpse()

//                      && mob->IsAIControlled()

                        && mob->GetPrimaryFaction() != 0

                        && mob->DistNoRoot(*sender) <= r

                        && !mob->IsEngaged()

                        )

                {

                        //if they are in range, make sure we are not green...

                        //then jump in if they are our friend

Is there a reason we are squaring the assist range of a mob before deciding to assist?

ChaosSlayerZ

06-26-2009 07:17 PM

I think it could be related how we have a SINGLE value in npc_types for Personal Agro and for Assist range, while looking at spell file you clearly see that ALL lull type spells have as much as 3 diffirent properties - Lull, Frenzy range, and Harmony, each of which supposely does something diffirent...

gaeorn

06-26-2009 07:23 PM

Quote:

Originally Posted by ChaosSlayerZ (Post 172954)

I don't understand what you are talking about. And not all spells have all 3 types of effects. The spells underlying the higher level monk phantom discs only have the harmony effect.

ChaosSlayerZ

06-26-2009 07:36 PM

Quote:

Originally Posted by gaeorn (Post 172956)

I don't understand what you are talking about. And not all spells have all 3 types of effects. The spells underlying the higher level monk phantom discs only have the harmony effect.

ok not all. but spells seem to diffirentiate between lull and frenzy radius, and in some cases harmony.

gaeorn

06-26-2009 07:41 PM

Although I still do not know why the range value in my first post was being squared, I believe I may have found out why the monk phantom disciplines don't work as desired. I'm using a version that is level 69, Phantom Cry. It only has the reaction radius effect (SE_Harmony), whereas lower level phantom discs have reaction radius, frenzy radius and lull effects.

According to this excerpt from the comments on this page:

Quote:

That's exactly how all the monk pacify disc's work. They are designed to split mobs only.
However, you notice they are unresistable.
Any monk that gets this will already be well aware of how they work and how to use it.
The other classes had the biggest adjustment with their OoWar pacify spells since they removed the frenzy radius of all the OoWar pacify spells. That means that the aggro range of the mob is not affected. You can't pacify a mob and walk up near it - it will aggro. Since monks could only ever pacify one mob at a time, they never really got used to pacifying their way through groups of mobs.
It still works great for pulling, but it does take adjusting.

the reaction radius affects assist radius, not aggro radius. But looking at the code, that effect (SE_Harmony) is adjusting aggro radius (frenzy radius, SE_ChangeFrenzyRad, is adjusting assist radius). So from what I am reading, it appears these should actually be frenzy radius = aggro range and reaction radius = assist range. This would have been difficult to determine initially as the level 65 and lower pacify type spells all have both effects. In fact, according to an allakhazam search, all spells that have the frenzy radius effect also have the reaction radius effect.

KLS	06-26-2009 08:08 PM

The reason is to do the range check without the square root.

ChaosSlayerZ

06-26-2009 08:16 PM

KLS, but does Emu code actualy sees the diffirence between frenzy range and assist range?

Also could someone clearfy what then "harmony" value does, since these allready seem to cover both effects?

gaeorn

06-26-2009 08:28 PM

Quote:

Originally Posted by KLS (Post 172960)

The reason is to do the range check without the square root.

I see. Still, I think the frenzy and reaction radius effects are reversed.

Quote:

Originally Posted by ChaosSlayerZ (Post 172962)

KLS, but does Emu code actualy sees the diffirence between frenzy range and assist range?

Also could someone clearfy what then "harmony" value does, since these allready seem to cover both effects?

In answer to your first question, yes the code does different effects for both. In answer to the second, the SE_Harmony effect currently reduces the aggro radius of the mob, but does not reduce the assist radius of the mob.

ChaosSlayerZ

06-26-2009 08:51 PM

Quote:

Originally Posted by gaeorn (Post 172963)

I see. Still, I think the frenzy and reaction radius effects are reversed.

.

well we can test this easily enough by making 2 custom spells one just with frenzy and one just with lull.

Quote:

ok so then:

Frenzy Radius - distance from which mob will agro
Lull - distance from which mob will assist
Harmony - ALSO distance from which mob will agro?

so Harmony just repeats Frenzy range effect?

gaeorn

06-26-2009 09:00 PM

Quote:

Originally Posted by ChaosSlayerZ (Post 172964)

well we can test this easily enough by making 2 custom spells one just with frenzy and one just with lull.

Quote:

Originally Posted by ChaosSlayerZ (Post 172964)

Harmony and lull are both aggro radius and frenzy is assist radius according to the emulator code. I suspect harmony and frenzy should be reversed to match live.

KLS	06-26-2009 09:20 PM

Honestly it's code that I haven't ever really touched, it was like that when I got here and other than some minor changes it's mostly the same.

If something didn't work right it wouldn't completely surprise me.

ChaosSlayerZ

06-26-2009 09:23 PM

Quote:

Originally Posted by gaeorn (Post 172965)

Great, except which one is supposed to do what on live? The problem isn't knowing which does what on the emulator. One can look at the code and see that reaction radius = harmony = aggro radius and frenzy radius = assist radius. How do you propose we make those custom spells on live? That is where we need to figure it out.

Harmony and lull are both aggro radius and frenzy is assist radius according to the emulator code. I suspect harmony and frenzy should be reversed to match live.

wait wait, you saying yourself in that post that Frenzy radius effect was removed from OoW+ lulls.
you also saying that effects on emu are reversed.

which means if we take a LULL only spell and it does NOT lull - then you have proved that emu uses these effects in reverse.

gaeorn

06-26-2009 09:35 PM

Quote:

Originally Posted by ChaosSlayerZ (Post 172968)

I don't see how this tells us what the effects actually are on live, which is all we need to know. In any case, I am already pretty sure the two effects are reversed on the emulator.

Quote:

Originally Posted by KLS (Post 172967)

I'm going to do a bit more research to back up my theory that the effects are reversed before I post patches to swap the two around.

ChaosSlayerZ

06-26-2009 09:48 PM

Quote:

Originally Posted by gaeorn (Post 172969)

I don't see how this tells us what the effects actually are on live, which is all we need to know. In any case, I am already pretty sure the two effects are reversed on the emulator.

i will trust Alakhazam on this and go with Ala shows as frenzy and lull

look

http://everquest.allakhazam.com/db/s...&action=search

you see that lower level spells all have:

Frenzy Radius
Reaction Radius

then you see Pacify added (whats that do now?)

then on lev 67 Placate spell you see 2 of effects dissapear and only Reaction Radius remains.

this spell IS in Titanium spell list.
So take this spell, run Emu and cast it. - see which of the two it reduces.
Here is your answer.

gaeorn

06-26-2009 10:03 PM

Quote:

Originally Posted by ChaosSlayerZ (Post 172970)

You simply are not understanding me. I KNOW what it does in the emulator. I've read the c++ code for it already.

What I DON'T know is:
On live, is reaction radius the aggro radius or the assist radius? The same for frenzy radius. Does it affect aggro or assist radius on live?

I don't see how doing anything at all on the emulator will answer that since the emulator could be wrong.

ChaosSlayerZ

06-27-2009 12:22 AM

ok, so ist the LIVE values you concerned with- this one is actualy MUCh easier to answer =)

you simply ASK any 67+ chanter who is playing on LIVE =)
or- I am sure you can find this information posted.

Logicly- looking at spell properties, lev 67 Placate only has
"Reaction Radius "

Here is a post writen on Ala boards for Placate:

Quote:

this spell IS NOT so u can get closer to mobs...this spell is so U can pull from a group of mobs without getting adds. .

which leads to: Reaction Radius = ASSIST RANGE

which means Frenzy = Presonal Agro

PS.
what I don't understand why they needed as much as 4 diffirent effects (reaction, frenzy, pacify and harmony) to represent only 2 things (assist and agro)

ChaosSlayerZ

06-27-2009 12:50 AM

ok corection. apperently "harmony" is an actual effect. It just something writen as name of effect in Bleh/Ala editor INSTEAD of "Reaction Radius" - effect ID 86

which leaves us with 3 effects:

Reaction Radius
Frenzy Radius
Pacify

Pacify can be found on some "lulls" but not on others. Its missing from 2 lowest rank lulls and from druids AoE Harmony, and then its present on all mid level lulls up until 67+ nerfs

my BEST guess is that "pacify" effect is what MAY cuase mob to agro on you when spell is resisted, so lulls wihout pacify effects are never produce agro even if resisted. This theory however needs confirmation from LIVE.

In Bleh/Ala editor "Pacify" effect is named as "Lull" ID #18 - which could also be missleading

gaeorn

06-27-2009 03:50 AM

Quote:

Originally Posted by ChaosSlayerZ (Post 172976)

Quote:

this spell IS NOT so u can get closer to mobs...this spell is so U can pull from a group of mobs without getting adds. .

Thanks for looking that up. That is the kind of confirmation I needed that the reaction range effect is simply assist range on live.

Quote:

Originally Posted by ChaosSlayerZ (Post 172978)

Yes, there are only 3 effect IDs in the spells as far as I have read. Only two of them, reaction radius #86 and frenzy radius #30 actually do anything in the emu code. Lull #18 is not used as an effect anywhere I could find in the code based upon searches for the define SE_Lull.

In any case, according to the information I found and the confirmation you found, the effects for reaction radius and frenzy radius are reversed in the emu code.

This patch will correct those effects to match live:

Code:

Index: zone/bonuses.cpp

===================================================================

--- zone/bonuses.cpp    (revision 715)

+++ zone/bonuses.cpp    (working copy)

@@ -647,11 +647,8 @@

                switch (spells[spell_id].effectid[i])

                {



-                       case SE_Harmony:

+                       case SE_ChangeFrenzyRad:

                        {

-                               // neotokyo: Harmony effect as buff - kinda tricky

-                               // harmony could stack with a lull spell, which has better aggro range

-                               // take the one with less range in any case

                                if

                                (

                                        newbon->AggroRange == -1 ||

@@ -663,8 +660,11 @@

                                break;

                        }



-                       case SE_ChangeFrenzyRad:

+                       case SE_Harmony:

                        {

+                               // neotokyo: Harmony effect as buff - kinda tricky

+                               // harmony could stack with a lull spell, which has better aggro range

+                               // take the one with less range in any case

                                if

                                (

                                        newbon->AssistRange == -1 ||

All times are GMT -4. The time now is 07:54 AM.