Rev. 513 Blacklisting Shrines/Pylons Time Too Short

[mmmzq]

First of all, thanks for all your great works.

I'm running with Nemesis Bracer. After update to rev. 513, bot still tries to check a tired Shrines/Pylon sometimes.

Then I found in the log:

14:41:05.801 DEBUG Logger [Trinity 2.14.513] Blacklisting x1_LR_Shrine_Damage (330695) for 15 seconds after 5 interactions

15 sec is a little pressing for my yellow elite combat (from Nemesis Bracer). My sader normally needs at least 2 rotations of CE to finish it. Should be 60 sec or more IMHO.
But compare this to previous revs, rev. 513 definetly made a great improvement on shrine/pylon checking.

 

[chongeryan]

check my post here: https://www.thebuddyforum.com/demon...ty/241366-double-checking-shrines-pylons.html

blacklisting method is not the problem. in trinity they use wrong id of Shrine/Pylon, like:

Blacklist15Seconds.Add(CurrentTarget.RActorGuid);

in one game, RActorGuid will change for no reason. when bot use RActorGuid to blacklist a pylon, after a while, this pylon have new RActorGuid! Clearly jubisman and xzjv haven't seen this.

 

[chongeryan]

and also Blacklist15Seconds is not good enough. Just like the method name says, this "blacklist" thing will refresh within 15 secs. In my opnion, jubisman can use a new blacklist method which only reset at townrun or newgame.