Go Down

Topic: wie Variable in if-abfrage ändern? -> neues problem: PulseIn, map und if/else (Read 2880 times) previous topic - next topic

Daniel_N

NEUES PROBLEM WEITER UNTEN!

Hallöschen :)

Gleich am Anfang:
Ich bin Arduino Neuling, also wird mein Code alles andere als perfekt sein  ;)

Mein vorhaben:

Ich will 2 LEDs faden und gleichzeitig 2 LEDs als Doppelblitzer haben.

Da ein Delay ja den ganzen code sozusagen zerstört, hab ich mit Hilfe der tuts und google diesen Code geschrieben, um die LED erstmal blitzen zu lassen (das Doppeln wollte ich später einfügen, wenn das Blitzen funktioniert):
Code: [Select]

// constants won't change. Used here to
// set pin numbers:
const int ledPin1 =  3;      // the number of the LED pin
const int ledPin2 =  5;      // the number of the LED pin

// Variables will change:
int ledState = LOW;             // ledState used to set the LED
long previousMillis = 0;        // will store last time LED was updated

// the follow variables is a long because the time, measured in miliseconds,
// will quickly become a bigger number than can be stored in an int.
int interval = 0;           // interval at which to blink (milliseconds)


#define led1 3
#define led2 5
#define fled1 10 //Led on digital pin 10
#define fled2 11 //Led on digital pin 10

unsigned long btnMillis = 0; //clock value to be stored at the time milli() is called

//Fade a led without delaying the rest of the code
//START
unsigned long fadeMillis = 0;
boolean fadeUp = true;
int fadeValue = 0;
void analogFade()
{
 if (millis() - fadeMillis >= 2)
 {
   fadeMillis = millis();

   if (fadeUp == true)
   {
     if (fadeValue < 255)
     {
       fadeValue++;
     }
     else
     {
       fadeUp = false;
       fadeValue--;
     }
   }
   else
   {
     if (fadeValue > 0)
     {
       fadeValue--;
     }
     else
     {
       fadeUp = true;
       fadeValue++;
     }
   }
   analogWrite(fled1, fadeValue);
   analogWrite(fled2, fadeValue);
 }
}
//STOP

void setup()
{
 pinMode(led1, OUTPUT);
 pinMode(led2, OUTPUT);
   // set the digital pin as output:
 pinMode(ledPin1, OUTPUT);
 pinMode(ledPin2, OUTPUT);  
}

void loop()
{  
 analogFade();

unsigned long currentMillis = millis();

 if(currentMillis - previousMillis > interval) {
   // save the last time you blinked the LED
   previousMillis = currentMillis;  

   // if the LED is off turn it on and vice-versa:
   if (ledState == LOW)
     ledState = HIGH;
   else
     ledState = LOW;

   // set the LED with the ledState of the variable:
   digitalWrite(ledPin1, ledState);
   digitalWrite(ledPin2, ledState);

 if (interval == 100)
    interval = 800;
 else
    interval = 100;
 
 }

}


Das Problem:
er ändert zwar die variable "interval" in 800, aber nicht zurück.
Ich hätte aber gerne folgendes:

LED an
100ms warten
LED aus
800ms warten
...und wieder von vorne

Wie mach ich das?

Danke im voraus.

Gruß,
Daniel

maverick1509

Hallo Daniel,
ich habs nur kurz überflogen, füge mal in der Void setup " Serial.begin(9600);" ein und in der
void loop hinter der if abfrage "Serial.println(interval);" dann wirst du am Monitor
verfolgen können, das sich interval doch immer von 100 nach 800 und zurück ändert.
Das funktioniert auf jeden Fall, der Fehler liegt wohl an anderer Stelle.
Gruß
Bernward

maverick1509

Hallo Daniel,
versuch es mal mit dieser void loop:
Code: [Select]


void loop()

analogFade();
unsigned long currentMillis = millis();


if(currentMillis - previousMillis > interval) {
   // save the last time you blinked the LED
   previousMillis = currentMillis;   

  // if the LED is off turn it on and vice-versa:

  if(interval==100){
    digitalWrite(ledPin1, LOW);
    interval=800;
  }
  else{   
   digitalWrite(ledPin1, HIGH);
   interval=100;
  }
}

}


Gruß
Bernward

uwefed

hallo Daniel_N

Bei mir funktioniert der Code.

Du hast aber einige kleinere Fehler:
Code: [Select]
const int ledPin1 =  3;      // the number of the LED pin
const int ledPin2 =  5;      // the number of the LED pin
...
#define led1 3
#define led2 5


und dann:
 
Code: [Select]
pinMode(led1, OUTPUT);
  pinMode(led2, OUTPUT);
    // set the digital pin as output:
  pinMode(ledPin1, OUTPUT);
  pinMode(ledPin2, OUTPUT); 


Bitte das eine (led1) oder das andere (ledPin1). Beide braucht es nicht.

Code: [Select]
long previousMillis = 0;
Bitte eine Variable unsigned long;

Kontrolliere mal ob Du die LEDs auf den Pins 3,5 10 und 11 hast.

Viele Grüße Uwe


Daniel_N

#4
Jul 12, 2011, 10:18 am Last Edit: Jul 12, 2011, 10:27 am by Daniel_N Reason: 1
Nachdem ich das led-Chaos beseitigt habe (Danke uwe), funktioniert es :)
Hab auch mal den Serial Monitor ausprobniert mir dem " Serial.begin(9600);" und dem "Serial.println(interval);"
Der monitor spuckt mir aber nichts aus. er springt zwar irgendwie rum und der scrollbalken wird kleiner, aber es erscheinen keine zahlen*?*


//EDIT:
Hab dann uach ma eben die if-abfrage erweitert, so dass ich jetzt den doppelblitzer habe. Sicher nicht die beste Lösung, aber es funktioniert:

Aktueller kompletter Code:
Code: [Select]
// Variables will change:
int ledState = LOW;             // ledState used to set the LED
long previousMillis = 0;        // will store last time LED was updated

// the follow variables is a long because the time, measured in miliseconds,
// will quickly become a bigger number than can be stored in an int.
int interval = 0;           // interval at which to blink (milliseconds)


#define led1 3
#define led2 5
#define fled1 10 //Led on digital pin 10
#define fled2 11 //Led on digital pin 11

unsigned long btnMillis = 0; //clock value to be stored at the time milli() is called

//Fade a led without delaying the rest of the code
//START
unsigned long fadeMillis = 0;
boolean fadeUp = true;
int fadeValue = 0;
void analogFade()
{
 if (millis() - fadeMillis >= 2)
 {
   fadeMillis = millis();

   if (fadeUp == true)
   {
     if (fadeValue < 255)
     {
       fadeValue++;
     }
     else
     {
       fadeUp = false;
       fadeValue--;
     }
   }
   else
   {
     if (fadeValue > 0)
     {
       fadeValue--;
     }
     else
     {
       fadeUp = true;
       fadeValue++;
     }
   }
   analogWrite(fled1, fadeValue);
   analogWrite(fled2, fadeValue);
 }
}
//STOP

void setup()
{
 pinMode(led1, OUTPUT);
 pinMode(led2, OUTPUT);
}

void loop()
{  
 analogFade();

unsigned long currentMillis = millis();

 if(currentMillis - previousMillis > interval) {
   // save the last time you blinked the LED
   previousMillis = currentMillis;  

   // if the LED is off turn it on and vice-versa:
   if (ledState == LOW)
     ledState = HIGH;
   else
     ledState = LOW;

   // set the LED with the ledState of the variable:
   digitalWrite(led1, ledState);
   digitalWrite(led2, ledState);

 if (interval == 99)
    interval = 100;
 else if (interval == 100)
    interval = 101;
 else if (interval == 101)
    interval = 800;
 else
    interval = 99;
 }

}


Ich finde, dafür dass ich erst vor 2 Tagen mit dem Arduino angefangen habe ist es schon nicht schlecht.

Das ganze ist übrigens für die Beleuchtung eines ferngesteuerten Autos.

@ Uwe:
3 & 5, sowie 10 & 11 sind jeweils eine LED vorne und hinten links und rechts.


Als nächstes kommt die Schaltung per Fernsteuerung, kann mir da mal jemand ein Beispiel geben, wie ich das Signal auslese?
Habe in den Beispielen spontan nichts gefunden.
Ich weiss dass es 3 Positionen gibt, 1ms, 1,5ms, und 2ms.

Danke erstmal für eure Hilfe :)

uwefed


Hab auch mal den Serial Monitor ausprobniert mir dem " Serial.begin(9600);" und dem "Serial.println(interval);"
Der monitor spuckt mir aber nichts aus. er springt zwar irgendwie rum und der scrollbalken wird kleiner, aber es erscheinen keine zahlen*?*

Bei mir funktionierte es heute Morgen.
Hast Du im Terminal auch 9600 eingestellt (rechts unten)?
Grüße Uwe

Daniel_N

Jetzt hab ichs eingestellt und es funktioniert.
Man sollte sich wohl auch darüber informieren, was man da einträgt und nicht einfach nur tun :smiley-red:

uwefed

Normalerweise steht da 9600 deswegen sagt man es nicht explizit.
Grüße Uwe

Gurkengraeber

Hallo Daniel_N !

Für dein Problem gibts 'ne Lösung: die Interrupts

Der kleinste Arduino hat 2 externe Interrupts. Diese Interrupts können sogar Warte-Schleifen  ( delay ) durchbrechen.
Das heisst, während ein Delay ausgeführt wird,  wird das Programm unterbrochen, und führt etwas anderes aus. Danach kehrt das Programm wieder in den Normal-Modus zurück, und führt das Delay zu Ende.

Die Befehle siehst du hier: http://arduino.cc/en/Reference/AttachInterrupt

Aber sauber sind die nicht implementiert. Ich versuche gerade 2 Tasten-Abfragen über einen Interrupt.

MfG



Daniel_N

Neues Problem:

Erstmal der aktuelle Code:
Code: [Select]
// Variables will change:
int ledState = LOW;             // ledState used to set the LED
long previousMillis = 0;        // will store last time LED was updated

// the follow variables is a long because the time, measured in miliseconds,
// will quickly become a bigger number than can be stored in an int.
int interval = 0;           // interval at which to blink (milliseconds)


#define led1 3
#define led2 5
#define fled1 10 //Led on digital pin 10
#define fled2 11 //Led on digital pin 11

unsigned long btnMillis = 0; //clock value to be stored at the time milli() is called

//Fade a led without delaying the rest of the code
//START
unsigned long fadeMillis = 0;
boolean fadeUp = true;
int fadeValue = 0;
void analogFade()
{
 if (millis() - fadeMillis >= 2)
 {
   fadeMillis = millis();

   if (fadeUp == true)
   {
     if (fadeValue < 255)
     {
       fadeValue++;
     }
     else
     {
       fadeUp = false;
       fadeValue--;
     }
   }
   else
   {
     if (fadeValue > 0)
     {
       fadeValue--;
     }
     else
     {
       fadeUp = true;
       fadeValue++;
     }
   }
   analogWrite(fled1, fadeValue);
   analogWrite(fled2, fadeValue);
 }
}
//STOP

  void frontflash()
  {

unsigned long currentMillis = millis();

 if(currentMillis - previousMillis > interval) {
   // save the last time you blinked the LED
   previousMillis = currentMillis;  

   // if the LED is off turn it on and vice-versa:
   if (ledState == LOW)
     ledState = HIGH;
   else
     ledState = LOW;

   // set the LED with the ledState of the variable:
   digitalWrite(led1, ledState);
   digitalWrite(led2, ledState);

 if (interval == 59)
    interval = 100;
 else if (interval == 100)
    interval = 61;
 else if (interval == 61)
    interval = 800;
 else
    interval = 59;
 }
  }
 
  void flashlight()
  {
   
 analogFade();

 frontflash();
  }



int PPMin1 = 2;  // connect the desired channel (PPM signal) from your RC receiver to analog pin2 on Arduino.

   int RCpuls1;  // store RC signal pulse length
   int adj_puls1;

   void setup()
   {
     Serial.begin(9600); //serial library start
   
     pinMode(PPMin1, INPUT); //Pin 2 as input
     pinMode(led1, OUTPUT);
     pinMode(led2, OUTPUT);

   }


   void loop()
   {
       RCpuls1 = pulseIn(PPMin1, HIGH);      //read RC channel 1
       adj_puls1 = map(RCpuls1, 1000, 2000, 2000, 1000);  // my observed RC values are between 1000 and 2000.. these might need to be changed, depending on your RC system.

     
       //light1
       if (adj_puls1 > 1700) {            
       //write here what to do
       flashlight();
     }
       else
   
     Serial.print ("puls1: ");
     Serial.print (RCpuls1);  // if you turn on your serial monitor you can see the readings.
     Serial.print ("       ");
     Serial.print ("adjusted:  ");
     Serial.print (adj_puls1);
     Serial.println ("  ");
   
   }


vorlage für das ganze ist dies hier: http://forums.adafruit.com/viewtopic.php?f=8&t=11119

So, Probleme:

1. Wozu brauch man 4 stellen bei map? die ersten beiden sind die range, also von bis und die beiden dahinter?
2. es funktioniert soweit recht gut, aber das analogfade und das frontflash wird vieeel zu langsam ausgeführt. wieso? und wie kann ich das lösen?

TelosNox

So, Probleme:

1. Wozu brauch man 4 stellen bei map? die ersten beiden sind die range, also von bis und die beiden dahinter?
2. es funktioniert soweit recht gut, aber das analogfade und das frontflash wird vieeel zu langsam ausgeführt. wieso? und wie kann ich das lösen?


zielWert = map(quellWert, quellRangeMin, quellRangeMax, zielRangeMin, zielRangeMax);

du hast einen QuellWert und der liegt irgendwo innerhalb der QuellRange, die du definierst. Du sagst map dann in welche neue Range er den Wert transformieren soll und das tut er dann.
map (50, 1, 100, 1, 10) macht folgendes.

Er schaut wo zwischen 1 und 100 die 50 liegt. Sie liegt in der Mitte (also bei 50%). Nun sucht er zwischen 1 und 10, wo die Mitte liegt und das ist dann der Wert, den map zurückgibt. Oder als Formel ausgedrückt: 50/(100-1) * (10-1).

Zu deinem 2. Problem.
Dein Fade erhöht den Wert bei jedem Durchlauf um 1 (bis zum Maximum von 255). Ein Durchlauf kommt alle 2 Millisekunden (dafür hast du die Abfrage mit den fadeMillis). Daher dauert der Fade 500ms.
Wenn dir das zu lang ist, dann musst du es anders lösen. Ich würde es aber komplett anders machen. Ich würde den Zeitraum definieren, den die Led brauchen darf, bis sie vollständig hell ist. Und den würde ich dann dazu nutzen, um zu errechnen, wie hoch der Fadevalue sein muss.
Etwa so:

const unsigned long fadeTime = 200; //200ms

analogFade(){
  static unsigned long fadeStartMillis = 0;
  static boolean fadeUp = true;

  if (fadeStartMillis == 0) {
    fadeStartMillis = millis();  //der Fade beginnt
  } else {
    unsigned long actualMillis = millis();
    if (fadeStartMillis + fadeTime > actualMillis) {
      fadeStartMillis = 0;   //der Fade ist fertig, jetzt umkehren und neu anfangen
      fadeUp != fadeUp;
    } else {
      int fadeDuration = actualMillis - fadeStartMillis;
      int fadeValue = 0;
      if (fadeUp) {
        fadeValue = map(fadeDuration, 0, fadeTime, 0, 255);
      } else {
        fadeValue = map(fadeDuration, fadeTime, 0, 0, 255);
      }
      analogWrite(fled1, fadeValue);
      analogWrite(fled2, fadeValue);     
    }
  }
}


Ist jetzt einfach mal schnell hingeschrieben. Ich nutze kein festes Intervall sondern ich schaue in jedem Loop nach, wieviele Millis seit Beginn des Fadevorgangs vergangen sind und berechne die FadeValue passend zum aktuellen Zeitpunkt. Das mache ich dann einfach mit Map. Das eine mal anschwellend und das andere mal abschwellend (einfach in Map eins der Wertepaare vertauschen). Ob der Code kompiliert weiß ich grad nicht, habs direkt ins Forum getippt. Wenn dann fehlt ne Klammer oder sowas. Aber als Orientierung und zum Verstehen des Algorithmus sollte es reichen.

Der Vorteil dieser Methode: Man kann jede beliebige FadeDuration realisieren.

uwefed

Normalerweise ist neues Problem - neue Tread


So, Probleme:
1. Wozu brauch man 4 stellen bei map? die ersten beiden sind die range, also von bis und die beiden dahinter?
2. es funktioniert soweit recht gut, aber das analogfade und das frontflash wird vieeel zu langsam ausgeführt. wieso? und wie kann ich das lösen?


So und jetzt die Antworten:
Hier eine Referenz: http://arduino.cc/en/Reference/HomePage
1) Map mapt einen Wert im Eingangsbereich in den Ausgangsbereich um. Beispiel:
risultat = map (Wert;100, 200, 0, 20);
De Bereich 100 bis 200 wird in 0 bis 20 ungerechnet; also ist zB Wert=150 ist risultat =10.
http://arduino.cc/en/Reference/Map

2) also muß ich dennoch den Kode lesen  ;) ;)
pulseIn() blockiert den Arduino gleich wie delay(); deshalb wartet er dort bis er den Puls gemessen hat.

Statt "adj_puls1 = map(RCpuls1, 1000, 2000, 2000, 1000); "
ist "adj_puls1= 3000-RCpuls1;" einfacher.
Statt
"if (ledState == LOW)
     ledState = HIGH;
   else
     ledState = LOW;"
einfach
"ledState = 1-ledState;"

Grüße Uwe

Daniel_N

Sry, dachte, da es auf dem "alten" Code aufbaut bleibe ich mal in dem Topic :/

Ein fade dauert aktuell geschlagene 16Sek (ca Angabe).
Vorgabe ist das: http://www.youtube.com/watch?v=tQQKtW60QQc (Blinker bzw Warnblinker)

ich probier mal den Code von TeloxNox aus und berichte dann.

Wieso aber wird der Fade verzögert, der Blitzer aber nicht?
Die Reference bin ich übrigens immer am durchforsten wenn ich nicht weiter weiss. und google ;)

uwefed

Hallo Daniel_N
Ich weiß nicht welche Frequenz das PPM-Signal hat; wenn es von einer Fernsteuerung ist, dann hat es 50 Hz und eine HIGH Impuls-Zeit von 1 bis 2mSek und Low-Zeit von 18 bis 19 mSek. Das heißt, daß ein pulsIn() bis zu 20mSek wartet bis es den HIGH Impuls mißt.
Der Fade hätte eine Wartezeit von 2mSeK und von 0 bis 255 und zurück bräuchte es ca 1 Sek. Durch die pulsIn() Funktion hast Du eine Zeitverzögerung um den Fakto 15 ca (laut Deinen Schätzungen)
Das Blink hat zeiten von 59,61,100 und 800 mSek da sind 79, 61,120 und 820 mSek keine großen Unterschiede die man bemerken könnte.

Grüße Uwe


 

Daniel_N

Das erklärt natürlich die extreme Verzögerung.
Der echte Mustang braucht übrigens für einmal blinken 750mSek.

Den frontflash (Doppelblitzer) kann ich ja anpassen indem ich die Werte um 20 runter setz, das funktioniert auch super.

Aber für den "Warnblinker" analogFade will mir nichts einfallen um das delay durch den pulsein zu verhindern :(

Gibt es eine gute Alternative zum PulseIn? Ich möchte am Ende einfach nur 3 Schaltzustände haben:
< 1200mSek für eine Lichtsequenz
1400 - 1600mSek für aus
> 1800mSek für eine andere Lichtsequenz

Am wichtigsten ist hierbei die > 1800mSek-Sequenz, da diese später den fade beinhaltet, die 1200er hat nur "lampe an".

Ich hab per google was von wegen timings auslesen gelesen, bin aber absolut nicht durchgestiegen und es war wohl auch kein "arduino C"

//EDIT:
Ganz vergessen, wollte ich schon vor ein paar Posts schreiben, da ich nicht weiss ob es wichtig ist.
Mein Arduino ist ein Pro Mini 328 - 5V/16MHz

Go Up