[REC] Blood rotation should consume Reaper of Souls procs (Deathbringer)

Before You Begin

[X] I confirm that I have downloaded the latest version of the addon.
[X] I am not playing on a private server.
[X] I checked for an existing, open ticket for this issue and was not able to find one.
[X] I edited the title of this issue (above) so that it describes the issue I am reporting.
[X] I am reporting an issue with the default priority included with the specialization (imported or edited priorities are not supported).

Spec

DK - Blood

Describe the Issue

When using the default profile for Blood, it does not appear that Soul Reaper is recommended in the rotation after Reaper's Mark is applied to a target.

Here are the details of the hero talent:

Here's a screenshot of the recommended rotation before Reaper's Mark is used:

Notice that after Reaper's Mark is applied to the enemy, Soul Reaper is not recommended at all in the rotation:

I have verified that this is not user error by identifying the buff on my character after Reaper's Mark is applied to an enemy:

I've also verified that following the recommended BDK rotation results in a DPS loss without this, as one would expect.

Following the rotation resulted in 489,871.0 DPS on a raid target dummy.

Applying Soul Reaper in the rotation per this recommendation resulted in 529,580.1 DPS on the same dummy.

Therefore, I am recommending that the Deathbringer rotation for blood death knights follow this type of pattern if possible:

flowchart TD
    A[Reaper's Mark cast]
    B[Use Soul Reaper]
    C[Continue with rotation]

    A --> B --> C

There may need to be a change added when Exterminate procs are used as well, as Reaper's Mark has a chance to be applied to the target whenever Exterminate is cast. In that case, we would expect the following:

flowchart TD
    A[Cast Exterminate]
    B{Reaper's Mark applied?}
    C((Yes))
    D((No))
    E[Cast Soul Reaper]
    F[Continue rotation]

    A --> B --> C --> E --> F
              B --> D --> F

How to Reproduce

1.) Log into the game as a level 80 blood death knight. 2.) Choose the Deathbringer hero talent tree. 3.) Choose the Reaper of Souls hero talent. 4.) Find any target. 5.) Cast Reaper's Mark on it. 6.) You will see Soul Reaper is not properly displayed in the rotation recogizing the hero talent's effect.

Snapshot (Link)

https://pastebin.com/chBqLiU9

Raidbots Sim Report (Link)

https://www.raidbots.com/simbot/report/qdLkJr7DHRiRmf7jhScg3J

Additional Information

Looking through the sample ability log, it looks like Raidbots is not properly taking advantage of this talent either in the sims, leading an underestimate in potential player output.

(501,835 DPS vs 529.580.1 DPS over the same approximate 3 minute interval)

Contact Information

No response

Hekili / hekili