wowsims / cata

World of Warcraft Cataclysm Classic simulations.
https://wowsims.github.io/cata/
MIT License
20 stars 44 forks source link

Implement t12 procs #803

Open zoristaken opened 2 weeks ago

zoristaken commented 2 weeks ago

1. Raid items

2. Avengers of Hyjal and Quest Rewards

There seems to be heroic versions of the avengers of hyjal items in the database, but those were not available back in the day, just like the heroic trash loot from tier 11 raids.

Brewfest

Do we even bother implementing the tank's brewmaiden on use? bubblier and bitterer

TBD

The legendary staff will require more thought than any of the other items to implement. There needs to be a discussion if we even bother implementing it following simc and old data for now, or we wait for more information.

zoristaken commented 2 weeks ago

Issue #806 will help implementing the Matrix Restabilizer and Apparatus of Khaz'goroth trinkets

zoristaken commented 1 week ago

Some additional comments, the trinkets that have not been implemented yet are either healing trinkets that I don't know how to go about, or have some sort of mechanic that I didnt investigate if the core allows it right now (example: stacking mana cost reduction during aura on Jaws of Defeat).

I also have some TODO comments that need addressing. Need to double check that I didn't mess up any numbers but otherwise would like a review of what's already implemented

hillerstorm commented 5 days ago

Tank trinkets should be pretty straight forward to implement, at least by someone with a bit of absorb-knowledge (e.g. @rosenrusinov 😅).

Jaws of Defeat should be possible to implement with a dynamic spell mod using SpellMod_PowerCost_Flat right? Just reset the FloatValue on gain and add 110 to the current each stack?

zoristaken commented 1 day ago

Tank trinkets should be pretty straight forward to implement, at least by someone with a bit of absorb-knowledge (e.g. @rosenrusinov 😅).

Jaws of Defeat should be possible to implement with a dynamic spell mod using SpellMod_PowerCost_Flat right? Just reset the FloatValue on gain and add 110 to the current each stack?

I might leave tank/healing trinkets not implemented by now and we can just focus and making sure the ones I have are correct. If that's the case I will just open an issue to track what's left to implement