more work on PID controller