Afficher température capter

system · April 7, 2014, 7:38am

Bonjour
J’essaye de creer un programme permettant de retranscrire la température et la luminosité capté par un capteur de température LM35 et un capteur de luminosité BH1750 en temps réel sur un écran arLCD ( écran tacticle + carte arduino). Grace au programme que j’ai actuellement, j’arrive à retranscrire seulement la température. J’aimerais savoir ce qu’il ne va pas dans mon programme en ce qui concerne la température.

Voici mon programme:

#include <ezLCDLib.h>
#include <Wire.h> 
#include <math.h> 

ezLCD3 lcd;

int xPos = 190;
int y1Pos = 30; 
int y2Pos = 95; 
int x1Pos = 110;
int y3Pos = 190;
int width = 100; 
int height = 40;
int radius = 10;
int alignment = 0;
int option = 1;
int touch=0;  
int xPos1 = 90;
int yPos2 = 100;
int yPos1 = 35;
int width1 = 60;
int option1 = 14; 
int digits = 3; 
int dp = 0;
int x2Pos = 30;
int y4Pos = 190;
int xpo = 10; 
int ypo = 65;
int width2 = 45; 
int height2 = 90;
int resolution = 1;
int maxi = 23;
int initial = 0;
int option2 = 3; 
int xpo1 = 265;  
int max1 = 59; 
int x3Pos = 115;
int y5Pos = 50; 
int y6Pos = 100;
int xPos6 = 70; 
int yPos6 = 70;
int width3 = 80;
int height3 = 80;
int digits1 = 3;
int dp1 = 0;
int xPos7 = 170;
int tempPin = 0;
int tempPin2 = 1;
int dat;
int val2;
int val3;
int val;
int BH1750address = 0x23;
byte buff[2]; 


void setup()
{ Wire.begin();
  accueil();
}

 
void loop(){ 
  val2 = analogRead(tempPin);
  dat= (500 * val2) / 1024;
  
  void luminosite();
  val3 = analogRead(val);
  
  
  touch=lcd.wstack(0);
  if (touch==1) temp();
  if (touch==2) eclairage();
  if (touch==3) heure();
  if (touch==6) accueil();
  if (touch==1) R();
  if (touch==2) R();
  if (touch==3) R();
  if (touch==1) V();
  if (touch==2) V();
  if (touch==9) midi(); 
  if (touch==10) soir(); 
  if (touch==11) heure(); 
  lcd.digitalMeter( 4, xPos1, yPos1, width1, height1, option1, dat, digits, dp, 4); 
  lcd.theme( 4, 155, 152, 3, 130, 0, 0, 1, 147, 153, 1 );
  lcd.digitalMeter( 5, xPos1, yPos2, width1, height1, option1, val3, digits, dp, 5);
  lcd.theme( 5, 155, 152, 3, 130, 0, 0, 1, 147, 153, 1 );
  lcd.wvalue(4, dat);
  lcd.wvalue(5, val3);
}

void accueil(void) { 

  lcd.begin( EZM_BAUD_RATE );
  lcd.cls( WHITE, BLACK );
  lcd.font("0");
  lcd.printString("\\[5x\\[5yConsigne de temperature :");
  lcd.fontw( 1, "sans24" );
  lcd.theme( 1, 0, 3, 0, 2, 3, 2, 2, 2, 1, 1 );
  lcd.string( 1, "Modifier" );
  lcd.button( 1, xPos, y1Pos, width, height, option, alignment, radius, 1, 1 ); 
  lcd.printString("\\[5x\\[70yConsigne d'eclairage :");
  lcd.theme( 2, 0, 3, 0, 2, 3, 2, 2, 2, 1, 1 );
  lcd.string( 2, "Modifier" ); 
  lcd.button( 2, xPos, y2Pos, width, height, option, alignment, radius, 2, 2 );
  lcd.printString("\\[100x\\[160yReglage horaire");
  lcd.theme( 3, 0, 3, 0, 2, 3, 2, 2, 2, 1, 1 );
  lcd.string( 3, "Regler" ); 
  lcd.button( 3, x1Pos, y3Pos, width, height, option, alignment, radius, 3, 3 );
  lcd.printString("\\[5x\\[40yActuelle :");
  lcd.printString("\\[5x\\[105yActuelle :");
}


void luminosite(void)
{
  int i;
  uint16_t val=0; 
  BH1750_Init(BH1750address);
  //delay(200);

  if(2==BH1750_Read(BH1750address))
  {
    val=((buff[0]<<8)|buff[1])/1.2;
  }
    delay(150);
 }
 
 int BH1750_Read(int address)
{
  int i=0;
  Wire.beginTransmission(address);
  Wire.requestFrom(address, 2);
  while(Wire.available())
  {
    buff[i] = Wire.read();
    i++; //incrémentation
  }
  Wire.endTransmission();
  return i; //retour à "int i" 
}
 
void BH1750_Init(int address) 
{
  Wire.beginTransmission(address);
  Wire.write(0x10);
  Wire.endTransmission();
}

Je serais vraiment reconnaissant à la personne m’aidant.

haifger · April 7, 2014, 7:57am

Salut,

je n’ai pas tout regardé en détail, mais cette ligne n’est pas logique :

val3 = analogRead(val);

Si j’ai bien compris, à la fin de la fonction luminosite() (qui est appelée juste avant cette ligne), la valeur de la luminosité ambiante est contenue dans la variable globale val.
Supprime donc cette ligne de code et remplace pour l’affichage la valeur val3 par val :

lcd.wvalue(5, val);

Ça devrait être un peu mieux, même si il y a peut-être d’autres problèmes que je n’ai pas vus…

Christian_R · April 7, 2014, 8:35am

L'usage de Serial.print() permet de vérifier sur l'écran du PC la valeur que prennent les variables.

Attention au int tempPin = 0;
les pin0 et pin1 servant aussi au port série.

B_tto · April 7, 2014, 8:57am

Christian_R:
Attention au int tempPin = 0;
les pin0 et pin1 servant aussi au port série.

Sauf si tempPin est utilisé pour une lecture analogique

68tjs · April 7, 2014, 9:30am

Christian_R:
L’usage de Serial.print() permet de vérifier sur l’écran du PC la valeur que prennent les variables.

Tout à fait.
C’est même le minimum que tu dois faire avant d’appeler à l’aide.

Autre point :
Pourquoi avoir créer un nouveau post sur le même sujet 7 jours seulement après le premier?
Il fallait continuer sur l’ancien en ajoutant :
“J’ai une autre question :…”

fdufnews · April 7, 2014, 10:33am

haifger:
Salut,

je n’ai pas tout regardé en détail, mais cette ligne n’est pas logique :
val3 = analogRead(val);
Si j’ai bien compris, à la fin de la fonction luminosite() (qui est appelée juste avant cette ligne), la valeur de la luminosité ambiante est contenue dans la variable globale val.

val est une variable locale à luminosite().

void luminosite(void)
{
  int i;
  uint16_t val=0;   <----------

Donc les 2 variables sont distinctes.

Si on suit la logique dans laquelle ce programme est écrit, il faudrait supprimer la ligne que j’ai indiqué ci-dessus.
Si on voulait coder proprement il vaudrait mieux supprimer la variable globale et faire retourner la mesure à la fonction luminosité() mais on en est pas encore à ce niveau là.

Il y a toujours une erreur de type de variable sur cette ligne:

  dat= (500 * val2) / 1024;

Il me semble que cela avait déjà été évoqué dans un autre post.
val2 peut retourner une valeur comprise entre 0 et 1023 donc le produit par 500 dépasse largement la capacité d’un int.
Donc:

  dat=(int)( (500L * (long)val2) / 1024);

haifger · April 7, 2014, 11:19am

fdufnews:
val est une variable locale à luminosite().
void luminosite(void)

{
int i;
uint16_t val=0; <----------



Donc les 2 variables sont distinctes.

Ouch ! Effectivement j’étais passé à côté de ça. Les variables locales qui masquent les variables globales, c’est vraiment le mal absolu.

fdufnews · April 7, 2014, 12:41pm

haifger:
Ouch ! Effectivement j'étais passé à côté de ça. Les variables locales qui masquent les variables globales, c'est vraiment le mal absolu.

system · April 8, 2014, 11:30am

Je viens de modifer mon programme en modifiant cette ligne

void luminosite(void)
{
  int i;
  uint16_t val=0;   <----------

Mais ca n’apporte pas de changement, mon écran m’indique toujours une luminosité de 58/59 sans changer.
Quand à la température, elle est bien retranscrit avec ce qui est déjà codé.

infobarquee · April 8, 2014, 12:07pm

bonjour,
moi je trouve de grosses différences avec l’exemple de dfrobot pour le bh1750
http://www.dfrobot.com/index.php?route=product/product&product_id=531#.U0PnKaaUcWM

/*
 Sample code for the BH1750 Light sensor
 Version 0.1
 website:www.DFRobot.com
 
 Connection:
 
 VCC-5v
 GND-GND
 SCL-SCL(analog pin 5)
 SDA-SDA(analog pin 4)
 ADD-NC
 */


#include <Wire.h> //BH1750 IIC Mode 
#include <math.h> 
int BH1750address = 0x23; //setting i2c address

byte buff[2];
void setup()
{
  Wire.begin();
  Serial.begin(57600);//init Serail band rate
}

void loop()
{
  int i;
  uint16_t val=0;
  BH1750_Init(BH1750address);
  delay(200);

  if(2==BH1750_Read(BH1750address))
  {
    val=((buff[0]<<8)|buff[1])/1.2;
    Serial.print(val,DEC);     
    Serial.println("[lx]"); 
  }
  delay(150);
}

int BH1750_Read(int address) //
{
  int i=0;
  Wire.beginTransmission(address);
  Wire.requestFrom(address, 2);
  while(Wire.available()) //
  {
    buff[i] = Wire.receive();  // receive one byte AU LIEU DE     buff[i] = Wire.read();
    i++;
  }
  Wire.endTransmission();  
  return i;
}

void BH1750_Init(int address) 
{
  Wire.beginTransmission(address);
  Wire.send(0x10);//1lx reolution 120ms AU LIEU DE  Wire.write(0x10);
  Wire.endTransmission();
}

haifger · April 8, 2014, 12:12pm

Yagura:
Je viens de modifer mon programme en modifiant cette ligne
void luminosite(void)

{
int i;
uint16_t val=0; <----------



Mais ca n'apporte pas de changement, mon écran m'indique toujours une luminosité de 58/59 sans changer.

Si c’est la seule modification que tu as faite, tu n’as manifestement pas lu l’ensemble des réponses qui t’ont été apportées…

system · April 8, 2014, 12:42pm

J'ai pris en compte toutes les suggestion que l'on m'a fait mais je les avait déjà essayait pour la plupart, j'ai donc essayer la seule que je n'avais pas tester mais aucune ne marche.

Il semblerait que le problème viennt de la carte arduino de mon écran tactil qui ne s'accorde pas avec le capteur de température alors qu'une carte "basique" arrive à le faire fonctionner.

Christian_R · April 8, 2014, 2:55pm

met ton code corrigé, et les erreurs qui s'affichent.

fdufnews · April 8, 2014, 3:09pm

Yagura:
Il semblerait que le problème viennt de la carte arduino de mon écran tactil qui ne s'accorde pas avec le capteur de température alors qu'une carte "basique" arrive à le faire fonctionner.

C'est bien connu, les mauvais ouvriers ont toujours de mauvais outils.

system · May 6, 2021, 2:31pm