Project Rising Water

#include <Wire.h>  // importing LCD libraries
#include <LiquidCrystal_I2C.h>

LiquidCrystal_I2C lcd(0x27,16,2);  // set the LCD address to 0x27 for a 16 chars and 2 line display

unsigned long previousMillis = 0; // millis needs unsigned log not int
float Red = 0; // output pin 11
float Blue = 0; // output pin 10
float Green = 0; // output pin 9
float bright = 0; // mapped color, sets brightness of led
float colouer = 0; // for picking color, input a3
float chrono = 0; // for picking frequency, input a2
float khrono = 0; // mapped timing, sets the blink rate
float ness = 0; // brightness of the colors
bool flip = true; // Bad variable name squad
// Thanks to paulS for cyberbullying some guy one arduino forums and also giving us a problem fix

void setup() {
  // put your setup code here, to run once:
  Serial.begin(9600);
  int previousMillis = 0;
  // serial begin for debugging purposes
  lcd.init(); // initialize the lcd 
  lcd.init(); // Second init is required
  // Print a message to the LCD.
  lcd.backlight();
    lcd.setCursor(4,0); // printing out the desired text.
    lcd.print("Project:");
    lcd.setCursor(2,1);
    lcd.print("Rising Water");
}

void loop() {
   /* One loop reads and sets each LED brigtness equal to the value read by the potentiometer, the seconds turns the LED off after the interval is met.
     The variable flip is used to track which loop must run next. Millis is used to determine the interval of time before running each loop. */
  chrono = analogRead(A2); //reading the frequency off the potentiometer
  khrono = map(chrono, 0, 1023, 1, 100); // mapping the input into out actual frequency. Second number pair sets high and low for frequency
  khrono = ((khrono / 100) + 5.4);
  colouer = analogRead(A3); //FLIP WILL NOT FLIP FROM TRUE TO FALSE ayay it works now
  Serial.println(khrono);

  
  if ( ((millis() - previousMillis) >= khrono) && (flip == false )) { // Turning the colors off when the interval is met AND the other loop has run through first
    analogWrite(10, 0); //Turning off each color
    analogWrite(11, 0);
    analogWrite(9, 0);
    previousMillis = millis(); // Resetting the interval
    flip = true; // Switching to use the other loop next run through
  }

  if ( ((millis() - previousMillis) >= khrono) && (flip == true)) {
     previousMillis = millis(); // Resetting interval
     

  /*The midpoint of each color range is where the color is the brightest. Overlap allows the colors to blend together.
 Subtraction is used to force the brightness into fitting the values needed.*/
  if (colouer > 111 && colouer < 573){ //Working 
     Blue = colouer-111; // Detecting and displaying blue, interval 111 to 573 on the pot
      if(Blue <= 231){
        bright = Blue; // Setting the LEDs brightness
      }
      if(Blue >= 232){
        bright = 462 - Blue; // accounting for the midpoint so the LED dims.
      }
        ness = map(bright,0,231,0,255); // Setting the brightness to send to the LED
        analogWrite (10,ness); // LED on
  }
  /* Red has its highest brightness set at zero, so it wraps around the circle and allows red to blend with green and blue.
  2 loops were needed to create the desired effect. */
  if (colouer > 793) { // First half of red loop.
    Red = colouer - 793;
    bright = 1024 - Red;
    ness = map(bright, 0, 174, 0, 255);
    analogWrite (11, ness); // Turning the LED on
  }

  if (colouer < 231) { // low end of red loop
    Red = 231 - colouer;
    bright = Red;
    ness = map(bright, 0, 341, 0, 255);
    analogWrite (11, ness); // LED on
  }
  if (colouer > 453 && colouer < 913){ // Setting up green, interval is 453 to 913
     Green = colouer-453; // Brininging the value into the range we need to map. This line starts at zero and goes up.
      if(Green <= 190){
        bright = Green;
      }
      if(Green > 341){ // When green passess the midpoint of its range it begins to go down to zero.
        bright = 913 - Green;
      }
        ness = map(bright,0,213,0,255);
        analogWrite (9,ness); // Turning the LED on
  }
  flip = false;
     }
}