While I fully endorse and appreciate removing “padding” from the encounters where it’s relevant (such as Botanist, Spellblade and Trilliax to name the three main contenders), I have one major issue with it:
It really fucking impairs my ability to see who’s fucking up
You’re removing the damage from all logs, including wipes. This includes casts on those “excluded” targets, as well as damage and debuffs. I literally have no way to check post-wipe if my raiders are actually hitting the correct targets, or if they’re “padding” for no reason by throwing in AOE spells because it looks shiny.
Likewise, I can’t check if my raiders are doing it when they should - an example being Botanist, where putting up vampiric touch on the two “extra” adds is a gain due to the insanity gain when VT ticks + selfhealing / absorb from VT, and never having to recast it (due to void bolt + 4 set) in the rest of the fight. Another being Spellblade, where a single havoc from a destro lock is (As far as I know, anyway) worth casting on the fel soul to get 2x immolates up on it for ember generation.
Another example where this is dreadful is the Fel Soul on Spellblade. You’re going to have guilds with brewmasters that will attempt to tank the soul with the statue to bypass the fel lash mechanic in the Arcane phase - but that requires a very strict “no hits at all so you don’t grab aggro by accident”-approach, and I can’t actually check if a raider hit it by accident (such as volley hitting it as it spawns, AOE trinket proc being in range of it etc).
In essence; By removing everything about these mobs that have no bearing on the DPS rankings, you’ve neutered my ability to analyze logs and actually figure out exactly what is going on during a fight / wipe.
Solution:
Allow us to see all the data of the logs as we used to. You already built in a functionality for removing “padding” with your “weighted damage”-option that you’ve really not touched on yet (IIRC it was supposed to give bigger numbers to a high priority target, so for example, damage done to fel lashers would count double because they’re far more important than tunneling the boss - this never happened as far as I can tell, though).
This is really what you’ve essentially already made default - hitting a scrubber has 0% impact on you getting a kill DPS-wise, so it’s weight would be set to “0”. Turn weighted damage on by default and use it for rankings if you must, but please give me back my full overview :(.