With the Savedvariables data changes is there a working version of DKPMonInit for v2.0?
We're using a 0-sum system with fixed prices, I was thinking to "migrate" our prices over from our current addon
and run them side by side for a short period of time before we decide if we make the switch over to DKPMon.
Thanks for this addon btw, it seems like a godsent with all the new dungeons coming shortly.
Especially the flexibility it offers regarding managing many / one pool of points.
If it's irrelevant with the savedvariables changes ignore my post I'll know soon enough :)
ehm edit2:
Not directly related to v2.
We're using a shared pull for MC/Ony/BWL since those instances share the class sets.
(t1 and part of t2 in mc, another part of t2 in Ony lair, another part of t2 in bwl)
We're also using an upgrade rule:
A t2 item can be worth the full price, or an upgrade price if it's an upgrade for t1 in the same slot.
That is to ensure that equal equipment costs our members the same regardless upgrade path,
and also encourage bidding on smaller upgrades as opposed to hording points for a t0->t2 upgrade.
Downgrade works the opposite way. t0 gets prio for t1, if all have it or not bidding t2 that doesn't have it
can take it for 0 points (if going for t1 setbonus fx) since it's already included in the full t2 price he paid for that slot.
Not to wander off further.
Can there be a way to assign multiple values to an item
("0", "upgrade price", "full price" would be enough for us in the above scenario)
or even better have an option to change the price of the item after winner is selected,
but before points are subtracted / distributed to raid?
Bullsi: You somehow managed to miss the whole discussion about decursive and "smart" emergency lists
purposely getting removed from the game by Blizzard.
Here's your file with the trivial lua 5.1 errors fixed.
(pairs added to table iterators, math.mod replaced with math.fmod etc)
Still the problems are not with lua.
A quick browsing of the code reveals that the addon uses alot of targeting and spellcasting commands that are now protected.
So even if it loads without errors, you'll still have to find alternate ways of doing things.
(minus libs and sounds) - 9kb - if you have other ace addons that utilize the same set of libraries
else you have to divide the kb per addon using them to get an approximation.. (as if addon size per se mattered)
Consistent UI and config interface across all Ace2'd addons.
(even if it is missing the chatbubble integration and settings for Raid Leader and Battleground Leader sounds)
when did Ace become micro soft?
I'm not picking a fight with Ace, 80% of my addons are Ace2 and I love them
(including core addons like ora2, grid, bigwigs as well as "fun" addons)
but there's something called common sense (and courtesy).
If the current author picked up the mod from Almostconnected (original author)
then I rename or discontinue my version to avoid confusion for end-users, and problem solved.
The original ChatSounds has been maintained by me for months now...
it was updated and posted on UI sites Chatsounds (worldofwar) and Chatsounds (wowi)
from the moment the ptr was up and again 2 days before the patch went live.
Furthermore I frankly can't see what a simple mod with no performance impact and about a 100 lines of code
can gain from being Aced.
local timetogoal = (endrep - real) / (repdiff/timediff) from line 668
needs to be moved outside the conditional starting at 659
if the second block (line 678) ever runs timetogoal is out of scope.
That actually happens if nothing is loading the Standalone Ace2 (and specifically AceComm-2.0) before GrindStatus.
If you rely on the distribution _with_ embeds and you don't have Ace2 loaded, that error comes up and is traced back to Quixote not finding AceComm.
You bring up a good point about Quixote needing AceComm, and I'll add that to the embeds, thanks :D . However, I don't think that's the error we're seeing here, as Gandharva's AceEvent that's throwing the error is at Interface\AddOns\Ace2\AceEvent-2.0\AceEvent-2.0.lua. Quixote should throw a "requires AceComm" error if that's missing.
You're absolutely right I mixed up two different issues I had :*)
the GrindStatus.lua:332 error seems to be GetRealNumFactions
not returning anything when called from OnEnable.
I put 'return realnumfactions or 0' to stop it from erroring but that's not a bugfix
it's a band-aid.
By the very fact the band-aid works to stop the visible error it seems that there's some situation
where the first conditional never runs (realnumfactions is nil) and the second conditional runs but
completes the full 500 repetitions without encountering 'header and nextheader' both 1 so ends up
returning realnumfactions nil which in turn gives the error at 332
edit: forgot to say this is an amazing mod.. I keep tinkering with it because nothing comes close
where faction monitoring is involved :D
One thing that would make this even more convenient (for me) would be a way to open the search from the commmandline:
/sanity 'partial item name'
(a feature I'm mostly used to from the late Possessions:
"/poss twilight" would open the Possesions frame with "twilight" filled as criteria and the result list.
An easy way to know many twilight [mantle|cowl|text] etc I have on my various mail alts fx.)
I can see how this might be a little tricky since Sanity uses the standardized Ace commands system,
but doesn't hurt to throw it out there :)
In any case it's a great mod, extremely usefull for me many thx for sharing your work.
That message can be very misleading.
Your best bet is to post a list of your addons so other users can cross-reference them.
(I have isolated a couple of addons that give errors with disenchanting myself
and needless to say they weren't the one reported by Blizz)
0
Consider taking a look at SmartBuff to see if it provides the functionality your previous addon did or a suitable equivalent.
Your jumping in to fix it is admirable btw, maybe keep working at it with less pressure if you find a replacement.
0
We're using a 0-sum system with fixed prices, I was thinking to "migrate" our prices over from our current addon
and run them side by side for a short period of time before we decide if we make the switch over to DKPMon.
Thanks for this addon btw, it seems like a godsent with all the new dungeons coming shortly.
Especially the flexibility it offers regarding managing many / one pool of points.
If it's irrelevant with the savedvariables changes ignore my post I'll know soon enough :)
ehm edit2:
Not directly related to v2.
We're using a shared pull for MC/Ony/BWL since those instances share the class sets.
(t1 and part of t2 in mc, another part of t2 in Ony lair, another part of t2 in bwl)
We're also using an upgrade rule:
A t2 item can be worth the full price, or an upgrade price if it's an upgrade for t1 in the same slot.
That is to ensure that equal equipment costs our members the same regardless upgrade path,
and also encourage bidding on smaller upgrades as opposed to hording points for a t0->t2 upgrade.
Downgrade works the opposite way. t0 gets prio for t1, if all have it or not bidding t2 that doesn't have it
can take it for 0 points (if going for t1 setbonus fx) since it's already included in the full t2 price he paid for that slot.
Not to wander off further.
Can there be a way to assign multiple values to an item
("0", "upgrade price", "full price" would be enough for us in the above scenario)
or even better have an option to change the price of the item after winner is selected,
but before points are subtracted / distributed to raid?
Hope I'm making some sense
0
Not 2.x compliant yet but an Ace2 version should be in the working.
I don't believe there's another addon for managing pets out there that does some of the things you want.
0
purposely getting removed from the game by Blizzard.
0
(pairs added to table iterators, math.mod replaced with math.fmod etc)
Still the problems are not with lua.
A quick browsing of the code reveals that the addon uses alot of targeting and spellcasting commands that are now protected.
So even if it loads without errors, you'll still have to find alternate ways of doing things.
0
Doesn't this accomplish the same? (regarding Bugsack)
Edit: thanks for explanation :)
0
when did Ace become micro soft?
I'm not picking a fight with Ace, 80% of my addons are Ace2 and I love them
(including core addons like ora2, grid, bigwigs as well as "fun" addons)
but there's something called common sense (and courtesy).
If the current author picked up the mod from Almostconnected (original author)
then I rename or discontinue my version to avoid confusion for end-users, and problem solved.
0
it was updated and posted on UI sites Chatsounds (worldofwar) and Chatsounds (wowi)
from the moment the ptr was up and again 2 days before the patch went live.
Furthermore I frankly can't see what a simple mod with no performance impact and about a 100 lines of code
can gain from being Aced.
0
needs to be moved outside the conditional starting at 659
if the second block (line 678) ever runs timetogoal is out of scope.
That's what's giving the nil errors.
r20440
0
GrindStatus\GrindStatus.lua:587: attempt to perform arithmetic on local 'real' (a nil value)
edit:
I reverted to r20317 from my local backups ('realnumfactions or 0' hack included)
That one seems to have no tooltip problems.
0
You're absolutely right I mixed up two different issues I had :*)
the GrindStatus.lua:332 error seems to be GetRealNumFactions
not returning anything when called from OnEnable.
I put 'return realnumfactions or 0' to stop it from erroring but that's not a bugfix
it's a band-aid.
By the very fact the band-aid works to stop the visible error it seems that there's some situation
where the first conditional never runs (realnumfactions is nil) and the second conditional runs but
completes the full 500 repetitions without encountering 'header and nextheader' both 1 so ends up
returning realnumfactions nil which in turn gives the error at 332
edit: forgot to say this is an amazing mod.. I keep tinkering with it because nothing comes close
where faction monitoring is involved :D
0
If you rely on the distribution _with_ embeds and you don't have Ace2 loaded, that error comes up and is traced back to Quixote not finding AceComm.
0
/sanity 'partial item name'
(a feature I'm mostly used to from the late Possessions:
"/poss twilight" would open the Possesions frame with "twilight" filled as criteria and the result list.
An easy way to know many twilight [mantle|cowl|text] etc I have on my various mail alts fx.)
I can see how this might be a little tricky since Sanity uses the standardized Ace commands system,
but doesn't hurt to throw it out there :)
In any case it's a great mod, extremely usefull for me many thx for sharing your work.
0
That message can be very misleading.
Your best bet is to post a list of your addons so other users can cross-reference them.
(I have isolated a couple of addons that give errors with disenchanting myself
and needless to say they weren't the one reported by Blizz)
0