LibHealComm-4.0

[Shadowed]

Since the previous topic is mostly irrelevant as me and xbeeps agreed on 4.0, starting a new topic.

Project: http://www.wowace.com/addons/libhealcomm-4-0/
Documentation: http://www.wowace.com/addons/libpend...1-0/pages/api/

I've got pretty much all of the code/api/event/documentation done at this point, I need to go through and verify it's all 100% and then I'll write the wrapper and it should be ready for testing.

[xbeeps]

The API looks good. However, for GetHealAmount doesn't seem possible to get the amount of healing incoming to a target from everyone else but yourself before your currently casting heal lands (which is kind of the core functionality used by VisualHeal and the unit frame addons that properly show these as separate, and only show the incoming heals from others that land before your own). For example, if i know that my own heal will land at the time in endTime i will call GetHealAmount like this:

HealComm:GetHealAmount(guid, HealComm.ALL_HEALS, endTime)

However, there's no way to know if the heal I'm currently casting will be included in that, because the timeframe ends exactly on the point where my heal lands. I guess it depends on both rounding errors and whether you use > or >= for comparison, or other subtle things, so it's not possible to subtract the players heal size from the value returned. Regardless, from an API point of view, it should be possible to get the heal amount without the player contribution. LHC-3.0 never returned the heal from yourself in this call, and relied on the addon to remember that value by itself whenever a heal was active (which is just a matter of saving the amount in a local variable whenever a heal start event is triggered with the player as the healer).

Apart from that, i think everything looks good. Perhaps you want to reconsider exposing the internal tables, because when they are exposed you get locked down in how they are laid out, because changing them is an API change.

[Shadowed]

Originally Posted by xbeeps

The API looks good. However, for GetHealAmount doesn't seem possible to get the amount of healing incoming to a target from everyone else but yourself before your currently casting heal lands (which is kind of the core functionality used by VisualHeal and the unit frame addons that properly show these as separate, and only show the incoming heals from others that land before your own). For example, if i know that my own heal will land at the time in endTime i will call GetHealAmount like this:

HealComm:GetHealAmount(guid, HealComm.ALL_HEALS, endTime)

However, there's no way to know if the heal I'm currently casting will be included in that, because the timeframe ends exactly on the point where my heal lands. I guess it depends on both rounding errors and whether you use > or >= for comparison, or other subtle things, so it's not possible to subtract the players heal size from the value returned. Regardless, from an API point of view, it should be possible to get the heal amount without the player contribution. LHC-3.0 never returned the heal from yourself in this call, and relied on the addon to remember that value by itself whenever a heal was active (which is just a matter of saving the amount in a local variable whenever a heal start event is triggered with the player as the healer).

Apart from that, i think everything looks good. Perhaps you want to reconsider exposing the internal tables, because when they are exposed you get locked down in how they are laid out, because changing them is an API change.

If you wanted to get your own healing, you'd pass your GUID as casterGUID or you could do what Pitbull and agUF do which is when it sees a players heal start/delayed/stopped through the callbacks it saves when the players heal is going to end and requests all heals within that time period.

Besides that for getting the end time it uses the same method 3.0 did with getting the end time through UnitCastingInfo/UnitChannelInfo.

In HealComm-3.0:

-- Add in the players own heals
local healed = playerHeals[target] or 0

-- Add in any heals from other people
local incoming = select(2, HealComm:UnitIncomingHealGet(target, 0))
if( incoming ) then
healed = healed + incoming
end

-- Apply any healing debuffs
healed = math.floor(healed * HealComm:UnitHealModifierGet(target))

Would be done something like the below in HealComm-4.0

	local playerHealed = HealComm:GetHealAmount(guid, HealComm.ALL_HEALS, nil, UnitGUID("player"))
	local restHealed = HealComm:GetHealAmount(guid, HealComm.ALL_HEALS) - playerHealed
	
	playerHealed = math.floor(playerHealed * HealComm:GetHealModifier(guid))
	restHealed = math.floor(restHealed * HealComm:GetHealModifier(guid))

If you needed to get both the total healing incoming and how much the player did.


That's true, the only reason the caster table was exposed is so people can see who has casted a heal or not, but I'll just replace that with a GUIDHasHealed called. The GUID -> Unit table will probably always be there/not changed unless Blizzard decides to let us get units from GUIDs.

[xbeeps]

Hmm, i'm not sure you understand the problem. I want two quantites:

1) The heal that is incoming *only* from others *before* my heal lands.
2) The heal that is incoming from me.

2 is trivial, but 1 is not with the current API. Calling this:

local restHealed = HealComm:GetHealAmount(guid, HealComm.ALL_HEALS) - playerHealed

Will give me all heals that finish before and after mine, because there is no time frame given. I need those that land before mine only. Therefore, the extra argument is added:

local restHealed = HealComm:GetHealAmount(guid, HealComm.ALL_HEALS, endTimeOfCurrentlyCastingPlayerSpell) - playerHealed

However, as the boundary time is exactly the same time as the player spell lands, it is not possible to know if the currently casting player spell is or is not included in the quantity returned from GetHealAmount.

Maybe this picture explains it better: http://www.wowace.com/addons/visualh...es/4-heal-bar/

Note the "Incoming heals from others that will complete before yours"

[Kollektiv]

In response to: http://forums.wowace.com/showpost.ph...6&postcount=90

You should've looked at how LHC4 did it first . The way it does it now is

r1 will do

lib.healingModifiers = lib.healingModifiers or {
[spellname1] = 0.5,
[spellname2] = 1,
}

and r2 will do

lib.healingModifiers = lib.healingModifiers or {
[spellname1] = 0.5,
[spellname2] = 1,
[spellname3] = 1,
}

It is not doing it the way you described. The new entries in the upgraded version flat out never get added.

Also, AuraHandler should be something like below

if AuraHandler then AuraHandler(...) end

[Shadowed]

Originally Posted by Kollektiv

In response to: http://forums.wowace.com/showpost.ph...6&postcount=90

You should've looked at how LHC4 did it first . The way it does it now is

-- Snipped --

It is not doing it the way you described. The new entries in the upgraded version flat out never get added.

Also, AuraHandler should be something like below

if AuraHandler then AuraHandler(...) end

What Xinhuan said is correct, in r1 it will initialize the table and then in r2 onward it will do nothing, if a new entry is added then it's added directly.

So if I add a new spell in r3 it becomes

lib.healingModifiers = lib.healingModifiers or {
    [1] = 0.50,
    [2] = 1,
}

lib.healingModifiers[3] = 2

Oops forgot that it uses UNIT_AURA for classes without data, fixed.

Originally Posted by xbeeps

Hmm, i'm not sure you understand the problem. I want two quantites:

1) The heal that is incoming *only* from others *before* my heal lands.
2) The heal that is incoming from me.

2 is trivial, but 1 is not with the current API. Calling this:

local restHealed = HealComm:GetHealAmount(guid, HealComm.ALL_HEALS) - playerHealed

Will give me all heals that finish before and after mine, because there is no time frame given. I need those that land before mine only. Therefore, the extra argument is added:

local restHealed = HealComm:GetHealAmount(guid, HealComm.ALL_HEALS, endTimeOfCurrentlyCastingPlayerSpell) - playerHealed

However, as the boundary time is exactly the same time as the player spell lands, it is not possible to know if the currently casting player spell is or is not included in the quantity returned from GetHealAmount.

Maybe this picture explains it better: http://www.wowace.com/addons/visualh...es/4-heal-bar/

Note the "Incoming heals from others that will complete before yours"

Perhaps instead an API like:

local healed = HealComm:GetOtherHealAmount(guid, bitFlag, endTime, casterGUID)

that will return all heals within the time band (if any) not counting the players heal, if you wanted to get the players heals + everyone else's before yours land it would be:

local playerHealed = HealComm:GetHealAmount(guid, HealComm.ALL_HEALS, endTime, UnitGUID("player"))
local beforePlayer = HealComm:GetOtherHealAmount(guid, HealComm.ALL_HEALS, endTime)

endTime would be something you kept track of through the heal events to figure out the longest player heal, it would look something like:

function HealComm:GetOtherHealAmount(guid, bitFlag, time)
	local amount = 0
	for casterGUID, spells in pairs(pendingHeals) do
		if( casterGUID ~= playerGUID ) then
			amount = amount + filterData(spells, guid, bitFlag, time)
		end
	end
	
	return amount > 0 and amount or nil
end

[Kollektiv]

Ah, I see how you'll add new entries. Sorry for wasting your time, I've never done something like that.

One more thing, I think you should register CLEU, CHAT_MSG_ADDON, UNIT_AURA, and the rest for healers only when the player is in a group.

[Xinhuan]

Originally Posted by Kollektiv

You should've looked at how LHC4 did it first . The way it does it now is
<snip>
It is not doing it the way you described. The new entries in the upgraded version flat out never get added.

Yes, if it did that then it was flawed upgrading code. This is why for libraries, authors are encouraged to let as many people review the code as possible before letting a beta version out for testing.

Especially lib upgrading/reembedding code.

[xbeeps]

Originally Posted by Shadowed

What Xinhuan said is correct, in r1 it will initialize the table and then in r2 onward it will do nothing, if a new entry is added then it's added directly.

So if I add a new spell in r3 it becomes

lib.healingModifiers = lib.healingModifiers or {
    [1] = 0.50,
    [2] = 1,
}

lib.healingModifiers[3] = 2

Oops forgot that it uses UNIT_AURA for classes without data, fixed.



Perhaps instead an API like:

local healed = HealComm:GetOtherHealAmount(guid, bitFlag, endTime, casterGUID)

that will return all heals within the time band (if any) not counting the players heal, if you wanted to get the players heals + everyone else's before yours land it would be:

local playerHealed = HealComm:GetHealAmount(guid, HealComm.ALL_HEALS, endTime, UnitGUID("player"))
local beforePlayer = HealComm:GetOtherHealAmount(guid, HealComm.ALL_HEALS, endTime)

endTime would be something you kept track of through the heal events to figure out the longest player heal, it would look something like:

function HealComm:GetOtherHealAmount(guid, bitFlag, time)
    local amount = 0
    for casterGUID, spells in pairs(pendingHeals) do
        if( casterGUID ~= playerGUID ) then
            amount = amount + filterData(spells, guid, bitFlag, time)
        end
    end
    
    return amount > 0 and amount or nil
end

That'll would work fine. You can also consider just adding a flag to the list of arguments, like this:

GetHealAmount(guid, bitFlag[, time[, includePlayer[, casterGuid]]])

Or perhaps a HealComm.PLAYER_HEALS bit ? Dunno if that would create some other consistency problems though. You decide!

[Kollektiv]

Not a bug, but I suggest you check that GetSpellInfo(spellid) exists outside the table constructor to safeguard the lib from breaking on patch days. You never know when blizz will remove a spellid.