Page 4 of 4 FirstFirst ... 2 3 4
Results 31 to 36 of 36

Thread: Rollover Management

  1. #31
    Junior Member
    Join Date
    Jun 2010
    Posts
    2

    Default

    I'm running into more case-sensitivity problems like the one you saw earlier invoking Snojo.

    Line 249: 'Witchess_solver.ash' should be 'witchess_solver.ash'
    Line 267: 'call farfuture mall' should be 'call FarFuture mall'
    Line 322: 'call briefcase.ash unlock' should be 'call Briefcase.ash unlock'

    Those changes seemed to work for me, YMMV.

  2. #32
    Senior Member Pazleysox's Avatar
    Join Date
    Feb 2009
    Posts
    310

    Default

    I'm running into more case-sensitivity problems like the one you saw earlier invoking Snojo.

    Line 249: 'Witchess_solver.ash' should be 'witchess_solver.ash'
    Line 267: 'call farfuture mall' should be 'call FarFuture mall'
    Line 322: 'call briefcase.ash unlock' should be 'call Briefcase.ash unlock'

    Those changes seemed to work for me, YMMV.
    Originally Posted by Zheel View Post
    Noted, and fixed.

  3. #33

    Default

    thanks for the additions and fixes. This is really great. A few things:

    1. Is it possible to move the preferences outside of the script itself and into another file which the script creates if doesn't exist, but is not downloaded from the svn? A few scripts already do that like OCD inventory. At the moment every time the script updates it resets settings to default.

    2.
    Code:
    set_property("ps_rollover", "turns");             // set this to TURNS for rollover turns or PVP for pvp fights
    it says TURNS but looking at the default it is actually turns, all lower case. does case matter on this one? I tested lower case pvp and it worked but I haven't tested all upper case.

    3. this is a really minor thing, but the in script versioning history is now missing v1.08 and higher changelog. (it used to actually have a v1.08 in r19).

    4.
    Code:
    You haven't used all of your free disco rests
    my free rests are not from disco. at the moment my free rests only come from http://kol.coldfront.net/thekolwiki/index.php/Dog_Tired

    5. when doing pvp OR adventure optimizing, it would be great if afterwards it tried to fill in the empty slots with the other option. that is, lets say I am doing pvp optimization. It equips my +3 pvp pants, +2 pvp shirt, and 3 bonus PVP accessories. This leaves the hat, back, and weapon slots empty. I actually have a +3 adventures item for the hat slot, which would have been nice to equip since said slot is open.
    If you are using mafia's built in maximizer (since it does show recent searches for stuff I never put down, I am assuming the script called it) than the way to do it is to first run the one you don't want to focus on. That is, if you set it to turns then do
    a. maximizer equip all: adventures -tie
    b. maximizer equip all: pvp -tie
    With the -tie it will (barring a bug) not unequip the prior items which gave bonus. So the end result of the above is that pvp is prioritized while adventures used for the otherwise empty slots
    Last edited by taltamir; 04-18-2018 at 04:18 PM.

  4. #34
    Senior Member Pazleysox's Avatar
    Join Date
    Feb 2009
    Posts
    310

    Default

    thanks for the additions and fixes. This is really great. A few things:

    1. Is it possible to move the preferences outside of the script itself and into another file which the script creates if doesn't exist, but is not downloaded from the svn? A few scripts already do that like OCD inventory. At the moment every time the script updates it resets settings to default.
    Originally Posted by taltamir View Post
    I fixed this in v1.09 I noticed it wouldn't change to +pvp gear when I set it, and the settings were changing back to +turns. It should be working properly now. If it's not let me know, and I'll look further into it.

    2.
    Code:
    set_property("ps_rollover", "turns");             // set this to TURNS for rollover turns or PVP for pvp fights
    it says TURNS but looking at the default it is actually turns, all lower case. does case matter on this one? I tested lower case pvp and it worked but I haven't tested all upper case.
    Originally Posted by taltamir View Post
    I fixed the wording to show all lower case, to avoid any confusion.

    3. this is a really minor thing, but the in script versioning history is now missing v1.08 and higher changelog. (it used to actually have a v1.08 in r19).
    Originally Posted by taltamir View Post
    fixed. I thought I had fixed this yesterday, not sure what happened. lol

    4.
    Code:
    You haven't used all of your free disco rests
    my free rests are not from disco. at the moment my free rests only come from http://kol.coldfront.net/thekolwiki/index.php/Dog_Tired
    Originally Posted by taltamir View Post
    Strange, because Mafia looks at your campground to find if you have disco rests.
    Code:
    [line 423]
    body = visit_url("campground.php");
    [line 435]
    if(contains_text(body,"_free.gif")) { print_html("You haven't used all of your <font color=FF0000>free disco rests</font>"); }
    5. when doing pvp OR adventure optimizing, it would be great if afterwards it tried to fill in the empty slots with the other option. that is, lets say I am doing pvp optimization. It equips my +3 pvp pants, +2 pvp shirt, and 3 bonus PVP accessories. This leaves the hat, back, and weapon slots empty. I actually have a +3 adventures item for the hat slot, which would have been nice to equip since said slot is open.
    If you are using mafia's built in maximizer (since it does show recent searches for stuff I never put down, I am assuming the script called it) than the way to do it is to first run the one you don't want to focus on. That is, if you set it to turns then do
    a. maximizer equip all: adventures -tie
    b. maximizer equip all: pvp -tie
    With the -tie it will (barring a bug) not unequip the prior items which gave bonus. So the end result of the above is that pvp is prioritized while adventures used for the otherwise empty slots
    Originally Posted by taltamir View Post
    Done. I'll upload it as v1.10

  5. #35

    Default

    I fixed this in v1.09 I noticed it wouldn't change to +pvp gear when I set it, and the settings were changing back to +turns. It should be working properly now. If it's not let me know, and I'll look further into it.
    Originally Posted by Pazleysox View Post
    Thank you, but what I meant is that when v1.11 (and v1.12 and v1.13... etc) will come out it will reset the settings because the settings are located within the script file itself. Instead of being kept in a separate file entirely.
    Some scripts like OCD inventory management keep their settings in a separate file entirely (it creates one on first run, named after the account being used to login).

    Strange, because Mafia looks at your campground to find if you have disco rests.
    Originally Posted by Pazleysox View Post
    then it is mislabeled in mafia or in baseline kol, no reason it has to be mislabeled in rollover adventures too. I am not saying to change how it looks it up, just changing the wording of rollover management script to say "free rests" instead of "free disco rests" since not all free rests come from disco.
    although honestly it doesn't really matter since its just me being OCD.

    1.10 - 4/18/18 Added rollover turns to +PVP fights if you don't have enough gear to fill all slots with +PVP. Fixed some wording.
    Originally Posted by changelog
    I am assuming the inverse is also true? where if you selected turns optimization, if you don't have enough gear to fill all slots with turns it fills the remaining slots with pvp items?

    oh also, it occurs to me that the method of using maximizer for turns then pvp with -tie will work for all slots except for accessory slots.
    For accessory slots you have 3 different slots, so for example if you have 1 pvp item and 1 turns item, using the maximizer twice could first replace accessory number 3 with turns, then replace accessory number 3 with pvp...

    For some scenarios (eg, having 2 pvp and 1 turns, or 1 pvp and 2 turns) first removing all 3 accessories (not all items, just accessories) might solve the problem (unless mafia maximizer decides to fill in those extra slots with random equipment which it sometimes does).

    but for a scenario where you have 2 pvp items and 2 turns items it would have a problem because it will have to replace one of two items with another, and it will just randomly select it instead of selecting the worse one to remove.
    To be more specific, lets say a person has:
    PVP +1 overnight accessory
    PVP +7 overnight accessory
    Turns +1 overnight accessory
    Turns +7 overnight accessory
    And it is set to maximize turns.

    First pass with pvp optimization
    Accessory slot 1: heart of volcano (giving +MP/HP per adventure)
    Accessory slot 2: PVP +1 overnight
    Accessory slot 3: PVP +7 overnight

    Second pass with turns optimzation
    Accessory slot 1: heart of volcano (giving +MP/HP per adventure)
    Accessory slot 2: turns +1 overnight
    Accessory slot 3: turns +7 overnight

    If you first remove all 3 accessories and you happen to get lucky and it decides to not fill empty slots with random junk then you get.
    First pass with pvp optimization
    Accessory slot 1: PVP +1 overnight
    Accessory slot 2: PVP +7 overnight
    Accessory slot 3: empty

    Second pass with turns optimzation will randomly be either:
    Accessory slot 1: PVP +1 overnight
    Accessory slot 2: turns +1 overnight
    Accessory slot 3: turns +7 overnight

    or:
    Accessory slot 1: turns +1 overnight
    Accessory slot 2: PVP +7 overnight
    Accessory slot 3: turns +7 overnight

    where obviously the latter is preferred. Fixing it though would be a real pain to program though... so maybe just document it and say "hey, you should make sure you have enough accessories for all 3 slots"
    Last edited by taltamir; Today at 06:39 PM.

  6. #36
    Senior Member Crowther's Avatar
    Join Date
    Nov 2006
    Posts
    1,369

    Default

    You can just let the maximizer handle all the for you: "maximize 100 adv, 1 pvp fights"

    Or whatever balance you think is right.

Posting Permissions

  • You may not post new threads
  • You may not post replies
  • You may not post attachments
  • You may not edit your posts
  •