Ovale Spell Priority

Ovale Spell Priority

6M Downloads

Allow more (all?) classes to have totems

johnnylam88 opened this issue ยท 0 comments

commented

Every spell or item that summons minions has those minions treated as totems by the game. Their information can be retrieved via the Blizzard API function GetTotemInfo(slot). It might be best now to treat each class as a TOTEM_CLASS and track the totem states appropriately from game events.

In addition, the simulationcraft modules need to be updated to translate various pet APL actions into the correct totem Ovale condition, e.g., pet.ghoul.remains<10 becomes TotemRemaining(raise_dead) < 10 for blood and frost death knight specializations.

Lastly, I'm not quite sure yet how to handle properly identifying the totems. The Blizzard API function GetTotemInfo(slot) only returns the texture ID as the feature to uniquely identify the totem that was summoned. That works just fine for almost every spell, but not so well with spells that are replaced by glyphs or other similar modifications and whose texture IDs change as a result. The particular case I'm tracking down involves unholy death knights that have the Summon Gargoyle talent and who have talked to a trainer in Acherus Hold to have that spell summon a Val'kyr instead. In this case, the spell on the action bar does not look any different than before, but the summoned minion has a different totem texture ID, which makes it hard to handle automatically.