Just a nit perhaps. The diagram shows pin 2 and the code uses pin 1. (Not sure pin 1 or 2 are a good choice anyway.)