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);
}
Gast
#6117188
Warum detach/attach im ISR? Du triggerst auf die falling Edge warum blockierst du den ISR solange der Pin Low ist?
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?
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.
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
Und hier ist ein Link zur Lösungen: https://www.google.de Wenn du Hilfe willst, häng dein Programm hier an. Oliver
Herzlichen Dank
- 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
Gast
#6117284
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