ebonhold / simc

Simulationcraft engine/GUI
GNU General Public License v3.0
0 stars 0 forks source link

10.0 Support Master list #882

Closed Armadk closed 1 year ago

Armadk commented 2 years ago
    First sketch of the new talent system.

    New player-scope options to specify the talents for the class and
    specialization trees. All options use the same format for the value: a
    '/' delimited list of <talent-name>:<talent-rank>, where <talent-rank>
    can be either a tokenized talent spell name, or a trait node entry id
    and <talent-rank> is a numeric value of the selected talent, including 0
    to indicate the talent is disabled.

    The new player scope options are:
    * class_talents -- sets the player class talent tree to this value
    * class_talents+ -- appends the value to the existing class talent tree
      of this player
    * spec_talents -- sets the player specialization talent tree to this
      value
    * spec_talents+ -- appends the value to the existing specialization
      talent tree of this player

    The option parsing happens sequentially. Talent rank values are allowed
    to be overwritten.

    The following expression changes have also been implemented:
    * talent.<talent-name>.enabled uses the new talent information
    * talent.<talent-name>.rank evaluates to the current rank of the talent

    For class modules, talents are now represented by a new wrapper object
    player_talent_t. For access, new set of player_t::find_talent_spell
    methods have been introduced to "find" talent spells based on player
    state.

    The player_talent_t wrapper object introduces two simple methods to
    access the scaling properties represented by the rank of the talent:
    * player_talent_t::base_value( effect_idx [1..N] ) returns the base
      value of the associated effect of the talent
    * player_talent_t::percent( effect_idx [1..N] ) returns the "percent
      value" of the associated effect of the talent
Armadk commented 2 years ago
Armadk commented 2 years ago

Basic plan

Armadk commented 2 years ago

Things to check as they were marked NYI in simc

Armadk commented 2 years ago

Blood tree Blood Fortification looks like it's intended to be a replacement for Veteran of the third war rank 2

Armadk commented 2 years ago

Unholy tree

Armadk commented 2 years ago

From Navv ok, pushed the changes you should be able to now do just talent->effectN( x ) and all is well in the world as if nothing ever changed

So we can ditch a lot of the spell() -> effectN () calls

Armadk commented 1 year ago

implemented