I'm not combat, but I agree that Agility is the way to go. You got to realize that 1 Attack power= Equals 1 dps. However 1 Agility =1 Attack power and some Crit and some Armor. in the end one Agility is worth about 2 times one Attack power ( other forums have much more detailed explanations of how this woks out.) After Agility, I've heard that Combat rouges need to stack Armor Pen. Though I have no real world knowledge of that. you appear to have good EXpertise, it is capped has a cap of 26.
I also agree with dumping Herb. JC allows the use of the dragon Eye Gems x3 which are major boost to any DPS class.
The enchant on your shoulders is a PVP enchant, if your raiding you should consider changing that.
Again I've been assassination spec for long time so cant offer much info on your talent build but it looks pretty close to standard.