re: Sedna Permanent Silence

• I looked at the code, but didn't see anything obvious. Without something more concrete and preferably a way to replicate, I don't think I can do anything about this.

re: http://www.box.net

• I added it as download location.
  

Had a look into oak shield interupt,

Both DA and Sedna use a loop affect and it might be the looping that cancels out a teleport:

ie Da has

Affects = {

Interrupt = {Bool = true},

},

and sedna uses:

 Affects = {
 Silence = {Bool = true},

 },

Da may be able to be fixed by changing interupt to an add affect (ie

 Affects = {
 Interrupt = {Add = 0},

 }

This should not cause the interrupt to not work as this is how pentinence is coded. However, I am not sure whether DA was given a loop to allow him to teleport sheilded oak, and the interrupt during shield is a side effect. Been trying to test this change on Ai but it is difficult.

Code would be (HDemon_abilitites.lua)

#################################################################################################################
# Buff: Shadow Swap
#################################################################################################################

BuffBlueprint {
    Name = 'HDemonShadowSwap01',
    Debuff = true,
    BuffType = 'HDEMONSHADOWSWAP',
    EntityCategory = 'HERO',
    Stacks = 'REPLACE',
    Duration = 0.1,
    Affects = {

#Change

        Interrupt = {Add = 0},

#End of Change   

},

}

_____________________________________________

For silence a different approach is required i think. This should work, and seems to cause no negative effects ingame on or the debugger. However, once again testing is difficult.

Code would be (BuffAffects.lua)

#################################################################################################################
# Silence
#################################################################################################################

function Silence( unit, buffName, buffDef, buffAffects, instigator, instigatorArmy, bAfterRemove )

#Added Code

  if Buff.HasBuff(unit, 'HOAKShield03') then
                    continue
                elseif Buff.HasBuff(unit, 'HOAKShield04') then
                    continue
                elseif Buff.HasBuff(unit, 'HOakPurity') then
                    continue
                end,
#End of Added Code

    local val, bool = BuffCalculate(unit, buffName, 'Silence', 0, false)
    unit.Sync.Silenced = bool
    unit.Silenced = bool

    if (instigator and not instigator:IsDead() and unit != instigator ) then
        FloatTextAt(unit:GetFloatTextPositionOffset(0, 1, 0), '<LOC floattext_0015>SILENCED', 'Stunned', unit:GetArmy())
    end
end

________________

Also one other note, has anyone considered this might be shield running out at the end of the teleport, 6 seconds is barely long enough to do caplock and tele, considering that there is always minimum response time of 0.3 seconds, fastest possible combination would take 5.6 seconds (ie 2*response time of 0.3 + 3 second cast teleport and +2 second cap lock) and would be a feasible cause of canceling teleport on the swoosh (ie shield ends at just before teleport ends casting)?

Could not replicate could you be more specific?

 I think it needs a fix, i dont think that it is working as intended

When the UB inflicts a unit with Plague and it dies from the damage from Plague, the unit does not explode with a Post Mortem explosion. Also, kills due to Post Mortem (and possibly Plague) do not give the UB gold.

I think so too, but I wanted to make sure that sufficient people agreed before I went ahead and did something about it. It should be simple enough.

Bug:

Orb of Defiance deals 500 weapon damage, not 500 skill damge. (read: it is affected by armor)

With Queen you do over 1000 damage to creeps/giants after Ground Spikes, but it sucks against other demigods.

re: creeps

The easy way to test it is to set creep strength to high, waves to frequent, and start at level 10. Then stand in the center of the creeps. Take Post Mortem and not plague. Watch how often enemies explode. Now put a point into Plague. You'll notice that the damage spike explosion occurs very infrequently.

re: mines

No, the Regulus's mines don't give kill credit, nor do Rook's towers. The demigod wiki has a reasonable description of of the kill mechanics and what gives credit and what doesn't. I'm not certain if Regulus mines and Rook Towers should give credit, however the AOE damage due to post mortem seems to be a reasonable extension. Otherwise the UB is penalized for picking up the ability and creeping.

re: Orb of Defiance

I'm not sure which item you are talking about hedgie. The orb of defiance gives armor & health. It doesn't have a damage effect.

oh.  reading a few posts up:

I did extensive testing on UB post mortem.  It does NOT provide XP, which is unlike every other aoe in the game (tested surge of faith, slam, heaven's wrath, reg mines, angelic fury, qot spikes and spikewave, orb of veiled storms, etc) which all give aoe.  Obviously no Stardock / GPG response when posted on the forums several months back.

I know that everyone has a lot of frustration about the status of Demigods. GPG is having problems with 3rd party netcode that failed them, coding hours, and budget problems. Stardock is having problems because they aren't the only distributor, which means they have to deal with Atari to get some things done, they have to deal with the tremendous amount of negative output by the community, and they've been pouring in money to save Demigods, when they had no responsibility to do so in the first place. The community has to deal with what they perceive as a lack of faith and support by GPG & Stardock.

This however, isn't the place for complaints. This thread and this mod is all about working with everyone, the community, with Stardock, and with GPG to make Demigods a better game.

• The community is responding by providing buglists, replication methods, and making bug fixes.
  
• Stardock is responding by trying to get this patch into the base install of Demigods.
• Gas Powered Games hasn't made a direct response yet, but Supreme Commander 2 is due out soon, so I'm sure they are crunched for time. However, in the past Sorian has done a lot for Demigods, and I'm sure he will end again.

re: Games at this time of day

• It's 2:00 AM. You silly people on the wrong side of the Earth need to fix your sleep schedules.

K found a the problem for Ub Post mortem/plague

#################################################################################################################
# Buff - Plague II
#################################################################################################################
BuffBlueprint {
    Name = 'HEPA01Plague02',
    DisplayName = '<LOC ABILITY_HEPA01_0062>Plague',
    Description = '<LOC ABILITY_HEPA01_0042>Taking poison damage. Spreads to allies.',
    BuffType = 'HEPA01PLAGUE',
    Debuff = true,
    CanBeDispelled = true,
    EntityCategory = 'MOBILE - UNTARGETABLE',
    Stacks = 'IGNORE',
    Duration = 30,
    DurationPulse = 30,
    Affects = {
       Health = {Add = -15},
    },
    Effects = 'Plague01',
    EffectsBone = -2,
    DamageSelf = true,
    CanBeEvaded = false,
    CanBackfire = false,
    CanDamageReturn = false,
    DamageFriendly = true,
    ArmorImmune = true,
    CanCrit = false,
    CanMagicResist = true,
    #NoFloatText = true,
    AffectRadius = 10,
    AffectChance = 100,
    NumIgnoreSpreadPulses = 1,                      # Number of pulses to wait before spreading plague
    Icon = '/DGUncleanBeast/NewUncleanPlague01',
    OnApplyBuff = function( self, unit, instigator )
        unit.Callbacks.OnKilled:Add(self.UnitOnKilledCallback, self)
    end,
    OnBuffAffect = function(self, unit, instigator)
        #LOG("*DEBUG: OnBuffAffect unit: "..unit:GetUnitId().. " inst:"..instigator:GetUnitId())
        if unit:IsDead() or instigator:IsDead() or not instigator.Plague then
            return
        end

        # Allows for ignoring OnBuffAffect pulses, so this doesn't trigger immediately on the first
        # application of plague.
        if unit.PlagueInstance.IgnoreAffectPulses then
            unit.PlagueInstance.NumIgnoredPulses = unit.PlagueInstance.NumIgnoredPulses + 1
            if unit.PlagueInstance.NumIgnoredPulses >= self.NumIgnoreSpreadPulses then
                unit.PlagueInstance.IgnoreAffectPulses = false
            end
            return
        end

        local targets = unit:GetAIBrain():GetUnitsAroundPoint((categories.MOBILE - categories.UNTARGETABLE), unit:GetPosition(), self.AffectRadius, 'Ally')
        PlagueSpread02( instigator, targets, self.AffectChance )
    end,

    OnBuffRemove = function(self, unit)
        #self:DecrementPlagueCounter(unit)
        Buff.ApplyBuff(unit, 'HEPA01PlagueImmune', unit)
    end,
    UnitOnKilledCallback = function( self, unit )

    unit.Callbacks.OnKilled:Remove(self.UnitOnKilledCallback)
        #self:DecrementPlagueCounter(unit)
    end,
    DecrementPlagueCounter = function(self, unit)
        local instigator = unit.PlagueInstance.Instigator
        if not instigator then
            LOG( 'No Instigator' )
        end
        if not instigator:IsDead() then
            instigator.Plague.NumPlaguedUnits = instigator.Plague.NumPlaguedUnits - 1
            unit.PlagueInstance = nil
        end
    end,
}

Problem code bolded.

This function removes the callback used by post mortem and so it doesnt activate. Removing it allows post mortem to work as usual. Seems to me tho that this is a development descision and not a coding accident.

Two work arounds that i can see to be fairly easy and effective:

1. delete these line of code but i am not sure what add on effect this will have.

2. Add post mortem effect to all units which die of plague. I cannot see this as a problem as i figure plague is caused by ub anyway.

Code for 2.:

    OnApplyBuff = function( self, unit, instigator )
        unit.Callbacks.OnKilled:Add(self.UnitOnKilledCallback, self)
    end,
    OnBuffAffect = function(self, unit, instigator)
        #LOG("*DEBUG: OnBuffAffect unit: "..unit:GetUnitId().. " inst:"..instigator:GetUnitId())
        if unit:IsDead() or instigator:IsDead() or not instigator.Plague then
            return
        end

        # Allows for ignoring OnBuffAffect pulses, so this doesn't trigger immediately on the first
        # application of plague.
        if unit.PlagueInstance.IgnoreAffectPulses then
            unit.PlagueInstance.NumIgnoredPulses = unit.PlagueInstance.NumIgnoredPulses + 1
            if unit.PlagueInstance.NumIgnoredPulses >= self.NumIgnoreSpreadPulses then
                unit.PlagueInstance.IgnoreAffectPulses = false
            end
            return
        end

        local targets = unit:GetAIBrain():GetUnitsAroundPoint((categories.MOBILE - categories.UNTARGETABLE), unit:GetPosition(), self.AffectRadius, 'Ally')
        PlagueSpread02( instigator, targets, self.AffectChance )
    end,

    OnBuffRemove = function(self, unit)
        #self:DecrementPlagueCounter(unit)
        Buff.ApplyBuff(unit, 'HEPA01PlagueImmune', unit)
    end,
    UnitOnKilledCallback = function( self, unit )

############Added Code
    unit.Callbacks.OnKilled:Add(PMExplode, self)

############ End Added Code

    unit.Callbacks.OnKilled:Remove(self.UnitOnKilledCallback)
        #self:DecrementPlagueCounter(unit)
    end,
    DecrementPlagueCounter = function(self, unit)
        local instigator = unit.PlagueInstance.Instigator
        if not instigator then
            LOG( 'No Instigator' )
        end
        if not instigator:IsDead() then
            instigator.Plague.NumPlaguedUnits = instigator.Plague.NumPlaguedUnits - 1
            unit.PlagueInstance = nil
        end
    end,
}

another thought:  perhaps a name change is in order?  Maybe something more generic to perhaps increase appeal or maybe get more people to use it if it gets bundled with demigod?  Maybe just Bug Fix 1.0. 

I'm all for whatever.  So long as its easily identified to most peeps.