For some reason my transmute cooldown isn't showing up on my Alchemist. I'm not getting any error messages via Swatter and disabling all other addons doesn't help.
For some reason my transmute cooldown isn't showing up on my Alchemist. I'm not getting any error messages via Swatter and disabling all other addons doesn't help.
ok, i'll check my cooldown stuff. i could understand how it might be broken.
290: attempt to index field 'reverseLookup' (a nil value)
Skillet-1.11-69259\UI\MainFrame.lua:1621: in function `StartQueue_OnClick'
<string>:"*:OnClick":1: in function <[string "*:OnClick"]:1>
290: attempt to index field 'reverseLookup' (a nil value)
Skillet-1.11-69259\UI\MainFrame.lua:1621: in function `StartQueue_OnClick'
<string>:"*:OnClick":1: in function <[string "*:OnClick"]:1>
yeah, that's a bug. there's a simple one line edit to fix that...
I've made a change to BankItems storage format a few days ago.
LibPossessions.lua on line 135 and 151 needs to be updated to
count = count + (bankPlayer[num].count or 1)
and
count = count + (theBag[bagItem].count or 1)
respectively. Basically, I no longer store .count if it is 1 item, so if it is nil, it is assumed to be 1. I did not update your branch, only the main Skillet trunk. The changes were made to reduce savedvariable file size (reduced to 60% of its original size) which makes for faster loading and smaller memory usage. Of course this wasn't the only thing that changed, but only this part is relevant.
I've made a change to BankItems storage format a few days ago.
LibPossessions.lua on line 135 and 151 needs to be updated to
count = count + (bankPlayer[num].count or 1)
and
count = count + (theBag[bagItem].count or 1)
respectively. Basically, I no longer store .count if it is 1 item, so if it is nil, it is assumed to be 1. I did not update your branch, only the main Skillet trunk. The changes were made to reduce savedvariable file size (reduced to 60% of its original size) which makes for faster loading and smaller memory usage. Of course this wasn't the only thing that changed, but only this part is relevant.
thanks for the head's up.
as it currently stands, i'm actually collecting inventory data inside of skillet. since viewing alts is a core function of skillet, it seems a little inconsistent to require the user to load an alt inventory management mod to fully exploit this feature. the data i'm storing is simple item counts for reagents of interest (those identified in known recipes) so there's not a ton being accumulated.
what i don't record, tho, is guild-bank inventories so i may end up having to revert to addons in order to handle this aspect (should guild-bank inventory be something people want to consider).
While all these features and "nice and handy to have", you should also keep in mind that ultimately Skillet is meant to be a simple crafting UI replacement that makes crafting stuff that little bit easier to organize over the default UI.
Adding in support for every minor thing may be quite opposite to that design and lead to an UI that overwhelms the user.
hmm... yeah, the inventory recording isn't to allow the user to inspect their alts for items or anything, it's purely for the sake of crafting counts which is pretty much what you'd expect with any trade skill interface.
hmm... yeah, the inventory recording isn't to allow the user to inspect their alts for items or anything, it's purely for the sake of crafting counts which is pretty much what you'd expect with any trade skill interface.
And by any you mean either... ;) I have yet to find any other tradeskill ui besides Skillet and Advanced Tradeskill Window. :p At least that does what I want it to, mainly expanding the window and sorting the recipes.
as it currently stands, i'm actually collecting inventory data inside of skillet. since viewing alts is a core function of skillet, it seems a little inconsistent to require the user to load an alt inventory management mod to fully exploit this feature.
On the other hand... someone with a lot of alts probably already has something like that installed, so they can see their inventory between characters at any time. No need to reinvent the wheel, I'd think. :)
what i don't record, tho, is guild-bank inventories so i may end up having to revert to addons in order to handle this aspect (should guild-bank inventory be something people want to consider).
Personally, I'd like to see that, but then my main's a guild officer with a 1000 withdrawal per day limit... so I'm not overly worried about withdrawal limits like some other people might be when using an option like this. Of course, it should be just that (an option) and should only count guild bank contents if someone enables it.
On the other hand... someone with a lot of alts probably already has something like that installed, so they can see their inventory between characters at any time. No need to reinvent the wheel, I'd think. :)
yeah, the skillet inventory data isn't meant to be comprehensive by any means. when skillet scans a recipe it id's all the reagents and records item counts on hand, in bank, and craftable (both from bank and on hand). all i'm doing is saving that data instead of tossing it. no real wheel re-inventing, cept maybe trying to be load-time conscious so i'm using strings for the data storage instead of tables.
i figure it's a reasonable amount of data to store to avoid forcing somebody to use a supported bank addon in order to get core functionality.
Personally, I'd like to see that, but then my main's a guild officer with a 1000 withdrawal per day limit... so I'm not overly worried about withdrawal limits like some other people might be when using an option like this. Of course, it should be just that (an option) and should only count guild bank contents if someone enables it.
guild banks are something that really ought to be available. there are people (like me) who don't really belong to real guilds and instead have guilds purely for their alts and rl friends. the guild bank is really just an easier way to give each other stuff.
but doing guild bank scanning would definitely be re-inventing the wheel. the only api call i'm working with for inventory scanning is "GetItemCount". at the moment, i'm guessing guild bank recording would require bag scanning type code that bag addons undoubtedly use. it'd be nice if blizz added guild banks to the GetItemCount function so that you could get a true accouning of items....
hope to see an easier ace updater downloadable version of this some time soon!
sounds like many cool changes have been made, i've been seeing some updates for the original Skillet, is this being merged into that, or are you going to be continuing your own built from this branch down the road?
After receiving more errors than I liked, and not seeing others with the same issues, I deleted Skillet and all its corresponding savedvariables.lua files. This fixed the missing cooldown timer, by the way. But unfortunately it did not get rid of the newest error, which prompted me to try a clean Skillet ala LilSparky. The only modification to your branch I made prior to logging onto my enchanter was to edit line 290 in skilletqueue.lua.
Date: 2008-04-29 15:04:29
ID: 51
Error occured in: Global
Count: 1
Message: ..\AddOns\Skillet\SkilletStitch-1.1.lua line 634:
attempt to perform arithmetic on local 'stackSizeMade' (a nil value)
Debug:
...ce\AddOns\Skillet\Libs\AceEvent-2.0\AceEvent-2.0.lua:299: TriggerEvent()
...ce\AddOns\Skillet\Libs\AceEvent-2.0\AceEvent-2.0.lua:910:
...ce\AddOns\Skillet\Libs\AceEvent-2.0\AceEvent-2.0.lua:903
[C]: UseAction()
..\FrameXML\SecureTemplates.lua:266: SecureActionButton_OnClick()
[string "*:OnClick"]:1:
[string "*:OnClick"]:1
I got this error on my alchemist prior to reinstalling, and reloading the ui worked to correct the issue. On my tailor/enchanter reloading the UI after reinstalling Skillet does nothing and makes crafting impossible. All I can see is one frame of recipes (the same list, no matter which profession), I can't scroll or sort or even use the recipes that I can see.
weird that it would have worked before, but not now. that code is related to buyable reagents and *might* be circumvented if periodic table isn't available. anyway, i'll look into it. i'm hoping to wrap up some stuff today and get up a new version.
pretty soon. the zip version is the same as the svn version still -- neither has been updated yet. i'm pretty happy with where things are going.
i'm hoping to get something up in the next couple days, but i don't wanna post it if it's not reasonably stable.
ok, i'll check my cooldown stuff. i could understand how it might be broken.
290: attempt to index field 'reverseLookup' (a nil value)
Skillet-1.11-69259\UI\MainFrame.lua:1621: in function `StartQueue_OnClick'
<string>:"*:OnClick":1: in function <[string "*:OnClick"]:1>
yeah, that's a bug. there's a simple one line edit to fix that...
line 290 in skilletqueue.lua should read:
DoCraft(skillIndexLookup[recipe.id], command.count)
thanks for the prompt reply <3
edit: hmmm, trying to edit the .lua to sort this, and line 290 according to me is the following:
DebugSpam("ClearQueue Complete")
do i replace that, or add another line, or .... ?
no, it should be a "DoCraft" line that looks kinda similar (search for "reverseLookup" which is causing the error...)
i really am hoping to have a new version up soon, but there's always one more thing to fix it seems...
-
View User Profile
-
View Posts
-
Send Message
Curse PremiumI've made a change to BankItems storage format a few days ago.
LibPossessions.lua on line 135 and 151 needs to be updated to
count = count + (bankPlayer[num].count or 1)
and
count = count + (theBag[bagItem].count or 1)
respectively. Basically, I no longer store .count if it is 1 item, so if it is nil, it is assumed to be 1. I did not update your branch, only the main Skillet trunk. The changes were made to reduce savedvariable file size (reduced to 60% of its original size) which makes for faster loading and smaller memory usage. Of course this wasn't the only thing that changed, but only this part is relevant.
thanks for the head's up.
as it currently stands, i'm actually collecting inventory data inside of skillet. since viewing alts is a core function of skillet, it seems a little inconsistent to require the user to load an alt inventory management mod to fully exploit this feature. the data i'm storing is simple item counts for reagents of interest (those identified in known recipes) so there's not a ton being accumulated.
what i don't record, tho, is guild-bank inventories so i may end up having to revert to addons in order to handle this aspect (should guild-bank inventory be something people want to consider).
-
View User Profile
-
View Posts
-
Send Message
Curse PremiumAdding in support for every minor thing may be quite opposite to that design and lead to an UI that overwhelms the user.
And by any you mean either... ;) I have yet to find any other tradeskill ui besides Skillet and Advanced Tradeskill Window. :p At least that does what I want it to, mainly expanding the window and sorting the recipes.
On the other hand... someone with a lot of alts probably already has something like that installed, so they can see their inventory between characters at any time. No need to reinvent the wheel, I'd think. :)
Personally, I'd like to see that, but then my main's a guild officer with a 1000 withdrawal per day limit... so I'm not overly worried about withdrawal limits like some other people might be when using an option like this. Of course, it should be just that (an option) and should only count guild bank contents if someone enables it.
yeah, the skillet inventory data isn't meant to be comprehensive by any means. when skillet scans a recipe it id's all the reagents and records item counts on hand, in bank, and craftable (both from bank and on hand). all i'm doing is saving that data instead of tossing it. no real wheel re-inventing, cept maybe trying to be load-time conscious so i'm using strings for the data storage instead of tables.
i figure it's a reasonable amount of data to store to avoid forcing somebody to use a supported bank addon in order to get core functionality.
guild banks are something that really ought to be available. there are people (like me) who don't really belong to real guilds and instead have guilds purely for their alts and rl friends. the guild bank is really just an easier way to give each other stuff.
but doing guild bank scanning would definitely be re-inventing the wheel. the only api call i'm working with for inventory scanning is "GetItemCount". at the moment, i'm guessing guild bank recording would require bag scanning type code that bag addons undoubtedly use. it'd be nice if blizz added guild banks to the GetItemCount function so that you could get a true accouning of items....
sounds like many cool changes have been made, i've been seeing some updates for the original Skillet, is this being merged into that, or are you going to be continuing your own built from this branch down the road?
:)
I got this error on my alchemist prior to reinstalling, and reloading the ui worked to correct the issue. On my tailor/enchanter reloading the UI after reinstalling Skillet does nothing and makes crafting impossible. All I can see is one frame of recipes (the same list, no matter which profession), I can't scroll or sort or even use the recipes that I can see.
-
View User Profile
-
View Posts
-
Send Message
Curse Premiumdoes funky things...
i try to build 3 injector (60 healpots)
skillet want to build 60 injectors