Dual BJT Constant Current Driver

Yeah thats my bad sorry we are using a TIP31CG NPN. I've done it for previous low power LED applications before and its worked completely fine. The whole circuit was done in the same manner with instead of a 3.3 there's a PWM signal to dim the lights. So im just a bit confused of why it wouldn't work for here as well.