What is GupPet:
GupPet is an add-on that gives an extension for mounting and calling critters. By a single click it mounts a ground, swimming or fly mount depending on the players location. Because most players have plenty of mounts and companions, and you probably like more than one, the add-on can randomly select mounts/companions depending on the selections in the option menu.
Basic Features:
- Randomly selects a mount / companion location dependent.
- Class support for Druids , Shamans and Mages
- Automatically summons a companion
- 3d model preview
- Masque support
- And many more !
In game usage:
There are 3 types of controls.
- Action buttons (Adds buttons and are move, hide ,scale and Changeable .)
- Key Binds
- Slash Commands
How it works:
GupPet uses a small database to split the mounts that the player currently has learned. The splitting is based on type (ground, swimming or fly) and speed. When using the auto mount function the add-on will always try to select the fastest possible option, but if you can fly in an area the add-on select the flyable over the ground type. If your are swimming the aquatic will be selected above the other two types.
For the critters GupPet uses a small database to detect if a critter uses a type of reagent to call it, for some types you need a snowball. The add-on detects if you have that reagent with you and if not, the add-on won't select that type of critter.
Language support:
The languages that are currently supported are English , German ( Tnx to Farook at Curse) , Spanish ( Tnx to StiviS at Curse ) and Russian ( Tnx to StingerSoft at WowInterface.) Also esMX ( Tnx to cancerberus at Curse ). With the none-supported languages the zone detection is not supported and therefore forces the two mount action buttons.
Commands
The start command is '/GP' or '/GupPet'. ( The mounting slash commands don't support abilities with spells )
Known Issues:
- If you or a patch clears your WoW Cache folder, you may experience an invalid list of mounts or a failure to select some mounts for your first login after the cache was cleared. Try logging out and then back in.
LOOOOVE this addon. In my opinion it's the best mount/companion manager out there. But is anyone else having issues with it remembering your companion selections? Any idea how I can edit/update the .lua files so my choices persist?
Just got this error (only in Nazjatar) over and over again out of a sudden :(
12x GupPet\GupP_Mount.lua:6: attempt to index a nil value
[string "@GupPet\GupP_Mount.lua"]:6: in function <GupPet\GupP_Mount.lua:1>
[string "@GupPet\GupP_Mount.lua"]:41: in function `GupPet_CanFly'
[string "@GupPet\GupPet_Interface_IngameButtons.lua"]:164: in function `GupPet_Interface_BuildMacro'
[string "*:OnLoad"]:1: in function <[string "*:OnLoad"]:1>
Locals:
(*temporary) = nil
(*temporary) = 1355
(*temporary) = 2
(*temporary) = nil
(*temporary) = nil
(*temporary) = "attempt to index a nil value"
In reply to Niyuni:
i remember when Nazj launched I had some problems with they stopped once i edited the lua as per previous comments and also made a separate ground/flying macro
I've made some mods to it to make it work with BfA flying and a few other BfA special cases - still doesn't work in island adventures or whatever they are (I've never done them myself). Still, it's better than a sharp poke with a stick.
You can download it from here, as long as you know how to manually install addons:
https://bitbucket.org/grimmlabs/guppet/src/master/
Such a sad thing that this awesome add-on never got that little update to work for flying in BFA and Nazjatar :-(
Any idea why the creator has abandonned it ? Will he come back for Shadowlands ?
51x GupPet\GupP_Mount.lua:240: attempt to index field 'ResummonFrame' (a nil value)
GupPet\GupP_Mount.lua:240: in function `GupPet_Mounten'
GupPet\GupP_Mount.lua:221: in function `GupPet_PreMounten'
GupPet\GupP_Mount.lua:170: in function `GupPet_AutoFly'
GupPet\GupPet-2.104.lua:39: in function `?'
FrameXML\ChatFrame.lua:4721: in function <FrameXML\ChatFrame.lua:4668>
[C]: in function `ChatEdit_ParseText'
FrameXML\ChatFrame.lua:4382: in function <FrameXML\ChatFrame.lua:4381>
[C]: in function `ChatEdit_SendText'
FrameXML\ChatFrame.lua:2895: in function <FrameXML\ChatFrame.lua:2888>
[C]: in function `RunMacro'
FrameXML\SecureTemplates.lua:436: in function `handler'
FrameXML\SecureTemplates.lua:632: in function <FrameXML\SecureTemplates.lua:580>
[C]: ?
FrameXML\SecureHandlers.lua:266: in function <FrameXML\SecureHandlers.lua:263>
[C]: ?
FrameXML\SecureHandlers.lua:296: in function <FrameXML\SecureHandlers.lua:279>
(tail call): ?
Locals:
MountType = "Fly"
Location = "Global World"
(*temporary) = nil
(*temporary) = nil
(*temporary) = nil
(*temporary) = nil
(*temporary) = nil
(*temporary) = nil
(*temporary) = nil
(*temporary) = nil
(*temporary) = nil
(*temporary) = nil
(*temporary) = nil
(*temporary) = "attempt to index field 'ResummonFrame' (a nil value)"
Fix from fishermansfriendmenthol is working for me until the addon is updated
No idea what is different in Nazjatar, but mapinf is nil and the routine exits with false. Maybe a bug in the blizzard api.
Replace
with
PS: Found the reason.
/run print(C_Map.GetMapInfo(C_Map.GetMapInfo(1355).parentMapID).mapType)
Shows 1. Which is Enum.UIMapType.World. Nazjatar is a zone without continent.
I really hope Guppet gets updated. In the meantime, a simple workaround in Nazjatar is just using the ground mount or fly mount only binds rather than the smart mount. Yes, it can be a slight inconvenience but it's not so bad in the overall scheme as half the reason I love this addon is the pet feature. Sorry, I'm totally LUA clueless and the fixes below were all greek to me.
For the addon to work in Nazjatar, I edited the "GupP_Mount_CanFlyThisContinent()" function in the GupP_mount.lua file:
local function GupP_Mount_CanFlyThisContinent()
-- To avoid errors
if InCombatLockdown() then return false end
local continent
if C_Map.GetBestMapForUnit("player") == 1355 then
continent = "Nazjatar"
else
continent = MapUtil.GetMapParentInfo(C_Map.GetBestMapForUnit("player"), Enum.UIMapType.Continent).mapID
end
if continent == 875 or continent == 876 or continent == "Nazjatar" then -- BfA, and also Nazjatar
if not IsSpellKnown(278833) then -- BfA Pathfinder
return false
end
elseif continent == 619 then -- Broken Isles
if not IsSpellKnown(233368) then -- Broken Isles Pathfinder
return false
end
elseif continent == 572 then -- Draenor
if not IsSpellKnown(191645) then -- Draenor Pathfinder
return false
end
elseif continent == 12 -- Kalimdor
or continent == 13 -- Eastern Kingdom
or continent == 101 -- Outland
or continent == 424 -- Pandaria
or continent == 113 -- Northrend
or continent == 948 then -- Maelstrom (Deepholm)
if not select(13,GetAchievementInfo(890)) then -- Expert Riding
return false
end
else
return true
end
end
I don't have flying yet but if anyone who does and tests this and it doesn't work, please let me know. :)
In reply to fishermansfriendmenthol:
Hi - I swapped out the function you mentioned with your code and got a couple of errors. Thanks for trying to fix this. I hope this is useful in finding a fix.
Firstly, on entering the game:
Date: 2019-07-12 00:09:36
ID: 1
Error occured in: Global
Count: 1
Message: ..\AddOns\GupPet\GupP_Mount.lua line 1:
'=' expected near 'function'
Debug:
Locals:
AddOns:
Swatter, v8.2.6241 (SwimmingSeadragon)
Ace3, v
AdvancedInterfaceOptions, v1.3.0
AllTheThings, v1.9.5b
AngryWorldQuests, v
AskMrRobot, v78
AucAdvanced, v8.2.6295 (SwimmingSeadragon)
AucStatSimple, v8.2.6232 (SwimmingSeadragon)
AucUtilFixAH, v8.2.6235 (SwimmingSeadragon)
BagBrother, v
Bagnon, v8.2.0
BattlePetBreedID, vr199
DBMArchaeology, v
DBMCore, v8.2.3
DBMDefaultSkin, v
DBMStatusBarTimers, v
Excavatinator, v3.0
FishingAce, v0.8.9
GatherMate2, v1.44.2
GryphonsRemover, vv8.2
GuildMemberOnline, v1.8
GupPet, v8.0.1.05
LibSharedMedia30, v3.0-110
SlideBar, v8.2.6273 (SwimmingSeadragon)
Stubby, v8.2.6274 (SwimmingSeadragon)
BlizRuntimeLib_enUS v8.2.0.80200 <none>
(ck=2d0)
and
Date: 2019-07-12 00:13:14
ID: 1
Error occured in: Global
Count: 1
Message: ..\AddOns\GupPet\GupP_Mount.lua line 1:
'=' expected near 'function'
Debug:
Locals:
AddOns:
Swatter, v8.2.6241 (SwimmingSeadragon)
Ace3, v
AdvancedInterfaceOptions, v1.3.0
AllTheThings, v1.9.5b
AngryWorldQuests, v
AskMrRobot, v78
AucAdvanced, v8.2.6295 (SwimmingSeadragon)
AucStatSimple, v8.2.6232 (SwimmingSeadragon)
AucUtilFixAH, v8.2.6235 (SwimmingSeadragon)
BagBrother, v
Bagnon, v8.2.0
BattlePetBreedID, vr199
DBMArchaeology, v
DBMCore, v8.2.3
DBMDefaultSkin, v
DBMStatusBarTimers, v
Excavatinator, v3.0
FishingAce, v0.8.9
GatherMate2, v1.44.2
GryphonsRemover, vv8.2
GuildMemberOnline, v1.8
GupPet, v8.0.1.05
LibSharedMedia30, v3.0-110
SlideBar, v8.2.6273 (SwimmingSeadragon)
Stubby, v8.2.6274 (SwimmingSeadragon)
BlizRuntimeLib_enUS v8.2.0.80200 <none>
(ck=2d0)
On trying to auto call a mount
Date: 2019-07-12 00:14:14
ID: 3
Error occured in: Global
Count: 1
Message: [string "GupPet_AutoMounten()"] line 1:
attempt to call global 'GupPet_AutoMounten' (a nil value)
Debug:
[string "GupPet_AutoMounten()"]:1: in main chunk
[C]: RunScript()
..\FrameXML\ChatFrame.lua:2171: ?()
..\FrameXML\ChatFrame.lua:4836:
..\FrameXML\ChatFrame.lua:4783
[C]: ChatEdit_ParseText()
..\FrameXML\ChatFrame.lua:4497:
..\FrameXML\ChatFrame.lua:4496
[C]: ChatEdit_SendText()
..\FrameXML\ChatFrame.lua:2991:
..\FrameXML\ChatFrame.lua:2984
[C]: RunMacroText()
..\FrameXML\SecureTemplates.lua:441: handler()
..\FrameXML\SecureTemplates.lua:632:
..\FrameXML\SecureTemplates.lua:580
Locals:
(*temporary) = nil
(*temporary) = "attempt to call global 'GupPet_AutoMounten' (a nil value)"
AddOns:
Swatter, v8.2.6241 (SwimmingSeadragon)
Ace3, v
AdvancedInterfaceOptions, v1.3.0
AllTheThings, v1.9.5b
AngryWorldQuests, v
AskMrRobot, v78
AucAdvanced, v8.2.6295 (SwimmingSeadragon)
AucStatSimple, v8.2.6232 (SwimmingSeadragon)
AucUtilFixAH, v8.2.6235 (SwimmingSeadragon)
BagBrother, v
Bagnon, v8.2.0
BattlePetBreedID, vr199
DBMArchaeology, v
DBMCore, v8.2.3
DBMDefaultSkin, v
DBMStatusBarTimers, v
Excavatinator, v3.0
FishingAce, v0.8.9
GatherMate2, v1.44.2
GryphonsRemover, vv8.2
GuildMemberOnline, v1.8
GupPet, v8.0.1.05
HandyNotes, vv1.5.5
HandyNotesDungeonLocations, v1.46
HandyNotesSuramarTelemancy, v1
HandyNotesWarfrontRares, v0.5.0
IceHUD, v1.11.11
Immersion, v1.1.0
Informant, v8.2.6246 (SwimmingSeadragon)
LargerMacroIconSelection, vv1.4.2
LibSharedMedia30, v3.0-110
MageNuggets, v6.1.1
MageNuggetsCooldownMonitor, v6.0.2
MageNuggetsStatMonitor, v6.0.2
Mapster, v1.8.3
MinimalArchaeology, v8.2.0
Omen, v3.2.2
oRA3, vv8.2.1
Outfitter, v5.22.1
ParagonReputation, v1.24
Pawn, v2.2.45
PetCaught, v1.1
PetTracker, v8.1.2
PetTrackerConfig, v8.1.2
Postal, vv3.6.0
Prat30, vr1195
Prat30Libraries, v
RareScanner, v5.5.2
Recount, vv8.2.0a
Rematch, v4.9.10
Routes, vv1.6.3
SellTrash, v
Skillet, v4.08
SlideBar, v8.2.6273 (SwimmingSeadragon)
Stubby, v8.2.6274 (SwimmingSeadragon)
TomTom, vv80200-1.0.3
TransmogTokens, v1.4.9
WeakAuras, v2.13.4
WonderRep, v2.1.1
ZPerl, v5.7.7
ZPerlParty, v
ZPerlPartyPet, v
ZPerlPlayer, v
ZPerlPlayerBuffs, v
ZPerlPlayerPet, v
ZPerlRaidFrames, v
ZPerlRaidHelper, v
ZPerlTarget, v
ZPerlTargetTarget, v
BlizRuntimeLib_enUS v8.2.0.80200 <none>
(ck=6b6)
In reply to ianhend:
The only clue I can find points to an error in the first line of the GupP_Mount.lua file. The first line of the file should be "local function GupP_Mount_CanFlyThisContinent()".
I put up the entire contents of my GupP_Mount.lua on paste bin at https://pastebin.com/MdSu7GaC if you'd like to do a compare.
Hope that helps!
In reply to fishermansfriendmenthol:
Thanks. I didn't compare the file, but I did completely replace the file with what you put on Pastebin.
I was in Boralus when I logged in, and no error.
Summoned mount ok in Boralus.
Portaled to Nazjatar - No error
Summoned mount in Nazjatar and it worked.
I guess I must've missed something.
Confirmed that the code on the Pastebin link works. Thanks again for taking a look at my comments.
To fly with "Battle for Azeroth Pathfinder":
replace in GupP_Mount.lua
with
I wrote a huge post about Nazjatar but rather than spreading the pain iI'll just leave this here.
/guppet am doesn't work in Nazjatar, you have to use /guppet agm.
In reply to MachinelfGaming:
In Nazj, use shift+shortcut, it will force use a ground mount. However, now that flying is being unlocked, there are many other problems.
In reply to MachinelfGaming:
Unofficial version of GupPet that works in Naz with ground mounts can be downloaded from here:
https://bitbucket.org/grimmlabs/guppet/src/master/
Flying mounts to come when I can actually spot the data.
In reply to grimmtooth:
A quick fix seems to be to look specifically for the Nazjatar zone:
function GupPet_CanFly()
local zone = C_Map.GetBestMapForUnit("player")
if (zone == 1355) then
if not IsSpellKnown(278833) then
return false
else
return true
end
end
[...]
LUA isn't a language I'm familiar with, so I'm not sure why I couldn't get it to work in GupP_Mount_CanFlyThisContinent() and had to hack it in to GupPet_CanFly() instead, but hopefully it'll help someone.
Obviously IsSpellKnown(278833) has to be added to GupP_Mount_CanFlyThisContinent() to get flying in the other areas, but most of that code is already in place.
In reply to houseofmabel:
Yep, which is exactly what I did in my "local" mods to the addon. Oddly, Mechagon doesn't have a problem. Yet another half-finished, half-tested addition to the Blizz API. Five bucks says they never fix it.
In reply to houseofmabel:
Thanks, I've made changes in this vein and we'll see how they work. I don't have pathfinding yet but the changes make sense so go grab
https://bitbucket.org/grimmlabs/guppet/src/master/
and give it a whirl if you like.
I'll get caught up eventually :)