Events Zählen mit Interrupt (Arduino, Zeitproblem)

[Daniel O. (daniel_o)]

Nach langer Zeit bin ich mal wieder zurück im Forum, gleich mit einer 
Frage.
Ich möchte einen Geigerzähler am Arduino anbinden. Alles was der Arduino 
machen soll, ist die Impulse in 10 sek. Zählen, mit 60 multiplizieren 
und anschließend ausgeben und dann wieder messen usw.
Die Impulse sind als Interrupt eingebunden, das klappt auch ganz gut. 
Allerdings hab ich das Gefühl, dass jeder Impuls und damit der Interrupt 
die Messzeit verlängert, und bei 5000 Imp. ist das schon sehr spürbar. 
Kann man das umgehen? Im Anhang der aufs Wesentliche gekürzte Code:






  Void setup(){
  attachInterrupt(0,countPulse,FALLING);
}

void loop(){
  if (millis()-timePreviousMeassure > 10000){
    countPerMinute = 6*count;
    radiationValue = countPerMinute * CONV_FACTOR;
    timePreviousMeassure = millis();
    Serial.print("cpm = ");
    Serial.print(countPerMinute,DEC);
    Serial.print(" - ");
    Serial.print("uSv/h = ");
    Serial.println(radiationValue,4);
    lcd.clear();
    lcd.setCursor(0, 0);
    lcd.print("CPM=");
    lcd.setCursor(4,0);
    lcd.print(countPerMinute);
    lcd.setCursor(0,1);
    lcd.print(radiationValue,4);
    lcd.setCursor(6,1);
    lcd.print(" uSv/h");

    count = 0;

  }

}

void countPulse(){
  detachInterrupt(0);
  count++;
  while(digitalRead(2)==0){
  }
  attachInterrupt(0,countPulse,FALLING);
}

[yesitsme (Gast)]

Warum detach/attach im ISR?
Du triggerst auf die falling Edge warum blockierst du den ISR solange 
der Pin Low ist?

[Daniel O. (daniel_o)]

Der Code ist von der Projektseite des Zählers. Ich hab vorher nie mit 
Interrupts arbeiten müssen. Ich dachte erst, damit nicht bei anstehendem 
LOW weitergezählt wird, aber es wird ja nur bei fallender Flanke 
gezählt. Könnte das Entfernen der while das Problem schon lösen?

[Peter D. (peda)]

Daniel O. schrieb:
> Im Anhang der aufs Wesentliche gekürzte Code:

Gekürzter Code ist immer Mist, wenn Du ihn nicht getestet hast.
Und Du hast ihn nicht angehängt, sondern eingefügt.
Häng den exakten getesteten Code an.

Was auffällt, zwischen
>    timePreviousMeassure = millis();
und
>    count = 0;
ist noch ein riesen Codewust für Ausgaben. Das kostet natürlich Zeit, wo 
nicht gezählt wird.

yesitsme schrieb:
> Warum detach/attach im ISR?

Ja, sowas macht man nicht.

[Daniel O. (daniel_o)]

Das Anhängen ist übers Handy schwierig, aber.hier ist ein Link zum 
Projekt:

https://www.cooking-hacks.com/documentation/tutorials/geiger-counter-radiation-sensor-board-arduino-raspberry-pi-tutorial

[Oliver S. (oliverso)]

Und hier ist ein Link zur Lösungen:

https://www.google.de


Wenn du Hilfe willst, häng dein Programm hier an.

Oliver

[Daniel O. (daniel_o)]

Herzlichen Dank

[Sven K. (quotschmacher)]

- count ist nicht volatile
- digitalRead ist so ziemlich die langsamste variante, um einen pin zu 
lesen -> würde ich nie in einer isr verwenden
- while in der isr? sofort raus damit!!
- wie schon geschrieben -> die isr kommt nur bei einer flanke. wozu also 
auf eine pinänderung warten?
- code verlinken ist blöd! schreib wenigstens dazu, dass er gaaaanz 
unten auf der seite ist - dann muss ich nicht so viel scrollen sondern 
kann gleich auf ende drücken
- count direkt nach der berechnung von countperminute nullen, nicht erst 
so spät

[Kris (Gast)]

Nur mal so ins blaue geratet.

Je mehr Signale du bekommst, desto langsamer ist dein Update auf dem 
Monitor/LCD? So habe ich dein Problem verstanden. Arduino Reference sagt 
dazu

millis() relies on interrupts to count, so it will never increment 
inside an ISR

Und wenn in deiner Main steht

if (millis()-timePreviousMeassure > 10000)

und millis bekommt kein update mehr, dann wird das ganze etwas länger 
dauern.

By the way, Arduino Reference solltest du dir wirklich mal anschauen. Da 
steht nämlich auch

attachInterrupt(digitalPinToInterrupt(pin), ISR, mode) (recommended)

Weil die Interruptzahl 0 hätte ich auf 0x00 geschätzt und das wäre der 
Reset Interupt.

Cheers