Hey guys, I've been struggling with this for a few days now and I've reached a wall..
Basically I have some neopixels and I want to set them in different modes controlled by a button, using the internal pullup, the button will ground an input when pressed and then will increment the mode by 1, until it reaches 6 where it wil loop back to mode 1 again and so on.. but I'm struggling, I've finally got the arduino to see the button and react but it will not loop back to 1 and just gets stuck at mode 6 and seems to require a 2 second push to make it change.. the more I play with it, the more it laughs at me.. any input would be greatly appreciated, and heres my code 
:
#include <Adafruit_NeoPixel.h>
#define PIN 6
Adafruit_NeoPixel strip = Adafruit_NeoPixel(25, PIN, NEO_GRB + NEO_KHZ800);
void setup() {
pinMode(4, INPUT);
digitalWrite(4, HIGH); // connect internal pull-up
strip.begin();
strip.show(); // Initialize all pixels to 'off'
}
int counter = 1;
void loop() {
if(digitalRead(4) == LOW && counter < 6)
{
counter++;
if(counter == 6){
counter = 1;
}
}
// Some example procedures showing how to display to the pixels:
colorWipe(strip.Color(255, 0, 0), 50); // Red
colorWipe(strip.Color(0, 255, 0), 50); // Green
colorWipe(strip.Color(0, 0, 255), 50); // Blue
// Send a theater pixel chase in...
theaterChase(strip.Color(127, 127, 127), 50); // White
theaterChase(strip.Color(200, 0, 0), 50); // Red
theaterChase(strip.Color(200, 0, 200), 50);
theaterChase(strip.Color( 0, 0, 200), 50); // Blue
rainbow(20);
rainbowCycle(20);
theaterChaseRainbow(30);
}
// Fill the dots one after the other with a color
void colorWipe(uint32_t c, uint8_t wait) {
if (counter == 1) {
for(uint16_t i=0; i<strip.numPixels(); i++) {
strip.setPixelColor(i, c);
strip.show();
delay(wait);
}
}
}
void rainbow(uint8_t wait) {
if (counter == 2) {
uint16_t i, j;
for(j=0; j<256; j++) {
for(i=0; i<strip.numPixels(); i++) {
strip.setPixelColor(i, Wheel((i+j) & 255));
}
strip.show();
delay(wait);
}
}
}
// Slightly different, this makes the rainbow equally distributed throughout
void rainbowCycle(uint8_t wait) {
if (counter == 3) {
uint16_t i, j;
for(j=0; j<256*5; j++) { // 5 cycles of all colors on wheel
for(i=0; i< strip.numPixels(); i++) {
strip.setPixelColor(i, Wheel(((i * 256 / strip.numPixels()) + j) & 255));
}
strip.show();
//delay(wait);
}
}
}
//Theatre-style crawling lights.
void theaterChase(uint32_t c, uint8_t wait) {
if (counter == 4) {
for (int j=0; j<10; j++) { //do 10 cycles of chasing
for (int q=0; q < 3; q++) {
for (int i=0; i < strip.numPixels(); i=i+3) {
strip.setPixelColor(i+q, c); //turn every third pixel on
}
strip.show();
delay(wait);
for (int i=0; i < strip.numPixels(); i=i+3) {
strip.setPixelColor(i+q, 0); //turn every third pixel off
}
}
}
}
}
//Theatre-style crawling lights with rainbow effect
void theaterChaseRainbow(uint8_t wait) {
if (counter == 5) {
for (int j=0; j < 256; j++) { // cycle all 256 colors in the wheel
for (int q=0; q < 3; q++) {
for (int i=0; i < strip.numPixels(); i=i+3) {
strip.setPixelColor(i+q, Wheel( (i+j) % 255)); //turn every third pixel on
}
strip.show();
delay(wait);
delay(wait);
delay(wait);
delay(wait);
for (int i=0; i < strip.numPixels(); i=i+3) {
strip.setPixelColor(i+q, 0); //turn every third pixel off
}
}
}
}
}
// Input a value 0 to 255 to get a color value.
// The colours are a transition r - g - b - back to r.
uint32_t Wheel(byte WheelPos) {
if(WheelPos < 85) {
return strip.Color(WheelPos * 3, 255 - WheelPos * 3, 0);
}
else if(WheelPos < 170) {
WheelPos -= 85;
return strip.Color(255 - WheelPos * 3, 0, WheelPos * 3);
}
else {
WheelPos -= 170;
return strip.Color(0, WheelPos * 3, 255 - WheelPos * 3);
}
}