Help with adding to my code

toxsickcity

Using someone else code is alaways tricky. You spend so much time figuring out what does waht.

Firstly

Do you have the 4k7 pullup resistor added between 5v and the data pin of the two sensors.?

Without this you won't be able to talk with the sensor, and therefore no reading.

If you open the Serial Terminal do you get either of the messages in Line 118 or 120?

I haven't tried talking to two sensors but you don't need to discover them every time you want to read the temperature. Line 112, 158

This is a sketch I did for controlling two relays using a single sensor (one was for heating the other cooling)

/*
  Temp Controlled Relay

 DS18B20 Temperature chip on pin 2
 4k7 pullup between pin 2 and +5v.

 Pin assignments
 Pin 0  Rx
 Pin 1  Tx
 Pin 2  Temp sensor DS18B20
 Pin 7  Relay 1 output (Low to make it turn ON)
 Pin 8  Relay 2 output (Low to make it turn ON)


 Created 15 Nov 2012
 by Mark Beckett

    
 */

// include the library code:

#include <OneWire.h>

// DS18B20 Temperature chip i/o
OneWire ds(2);                     // TempSensor on pin 2
byte i;
byte present = 0;
byte data[12];
byte addr[8];
word TReading;
byte HighByte, LowByte;
int SignBit, Tc_dec, Tc_100, Whole, Fract;

//inputs
const int TempSensor = 2;               // Tempsensor name

//Outputs
const int Relay1 = 7;               // Relay 1 Output
const int Relay2 = 8;               // Relay 2 Output
const int LedPin = 13;              // LED on pin 13

//Settings
int LowTempOn = 14;                   // Low Temperature On setting
int LowTempOff = 24;                  // Low Temperature Off setting
int HighTempOn = 29;                 // High Temperature On setting
int HighTempOff = 25;                // High Temperature Off setting

//-------------------------------------------------------------------------


void setup() 
{
   Serial.begin(57600);//Serial.begin(57600);
   
  //Define the pins
  pinMode(TempSensor, INPUT);
  pinMode(Relay1, OUTPUT);
  digitalWrite (Relay1, HIGH);    //Relay is OFF 
  pinMode(Relay2, OUTPUT);
  digitalWrite (Relay2, HIGH);    //Relay is OFF 
  
  !ds.search(addr);            //search for DS18B20 and identify its serial number
  Serial.println ("'The Shed' magazine Temperature Sensor example");
  Serial.println ("Refer to 2012 Dec/Jan issue for details");

}

void loop() 
{
    Whole = 0;                //reset to a known value
    Fract = 0;                //reset to a known value
    Tc_dec =0;                //reset to a known value
    
    //reset and setup the temp sensor for communicating
        ds.reset();
        ds.select(addr);
        ds.write(0x44,1);        // start a temp reading
        
        delay(1000);             // wait 1 sec for data to be present
        
        present = ds.reset();
        ds.select(addr);    
        ds.write(0xBE);          // Read Scratchpad in the temp sensor
    
      for ( i = 0; i < 9; i++)   // we need 9 bytes
      {
        data[i] = ds.read();     //read the 9 bytes of data for a temperature      
      }
      
    Serial.print("Temp = ");
      
      LowByte = data[0];                       // the first byte is the lowest
      HighByte = data[1];                      // second byte is the highest
      TReading = (HighByte << 8) + LowByte;
      SignBit = TReading & 0x8000;             // test most sig bit
      if (SignBit)                             // If its negative
      {
         TReading = (TReading ^ 0xffff) + 1;   // 2's comp 
      }
        Tc_100 = (6 * TReading) + TReading / 4;    // multiply by (100 * 0.0625) or 6.25

        Whole = Tc_100 / 100;                  // separate off the whole portions
        Fract = Tc_100 % 100;                  // separate off the fraction portions
        
        if (SignBit)                           // If its negative
        {
         Serial.print("-");
        }
       Serial.print (Whole);
       //Serial.print(".");
       // if (Fract < 10)
       // {
       //  Serial.print("0");
       // }
       // Serial.print(Fract);
       Serial.print("\n");
      
      
       //Relays are ON when the ouput pin is LOW
       if (Whole <= LowTempOn)                              //reading is below or equal to the setting
       {
         digitalWrite(Relay1,LOW);
         digitalWrite(LedPin, HIGH);
       }
       if (Whole >= LowTempOff && !digitalRead(Relay1))     //reading is above or equal to the setting and Relay1 is ON
       {
         digitalWrite(Relay1,HIGH);
         digitalWrite(LedPin, LOW);
       }
       if (Whole >= HighTempOn)                              //reading is below or equal to the setting
       {
         digitalWrite(Relay2,LOW);
         digitalWrite(LedPin, HIGH);
       }
       if (Whole <= HighTempOff && !digitalRead(Relay2))    //reading is above or equal to the setting and Relay2 is ON
       {
         digitalWrite(Relay2,HIGH);
         digitalWrite(LedPin, LOW);
       } 
       if (digitalRead(Relay1) && digitalRead(Relay2))      // Flash the LED if either relay is High (Off).
       {
         digitalWrite(LedPin,!digitalRead(LedPin));
       }
}

You can see the temperature on the serial port

Mark

Hi, Thankyou for your reply.

I do have a 4k7 on the sensors.

Currently the arduino with this code does control 2 LED Strips with 2 sensors (each sensor control 1 LED Strip).

I am very happy with that.

I want to go with additional feature using the same 2 sensors to also control the that relay board.

when I get home I will have a go at editing my code.

Lastly, Can you explain to me how to view my arduino in a terminal window.

It never works for me

I read about it, and dont know why it displays nothing!

Lastly, Can you explain to me how to view my arduino in a terminal window.

As long as you have the Serial.begin(57600)

You seem to have it twice Line 60 and 67

and some Serial.print or Serial.println somewhere it should work.

I'd add something like

     Serial.print("k :");

     Serial.println(((rgb[k] *bright[k])/256));

to line 94 and 95

     Serial.print("k2 :");

     Serial.println(((rgb2[k] *bright[k])/256));

to line 100 and 102 (or 104 and 106 as they'll become).

which should show the values that are being sent to the rgb pins.

I simply open the Serial Monitor

It will remember the last setting, so I tend to always use the same (9600) for my sketches as they are generally in debug mode and not timing critical.

Try my sketch and you should get something.

BTW Lines 59 and 66 are trying to read something on the Analogue pins ...not sure what but.

Mark

A Massive thank you for your help,

I was too tired after work last night to fiddle but after today, its the weekend!!

many thanks,

I will report my findings tonight if I have enough energy!

Hey Mark,

I got the outputs working, it reads 5v on my selected pins when the Set thermal goes over, that's for 2 channels working independently.

many thanks for your advise and your messages.

I got another request, if you have time.

I want to push my arduino board even harder and would save me a heap of space in my project if able to add another "Sensor/LED Strip/Relay Output".

To be able to do this I would need to change my system from RGB To fade Blue to Red.

This also means using only 2 not 3 PWM outputs for each LED Strip removing green which is exactly what I wanted from this project.

I never intended to have green show up during the heat changes, imo for my project.

Going from blue to green than to red looks a bit silly!!

I could use some help only if you have time.

I'll have a look, but first thought is you might be running out of PWM pins.

Pins 3,5,6,9,10 and 11 are all you get.

So in theory if you drop the green channel, you could have 3 x 2 channel Red/Blue.

You could set the green component using resistors.

As an alternative, have you considered using neoPixels (WS 2812) devices instead.

One output pin can drive hundreds individually both colour and brightness.

https://www.adafruit.com/?q=neopixel&

These are 60 LED's per meter

https://www.adafruit.com/products/1461

Mark

Hi Mark,

Those LEDs are cool!

I might have to buy some for future experiment.

I would like to stay with my current design tho,

Can you explain what you mean about the possible of keeping my green but using resistors?

I don't want to keep green but am interested in hearing how you would go about keeping green when not using the pwm output?

how you would go about keeping green when not using the pwm output

The PWM alters how much voltage the LED receives. ie at 50% it is approximately half the current.

You can use a digital Output to turn it ON or OFF and use two resistors to approximate the level you would have via a PWM output.

The two resistors form a voltage divider so the 5v outout is dropped to a differen tvoltage depending on the ratio between the two.

You need to be careful that the values aren't too high (or too low).

Do a google search on Voltage divider and there will be online calculators ...

Mark

Oh I understand, but I would have required those digital out pins, and having 3 sets of Blue/Red would mean that I cannot. because the digital pins are being used to switch relays on and where the sensor connects.

I will have the pins as such

6x PWM = LED (Blue1/Red1, Blue2/Red2, Blue3/Red3)

3x Sensor Inputs

3x Relay Trigger outputs.

so I think that is using all the digital pins on the Arduino Board.

Playing with my code, I was able to change a line that said 0000FF00 to 00000000 and it removed green completely,

but it didnt fade from blue to red nicely, it went off in between the 2 colours.

I will leave it with you so see if you can come up with anything.

Just a quick reply

so I think that is using all the digital pins on the Arduino Board.

The Analogue pins can also be used as Digital.

A0 = D14

A1 = D15

A2 = D16

A3 = D17

A4 = D18

A5 = D19

Mark