Regain that wasted space!

Could those who are having looping issues with the inventory sorting, and are willing to do a bit of debugging to help find the cause, post a ticket here with all the items that are being swapped around with? It's important to list all the items in that group, because the sorting algorithm has a few levels, and the reason it's looping is that the items are matching to the same comparison at different levels, so there ends up being conflicts in the order, depending on which gets compared first. I'll take those items, and give you a line to insert into the code so it'll output exactly how each of the items are being compared/sorted to one another, and from there we can debug where it's making the mistake.

To post a support ticket, please click here

So... uh. Yeah.

A year after the latest release, MrPlow is back. I went through the effort to kinda pimp him up a bit. He will work a lot faster (and without AceTimer/Event). I'd like some feedback on how it deals with special bags, currently it won't group them up well if you have multiple of the same bag, but it will sort within those bags (and consolidate to them all). Also, any suggestions as to the UI would be appreciated.

The commands are as follows:

/mrplow [[g]bank] [stack|defrag|sort|consolidate|theworks]

stack - Restack all items to fit in the smallest number of slots
defrag - Move items to leave all available space at one end of the bags
sort - Sort all inventory items in place
consolidate - Move items from the normal bags to special bags if they fit
theworks - Runs Stack->Consolidate->Defrag->Sort->Stack

this comes out to:

Normal bags

/mrplow stack
/mrplow defrag
/mrplow sort
/mrplow consolidate
/mrplow theworks

Your bank:

/mrplow bank stack
/mrplow bank defrag
/mrplow bank sort
/mrplow bank consolidate
/mrplow bank theworks

Guild bank (No special bags to consolidate with):

/mrplow gbank stack
/mrplow gbank defrag
/mrplow gbank sort
/mrplow gbank theworks

And, if anything goes wrong, and MrPlow appears to be swapping items forever:

/mrplow stop

will apply the emergency brakes.

(/mp is also a shortcut)

10.2.6 - Added a fix for the Spring Flowers (which don't actually have an official equipslot location... damn them..) and one for those who were getting the null reference.

10.2.5 - Er, this bugfix fixes the sorting issue between chests and robes. Also included are some debug switches to make it a little easier to submit a ticket. Check out the bug ticket if you're having issues and follow the prompts there.

10.2.4 - This bugfix fixes the sorting issue between chests and robes. Temporarily, if you want, use /script MrPlow:Ignore(bag, slot) to ignore a specific slot. I'll have it shoved into a config ui soon. Personal bags are 0,1,2,3,4 from the backpack, bank is -1,5,6,7,8,9,10,11. Remove the reference to the slot if you want to ignore a bag (MrPlow:Ignore(bag)). Guildbank stuff is a little more involved for setting the ignore settings. I'll have this all in a ui at some point soon.

10.2.3 - This version fixes the issue in regards to equipment not sorting correctly, along with tradeskill items. This will also fix some of the infinite sort loops that occur in regards to Simple Grinding Stones and others.

10.2.2 - This is a embeds fix for people who want a clean install to actually work. Apologies for the messup on that.

10.2.1 - If you want reversed functionality, hit /script MrPlow.db.profile.EmptySpace = "Top" for the moment, until I can get a UI config going. This will set a toggle to affect the normal direction of flow of the addon. Set /script MrPlow.db.profile.EmptySpace = "" to undo it.

10.1 has guildbank functionality, please be aware however, that this is not perfect (and I very much doubt it will ever be). The problem with guild banking, is that due to it's shared functionality with other players, the response time for any action is far longer than your personal inventory/bank.

Thus, applying the normal speeds to the operation results in a hozillion spammed attempts at moving items. It all sounds very pretty, but is less than effective, as it attempts to move stuff, checks the order, tries to move it again, checks the order, etc etc repeat til you hear a ringing sound in your ears.

I've put in a throttle to slow down the cycles and the gbank will move a whole lot closer to what the original MrPlow sorting speeds were, and also, due to lag and response times from the server reporting where items in the guild bank are, MrPlow -may- get utterly confused near the end and endlessly cycle some items around in a loop. As a patch for this eventuality, there's now a /mp stop command to cease his endless plowing if you notice a loop occurring. Generally, given enough time, it'll sort itself out, but I'm sure you all have better things to do, when it's down to two or three last items =) -

You must login to post a comment. Don't have an account? Register to get one!

22 comments
1
2
3
Next →

- #23
Ghostblades Sun, 10 Jan 2010 23:29:08
Hi,

For some reason, whenever I try to use the /mrplow commands, nothing happens and absolutely nothing is output to the chat box....it's as if I never entered a chat command at all. If I click on the mr.plow icon, it will sort my bags but thats all that I can get it to do. I can't execute any commands at all though.

Any ideas? Thanks for your work!

Last edited on 10 Jan 2010 by Ghostblades
- #22
wobin Tue, 29 Dec 2009 22:48:50
hm, yes. I'll adjust for that.

Although I'd not say there's no benefit to that! There's always the opportunity for amusing misequips at inopportune moments!

Last edited on 29 Dec 2009 by wobin
- #21
Natronx77 Tue, 29 Dec 2009 05:37:59
Is it feasible to organize equipable items by rarity before item slot?

Currently, all my hat items are lined up, sorted by rarity. This mixes seasonal items in with epics, and there is no benefit to having joke hats next to epic hats.

I would like all my white items lined up, sorted by slot and all my epic items together, sorted by slot.
- #20
spode2 Tue, 22 Dec 2009 19:45:12
Glad MrPlow is back! But I miss the gossip he used to say when driving around in my bags. Any chance of adding that? :-)
- #19
LaoTseu Sun, 13 Dec 2009 15:01:48
Hello, I wanted to beta test but the lack of menu on the LDB Data broker was stoping me. I added a some code to get a menu to my local version. It's not very polished and doesn't have any localization code but it does the job for me and might allow others to beta test the MrPlow more.

Here's the code

Note that I put Defrag instead of The Works as the default. It's my personal preference, feel free to change that. The menu also allow to hide or show the minimap icon.

BTW, for those who just want to hide the minimap icon and don't want to modify their local copy of the addon, use the following script:
/script LibStub("LibDBIcon-1.0"):Hide("MrPlowLDB");MrPlow.db.profile.hide = true
You only need to do it once since MrPlow will remember not to show the button the next time you log in.

To get the icon back, you run the following script:
/script LibStub("LibDBIcon-1.0"):Show("MrPlowLDB");MrPlow.db.profile.hide = nil
Anyways, I'm very glad you started working again on the addon. So far, it appears to be working way better then it used to.

Best
- #18
wobin Sat, 12 Dec 2009 22:39:00
MrPlow.lua line 95:

icon:Register("MrPlowLDB", myldb, db)

comment that one out
- #17
cyberian Sat, 12 Dec 2009 02:57:51
Error when I tried to sort my bank:

1x attempt to compare nil with table: <in C code>: in function `sort' MrPlow-10.1\PlowEngine.lua:736: in function `?' MrPlow-10.1\PlowEngine.lua:827: in function <MrPlow\PlowEngine.lua:813>

Locals: (*temporary) = <table> { 37 = <table> {} 160 = <table> {} } (*temporary) = nil (*temporary) = <table> { itemName = "Glacial Salmon" itemRanking = 8 itemEquipLoc = "" itemLevel = 70 slot = 13 Set = "" itemStackCount = 20 bag = 7 itemSubType = "Meat" itemRarity = 1 itemMinLevel = 0 itemID = "41809" itemType = "Trade Goods" itemTexture = "Interface\Icons\INV_Misc_Fish_51" itemArmWepRanking = 27 itemLink = "|Hitem:41809:0:0:0:0:0:0:0:71[Glacial Salmon]" pos = 80 } (*temporary) = nil

-
- #16
cyberian Sat, 12 Dec 2009 02:48:45
I'd like to get rid of that minimap button. What code do I need to comment out? Thanks!
- #15
wobin Sun, 06 Dec 2009 07:04:24
@Excelis is how I got MrPlow onto Docking Station

Last edited on 06 Dec 2009 by wobin
- #14
Excelis Thu, 03 Dec 2009 03:52:21
As of v10.2.2-Beta:
- Still hanging up on Simple Grinder when I have it in my bags.
- Still can't seem to make it's icon show on Docking Station. I'm not very familiar with Docking Station so it might be me. I'll keep working with it.
- Still looking forward to ignoring. :)