I'm puzzled why the optimizer constraints are per-model.
Tanks share a common set of desired constraints (uncritability at a minimum, %avoidance, %mitigation), casters share another (sp, regen), and melee/hunters another (AP), plus some stats everyone love: hit, crit. Seems odd that some models have these constraints and others don't.
Why not have every model have every constraint? It may not make any sense to have a Warlock be uncrittable, or a Prot Warrior to have spellpower>2000, but it seems like the code to check these could be shared across all models.
Then I wouldn't have to request an update to the Tank DK optimizer constraints, because there's no uncrittable option.
(or +hit, or expertise, or %mitigation, or anything else)