PQR - Rotation Bot

[krewetek]

I've got 2 questiont:

How to detect is "target" is enemy faction player (with valid pvp on, etc. attackable)... I need it to blow some cooldowns if target is ppl..

How to check how much hp does have my pet (I'm hunter)...

Thanks in advance...

[crystal_tech]

I've got 2 questiont:

How to detect is "target" is enemy faction player (with valid pvp on, etc. attackable)... I need it to blow some cooldowns if target is ppl..

How to check how much hp does have my pet (I'm hunter)...

Thanks in advance...

Unithealth("pet") --gives current hp of pet in a raw number not %

if UnitFactionGroup("player") ~= UnitFactionGroup("target") then do something end --faction check

[Decaed]

Ps decaed can you clear your inbox?

My inbox is now cleared.

[Envision]

Been getting flaged AFK as of late.

[smurfalmighty]

My inbox is now cleared.

Your inbox is full again already; assuming people messaged you for an updated version (wanted to myself). If you could possibly PM me your edits; also i'm going to work on a DW edit in the mean time.

[Weird0]

just curious were there ever offsets released for ptrs ?

[Gabbz]

edit:

I would really appreciate some help here, with the edits I have made recently, the profile bursts 170k~ single target/ 250k~ aoe and has an average of 80k with more than 1 mob, 65-70k single target, in 467 gear with a 476 wep
It would be a real shame not being able to release it, because I can't fix a few problems.

I ranked #4 world in the first week with this profile.

So what is the issue?
"How do I get the time of trinket13 or trinket14? " What time do u mean, the buff or icd?

local doubleJeopardy = { 41092, 57030, 54922, 121027 }
for i=1,#doubleJeopardy do
    if HaveGlyph(i) then 

Change to

local doubleJeopardy = { 41092, 57030, 54922, 121027 }
for i=1,#doubleJeopardy do
    if HaveGlyph(doubleJeopardy[i]) then 

[Hoblerhans]

Verdammt...Sheuron...r u leaving us? Loved ur profiles...would give u more rep to show my appreciation...if the system lets me. Pls dont bother with the dailys...try having fun with people in raids and maybe...just maybe...STAY!

[TheBolivianGuy]

GetDistance()

local PerfectAoETarget = {}
for i=1, #members do
    table.insert(PerfectAoETarget, {NumberNearby = UnitsClose(members[i].Unit, 85), Unit = members[i].Unit} )
end


table.sort(PerfectAoETarget, function(x,y) return x.NumberNearby > y.NumberNearby end)


if PerfectAoETarget[1].NumberNearby >= 4 then
    PQR_CustomTarget = PerfectAoETarget[1].Unit
    return true
end 

this is what i think should work in mine...... but like ive said in my own Thread.... i don't have a pally to test these new spells on so I don't want to release continually breaking profiles as u guys spot test it

Just tried it and it does work except I changed NumberNearby >= 4 to NumberNearby >= 1 just it gets used a little bit more (especially in pvp) thanks for the awesome profiles!

[averykey]

So what is the issue?
"How do I get the time of trinket13 or trinket14? " What time do u mean, the buff or icd?

local doubleJeopardy = { 41092, 57030, 54922, 121027 }
for i=1,#doubleJeopardy do
    if HaveGlyph(i) then 

Change to

local doubleJeopardy = { 41092, 57030, 54922, 121027 }
for i=1,#doubleJeopardy do
    if HaveGlyph(doubleJeopardy[i]) then 

About the trinket, I want to get the time of an on use trinket.
I want to compare trinket time for glove use so I don't delay the trinket 10 seconds

The doublejeopardy code

I tried this in my last post but it didn't seem to work, if I get rid of my double jeopardy glyph it breaks the whole rotation?

local doubleJeopardy = { 41092, 57030, 54922, 121027 } 
for i=1,#doubleJeopardy do 
    if HaveGlyph(doubleJeopardy[i]) then 

HaveGlyph function

function HaveGlyph(id)
    for i=1, GetNumGlyphSockets() do
        if select(4, GetGlyphSocketInfo(i)) == id then return true end
    end
    return false
end 

Judgment code

-- HoW Check
local howSTART, howDURATION = GetSpellCooldown(24275)
if howSTART - GetTime() + howDURATION < 0.1 and howSTART - GetTime() + howDURATION > 0 then
    return false
-- Holy Avenger Check
elseif UnitBuffID("player", 105809) and UnitPower("player", 9) >= 3 then
    return false
-- Prevents Wrong Spell Cast 
elseif PQR_SpellAvailable(879)
or PQR_SpellAvailable(24275) 
or PQR_SpellAvailable(35395) 
or UnitBuffID("player", 87138) then
    return false
end

-- Seal Check
if GetShapeshiftForm() ~= 0 then hasSeal = true else hasSeal = false end

-- Double Jeporady
if switchedTargets == nil then switchedTargets = false end

-- Judgment
if UnitCanAttack("player","target") and UnitAffectingCombat("player") then
    local doubleJeopardy = { 41092, 57030, 54922, 121027 }
    for i=1,#doubleJeopardy do
        if HaveGlyph(i) then
            if UnitExists("focus") and RangeCheck(20271, "focus") and switchedTargets == false then
                CastSpellByName(GetSpellInfo(20271), "focus")
                switchedTargets = true
            elseif UnitExists("target") and RangeCheck(20271, "target") and switchedTargets == true then
                CastSpellByName(GetSpellInfo(20271), "target")      
                switchedTargets = false
            end
        elseif not HaveGlyph(i) then
            if UnitExists("focus") and RangeCheck(20271, "focus") then
                CastSpellByName(GetSpellInfo(20271), "focus")
            elseif UnitExists("target") and RangeCheck(20271, "target") then
                CastSpellByName(GetSpellInfo(20271), "target")      
            end
        end
    end
end 

I will go try it again and see if it works.

[chaos17]

Today, I lag at lot when I use PQR.
Am I the only one ? I play on EU server.

[Gabbz]

@Avery

start, duration, enable = GetInventoryItemCooldown("unit", slotId) will give you cooldown. Use 10 for Gloves and 13 or 14 for trinket.

Regarding your code, it is hard to understand what you are trying to do but you need to perhaps structure it a bit better,

As i understand this we will not actually test all glyphs because the first glyph "41092" only checked since if it exist we execute first statement otherwise we execute the second statement.

if UnitCanAttack("player","target") and UnitAffectingCombat("player") then
local doubleJeopardy = { 41092, 57030, 54922, 121027 }
for i=1,#doubleJeopardy do
if HaveGlyph(i) then
-- We do this
elseif not HaveGlyph(i) then <----------------- This is wrong remove this
-- Or this
end
end
<------------------ Put logic regarding not having glyphs here instead
end


So you basically only check the first glyph.

[Donoverdose]

Hey, can anyone please help me with this addon. Can't seem to get it working and i would much rather talk to someone over mic skype name is "blackmamba19191" any help would be appreciated allot.

[averykey]

@Avery

start, duration, enable = GetInventoryItemCooldown("unit", slotId)
Regarding your code, it is hard to understand what you are trying to do but you need to perhaps structure it a bit better,

As i understand this we will not actually test all glyphs because the first glyph "41092" only checked since if it exist we execute first statement otherwise we execute the second statement.

if UnitCanAttack("player","target") and UnitAffectingCombat("player") then
local doubleJeopardy = { 41092, 57030, 54922, 121027 }
for i=1,#doubleJeopardy do
if HaveGlyph(i) then
-- We do this
elseif not HaveGlyph(i) then <----------------- This is wrong remove this
-- Or this
end
end
<------------------ Put logic regarding not having glyphs here instead
end


So you basically only check the first glyph.

I'm not sure which id is the double jeopardy glyph, so I got all the ids that have something to do with the glyph, so if any id is correct it will work.

Trink code?

local enable10 = GetInventoryItemCooldown("unit", 10)  
local start13, duration13, enable13 = GetInventoryItemCooldown("unit", 13)  
local start14, duration14, enable14 = GetInventoryItemCooldown("unit", 14)  

if (start13 - getTime() + duration13) > 10 and (start14 - getTime() + duration14) > 10 and enable13 == 1 and enable14 == 1 and enable10 == 0 then
UseInventoryItem(10)
end 

Spaced it out so it makes more sense

if UnitCanAttack("player","target") and UnitAffectingCombat("player") then

    local doubleJeopardy = { 41092, 57030, 54922, 121027 }
    for i=1,#doubleJeopardy do

        if HaveGlyph(doubleJeopardy[i]) then

            if UnitExists("focus") and RangeCheck(20271, "focus") and switchedTargets == false then
                CastSpellByName(GetSpellInfo(20271), "focus")
                switchedTargets = true
            elseif UnitExists("target") and RangeCheck(20271, "target") and switchedTargets == true then
                CastSpellByName(GetSpellInfo(20271), "target")      
                switchedTargets = false
            end

        elseif not HaveGlyph(doubleJeopardy[i]) then

            if UnitExists("focus") and RangeCheck(20271, "focus") then
                CastSpellByName(GetSpellInfo(20271), "focus")
            elseif UnitExists("target") and RangeCheck(20271, "target") then
                CastSpellByName(GetSpellInfo(20271), "target")      
            end

        end -- ends glyph check

    end -- ends double jep check

end -- ends combat/attack check, I have this because it will attack once from range but if I'm kiting it won't attack again because the only combat check I have is in pause 

Does someone have a better incombat check, the current one seems a bit slow? I have an idea for a better check.

tankcheckcode, check for roles in current group

if (tankhasagro or tankisattacking or tankstargetisattacking) or (playerhasagro or playerisattacking or playerstargetisattacking) then
incombat = true else incombat = false end

a better way to to check party members and if 2 have agro or the tank has agro or the player has agro then start attacking, but I don't know how to write that.

[Gabbz]

I perhaps was a bit unclear

if UnitCanAttack("player","target") and UnitAffectingCombat("player") then
    local doubleJeopardy = { 41092, 57030, 54922, 121027 }
    for i=1,#doubleJeopardy do
        if HaveGlyph(doubleJeopardy[i]) then
            if UnitExists("focus") and RangeCheck(20271, "focus") and switchedTargets == false then
                CastSpellByName(GetSpellInfo(20271), "focus")
                switchedTargets = true
                return true   -- Should add this
            elseif UnitExists("target") and RangeCheck(20271, "target") and switchedTargets == true then
                CastSpellByName(GetSpellInfo(20271), "target")      
                switchedTargets = false
                return true   -- Should add this
            end 
       end
    end  -- End of the loop going through all glyphs in list
     -- So we are here only if no hit in list 
     if UnitExists("focus") and RangeCheck(20271, "focus") then
        CastSpellByName(GetSpellInfo(20271), "focus")
        return true
      elseif UnitExists("target") and RangeCheck(20271, "target") then
         CastSpellByName(GetSpellInfo(20271), "target")      
        return true
     end
end 
-- ends combat/attack check, I have this because it will attack once from range but if I'm kiting it won't attack again because the only combat check I have is in pause