Dark pact of thrarthus doesn't take into account if you use traps/mines or not

Check version

• I'm running the latest version of Path of Building and I've verified this by checking the changelog

Check for duplicates

• I've checked for duplicate open and closed issues by using the search function of the issue tracker

Check for support

• I've checked that the calculation is supposed to be supported. If it isn't please open a feature request instead (Red text is a feature request).

What platform are you running Path of Building on?

Windows

What is the value from the calculation in-game?

I don't have the in game calculations, but i checked the code, and that is calculating ruin stacks, which do not work when you have traps (max is 1 on the trap) this causes the average to appear as if you're dealing 100 mill dps while in fact you'd do 1 million

What is the value from the calculation in Path of Building?

i checked the code, and that is calculating ruin stacks, which do not work when you have traps (max is 1 on the trap) this causes the average to appear as if you're dealing 100 mill dps while in fact you'd do 1 million

How to reproduce the issue

easiest value to see this change with is the quality setting on thrarthus, this adds damage eventhough it only allows you to stack ruin more quickly. nothing else.
I believe adding another setting along with average and max ruin. would do it
Part 1 = No Ruin
Part 2 = Average
Part 3 = Max Ruin

PoB for PoE1 build code

https://pobb.in/d8v0vUo1Z8Dz

Screenshots

Reproduce the issue with the linked PoB for PoE1 build code, then trace the ruin-stack calculation and its handling of traps and mines. Check how the Thrarthus quality setting affects the displayed damage; done means trap and mine damage no longer assumes unrestricted ruin stacks, with the requested ruin-state choices represented accurately.