[Windows 7 x64] Lol "Deposit Items" bug/flaw

[iudicium105]

Bug ReportOperating System:
Windows 7 x64 Short Description:
Lol "Deposit Items" bug/flaw Behaviors: Client Freeze/Crash
Feature error
Loss of or impaired functionality

Indepth Description:
This is the error I get when using the built-in deposit items function. My loot list is pretty long, needs to be that long though.

"one-liner executed by xenobot walker"
"not enough memory"

I will use old deposit function until this is fixed.

[iudicium105]

Went back to the old deposit items function i was using and that doesn't work now... it opens the dp then closes it and goes to next waypoint. I have backpacks minimized. I saw a post on another thread and tried using that which didn't work either.

[iudicium105]

Here's the loot list I am using.


<panel name="Looter">
<control name="LootList" first="1" skinner="0" unlisted="0">
<item ID="7407" action="4" deposit="2"/>
<item ID="813" action="4" deposit="2"/>
<item ID="7387" action="4" deposit="2"/>
<item ID="7418" action="4" deposit="2"/>
<item ID="7451" action="4" deposit="2"/>
<item ID="20201" action="3" deposit="1"/>
<item ID="16120" action="3" deposit="1"/>
<item ID="16123" action="3" deposit="1"/>
<item ID="238" action="3" deposit="1"/>
<item ID="16126" action="3" deposit="1"/>
<item ID="20199" action="3" deposit="1"/>
<item ID="5895" action="3" deposit="1"/>
<item ID="239" action="3" deposit="1"/>
<item ID="20198" action="3" deposit="1"/>
<item ID="9058" action="3" deposit="1"/>
<item ID="7368" action="3" deposit="1"/>
<item ID="20200" action="3" deposit="1"/>
<item ID="5925" action="3" deposit="1"/>
<item ID="5880" action="3" deposit="1"/>
<item ID="3049" action="5" deposit="3"/>
<item ID="3031" action="6" deposit="0"/>
<item ID="3035" action="0" deposit="0"/>
<item ID="7404" action="4" deposit="2"/>
<item ID="10389" action="4" deposit="2"/>
<item ID="20062" action="3" deposit="1"/>
<item ID="7454" action="4" deposit="2"/>
<item ID="7413" action="4" deposit="2"/>
<item ID="812" action="4" deposit="2"/>
<item ID="3079" action="4" deposit="2"/>
<item ID="3344" action="4" deposit="2"/>
<item ID="8092" action="4" deposit="2"/>
<item ID="3419" action="4" deposit="2"/>
<item ID="5913" action="3" deposit="1"/>
<item ID="20206" action="3" deposit="1"/>
<item ID="20205" action="3" deposit="1"/>
<item ID="20029" action="3" deposit="1"/>
<item ID="20202" action="3" deposit="1"/>
<item ID="16124" action="3" deposit="1"/>
<item ID="7642" action="1" deposit="3"/>
<item ID="3098" action="5" deposit="3"/>
<item ID="3052" action="5" deposit="3"/>
<item ID="7643" action="3" deposit="1"/>
<item ID="20207" action="3" deposit="1"/>
<item ID="5914" action="3" deposit="1"/>
<item ID="3428" action="4" deposit="2"/>
<item ID="7386" action="4" deposit="2"/>
<item ID="16121" action="3" deposit="1"/>
<item ID="8082" action="4" deposit="2"/>
<item ID="7452" action="4" deposit="2"/>
<item ID="3280" action="4" deposit="2"/>
<item ID="3415" action="4" deposit="2"/>
<item ID="8074" action="4" deposit="2"/>
<item ID="8084" action="4" deposit="2"/>
<item ID="5911" action="3" deposit="1"/>
<item ID="3554" action="4" deposit="2"/>
<item ID="3420" action="4" deposit="2"/>
<item ID="3371" action="4" deposit="2"/>
<item ID="3370" action="4" deposit="2"/>
<item ID="5021" action="3" deposit="1"/>
<item ID="3028" action="3" deposit="1"/>
<item ID="3048" action="5" deposit="3"/>
<item ID="3033" action="3" deposit="1"/>
<item ID="9636" action="3" deposit="1"/>
<item ID="7426" action="4" deposit="2"/>
<item ID="9637" action="3" deposit="1"/>
<item ID="3030" action="3" deposit="1"/>
<item ID="3027" action="3" deposit="1"/>
<item ID="6499" action="3" deposit="1"/>
<item ID="3055" action="4" deposit="2"/>
<item ID="3731" action="3" deposit="1"/>
<item ID="9057" action="3" deposit="1"/>
<item ID="3029" action="3" deposit="1"/>
<item ID="3032" action="3" deposit="1"/>
<item ID="3381" action="4" deposit="2"/>
<item ID="3281" action="4" deposit="2"/>
<item ID="5944" action="3" deposit="1"/>
<item ID="3026" action="3" deposit="1"/>
<item ID="3324" action="4" deposit="2"/>
<item ID="10312" action="3" deposit="1"/>
<item ID="10306" action="3" deposit="1"/>
<item ID="3039" action="5" deposit="2"/>
<item ID="10316" action="3" deposit="1"/>
<item ID="5741" action="4" deposit="2"/>
<item ID="5887" action="3" deposit="1"/>
<item ID="5906" action="3" deposit="1"/>
<item ID="821" action="4" deposit="2"/>
<item ID="5889" action="3" deposit="1"/>
<item ID="817" action="5" deposit="3"/>
<item ID="827" action="4" deposit="2"/>
<item ID="3071" action="4" deposit="2"/>
<item ID="3391" action="4" deposit="2"/>
<item ID="7412" action="4" deposit="2"/>
<item ID="7388" action="4" deposit="2"/>
<item ID="3016" action="5" deposit="2"/>
<item ID="281" action="3" deposit="1"/>
<item ID="3067" action="4" deposit="2"/>
<item ID="7382" action="4" deposit="2"/>
<item ID="3057" action="5" deposit="3"/>
<item ID="7416" action="4" deposit="2"/>
<item ID="3081" action="5" deposit="3"/>
<item ID="5888" action="3" deposit="1"/>
<item ID="3332" action="4" deposit="2"/>
<item ID="3010" action="3" deposit="2"/>
<item ID="2958" action="4" deposit="2"/>
<item ID="5479" action="4" deposit="2"/>
<item ID="3364" action="4" deposit="2"/>
<item ID="3284" action="4" deposit="2"/>
<item ID="3356" action="4" deposit="2"/>
<item ID="3063" action="4" deposit="2"/>
<item ID="811" action="4" deposit="2"/>
<item ID="16131" action="3" deposit="1"/>
<item ID="5910" action="3" deposit="1"/>
<item ID="3318" action="4" deposit="2"/>
<item ID="3037" action="4" deposit="2"/>
<item ID="818" action="4" deposit="2"/>
<item ID="6553" action="4" deposit="2"/>
<item ID="3038" action="4" deposit="2"/>
<item ID="826" action="4" deposit="2"/>
<item ID="7421" action="4" deposit="2"/>
<item ID="3322" action="4" deposit="2"/>
<item ID="3036" action="4" deposit="2"/>
<item ID="3019" action="4" deposit="2"/>
<item ID="3360" action="4" deposit="2"/>
<item ID="3414" action="4" deposit="2"/>
<item ID="8061" action="4" deposit="2"/>
<item ID="3340" action="4" deposit="2"/>
<item ID="6299" action="5" deposit="2"/>
<item ID="6525" action="4" deposit="2"/>
<item ID="5668" action="4" deposit="2"/>
<item ID="3342" action="4" deposit="2"/>
<item ID="9688" action="3" deposit="1"/>
<item ID="5801" action="4" deposit="2"/>
</control>
</panel>

[iudicium105]

I'm at a loss now. No clue how to get this to work. I knew I shouldn't have activated double before testing this update lol.

Here's what I'm using that is NOT working.


Self.DepositItems({20201,0},{9688,0},{9636,0},{963 7,0},{20029,0},{20205,0},{20206,0},{20207,0},{2020 2,0},{20199,0},{20198,0},{20200,0},{10422,0},{9632 ,0},
{10408,0},{10310,0},{10309,0},{9694,0},{10282,0},{ 10313,0},{5879,0},{5893,0},{11680,0},{11681,0},{11 447,0},{237,0},{3732,0},{2903,0},{3051,0},
{3557,0},{3066,0},{10312,0},{10316,0},{10306,0},{3 441,0},{3372,0},{16131,0},{3030,0},{3033,0},{3026, 0},{3028,0},{3027,0},{9057,0},{3029,0},{3032,0},
{16120,0},{16123,0},{16126,0},{16124,0},{16121,0},
{281,0},{238,0},{5895,0},{5887,0},{5944,0},{239,0} ,{9058,0},{7368,0},{5925,0},{5880,0},{20062,0},{59 13,0},{7642,0},{5914,0},{5911,0},
{5021,0},{6499,0},{3731,0},{5906,0},{5889,0},{5888 ,0},{5910,0})
Self.DepositItems({7451,1},{7454,1},{3280,1},{3415 ,1},{8092,1},{3419,1},{3392,1},{3079,1},{3061,1},{ 3351,1},{3373,1},{3265,1},{3357,1},{7412,1},
{3381,1},{4839,1},{10244,1},{3407,1},{10390,1},{10 406,1},{8896,1},{3038,1},{11679,1},{5884,1},{3073, 1},{3071,1},{3391,1},{3284,1},{3428,1},{3371,1},{3 370,1},
{3324,1},{7407,1},{8082,1},{8084,1},{3371,1},{3369 ,1},{7419,1},{3370,1},{7413,1},{3281,1},{3069,1},{ 3053,1},
{3067,1},{3318,1},{7421,1},{3322,1},{3554,1},{7426 ,1},{3055,1},{3420,1},{3436,1},{7404,1},{814,1},{8 11,1},{813,1},{7418,1},{7387,1},{3344,1},{9302,1}, {7452,1},
{3315,1},{812,1},{7386,1},{5741,1},{8052,1},{828,1 },{7456,1},{3554,1},{7396,1},{3342,1},{3055,1},{59 30,1},{6299,1},{821,1},{827,1},
{7388,1},{7382,1},{3332,1},{3010,1},{2958,1},{3364 ,1},{3356,1},{3063,1},{818,1},{6553,1},{826,1},{30 19,1},{3360,1},{3414,1},{8061,1},{10389,1},{8074,1 },
{3039,1},{7416,1},{5479,1},{3037,1},{3036,1},{3340 ,1},{6525,1},{5668,1})
Self.DepositItems({3098,2},{3052,2},{3049,2},{3048 ,2},{3081,2},{817,2},{3016,2},{3057,2})

This just opens up the dp, waits a second or so then closes the dp and walks away.

I just tested this on a different script that I haven't edited at all today and the function still works...


Self.DepositItems({5925, 0}, {9632, 0}, {10408, 0}, {5880, 0}, {10310, 0}, {10309, 0}, {7643, 0}, {3032, 0}, {9694, 0}, {3029, 0}, {10282, 0}, {10313, 0}, {5879, 0}, {5893, 0}, {11680, 0}, {11681, 0}, {11447, 0}, {3033, 0}, {7368, 0}, {7642, 0}, {237, 0}, {3732, 0})

What could possibly be in my script OTHER than the depositing function that would cause the depositer to stop working properly. Both are calling the actual function from the xb lib yet one works and the other doesn't. Both worked prior to this update.

[DarkstaR]

Your original depositor code should NOT have a problem, the depositor has not been changed. I'm investigating the "not enough memory" issue.

[shadowart]

I've experienced this with other very long one-liners as well, even before the latest update.

[DarkstaR]

These obviously aren't one-liners. One liners have a max length of 256 chars.

[iudicium105]

Could it be an issue with my script and/or my settings file being too big. I thought I read somewhere that the walker can only support so many waypoints.

Messing around with it more I can get older versions of the same script to work.

[DarkstaR]

The out of memory error makes it seem like you're running out of RAM. What's your system specs? How much memory is Tibia using in Task Manager?

[iudicium105]

I doubt that's the case. Do you limit the amount of memory each xenobot environment uses?

I am running win 7 x64. I have 8gb RAM and I was only running tibia, chrome and team speak.

I normally run 6 clients before I start running into any issues with any of them bugging (usually they start experiencing issues on client 7). When I was testing this yesterday and getting errors when I was only running 1 client. I always throttle my fps to 10-30 when using a script and bump it up to 60-100 when playing myself. I have never run I to this issue before.

Here's some other stuff I changed.

Added the deposit money and withdraw money waypoints and removed bank function. (I removed the function and waypoint but left the script in my library.)

Added script and two waypoints to switch path finder between map clicks and arrow keys when going to hunt and resupplying.

Added some waypoints and a script to sell empty vials before depositing loot after hunting. (I don't drop vials and the withdraw script would stand idle and log off because it didn't have enough cap to withdraw all items.)

Added reach dp waypoint and deposit item waypoint. (Went back to old deposit items script after the new waypoint crashed my client, took out the reach do script and left the new reach do waypoint.)

Updated the looter with deposit item destinations. (Set all to 0 when I switched back to the deposit script. Still doesn't work.)

I can give you more information about my script aftet work in private if it would help.

[DarkstaR]

If a basic deposit script isn't working without the waypoints, then you've fucked something up. There's absolutely no way that the waypoint in any way effected the regular Self.DepositItems.

[iudicium105]

Well I posted my self.deposititems line earlier, its formatted correctly but doesn't work. There is something causing it to fail I just can't figure out what other code could cause it to fail. This is why I asked if there is a limited amount of memory each script can use.

If it doesn't have the memory available to build the table then will it just not build the table and exit the function without producing any errors?

I call self.deposititems multiple times in this one function (waypoint). Even with this it starts the script on the first call by opening the dp then closing it and moving to the next waypoint without opening the dp again (which should happen since it calls the self.deposititems script again).

Maybe this isn't the cause either since I tested this with only passing 5 or so items to deposit and it still failed.

[iudicium105]

LMAO I just realized what my issue was with the Self.DepositItems script failing. When I switch back to using the script after attempting to use the new built-in depositer I just edited the name of the label back to "Deposit" from "Deposit Items" and left the tag at "252". So it was attempting to call the built-in depositer, saw that there was nothing to deposit and went to the next waypoint.

Such a fail... lol

Though the reason for this thread is still an actual problem/bug. Please continue to look into this as I would like to use the built-in depositer and cannot because it crashes my client.

[DarkstaR]

LMAO I just realized what my issue was with the Self.DepositItems script failing. When I switch back to using the script after attempting to use the new built-in depositer I just edited the name of the label back to "Deposit" from "Deposit Items" and left the tag at "252". So it was attempting to call the built-in depositer, saw that there was nothing to deposit and went to the next waypoint.

Such a fail... lol

Though the reason for this thread is still an actual problem/bug. Please continue to look into this as I would like to use the built-in depositer and cannot because it crashes my client.

Good! I'm glad you can at least use the old one for now! I'm going to experiment to see what is causing the crash.. I only tested with ~10 items before release so I'll try with like 1500.