Neopixel - LEDS leuchten obwohl sie es nicht sollten

[Beinhardt]

folgendes ist mir noch aufgefallen.

Code:

strip.setBrightness(5);
  strip.show();
  Serial.begin(9600);
  strip.begin();
  strip.show(); // Initialize all pixels to 'off'

die Reihenfolge sollte hier:

Code:

strip.begin();
strip.setBrightness(5);
strip.clear();

sein.

Fehlt hier der Wert 266 absichtlich?

Code:

{255, 256, 257, 258, 259, 260, 261, 262, 263, 264, 265, 267, 268, 269, 270};

Code:

#define LED_COUNT 270

vs. deine Schleifenzähler (0-255) sind allerdings nur im Byte Bereich:

Code:

for (byte w = 0; w < lange; w++) {

Warum ist diese Zeile auskommentiert?

Code:

//weg[0] = n; // Kopf der Schlange ist Pixel Nr. n

weg[0] ist in allen folgenden Funktionen undefiniert, bis Du wieder zur Hauptloop kommst. Speziell bei essen_aufnehmen gibts Probleme.

Ich sehe Funktionen als void definiert, haben aber ein "return".
Bitte schalte mal alle Warnungen im Compiler an ...

[Mauser]

(29.11.2020 17:19)Beinhardt schrieb:  folgendes ist mir noch aufgefallen.

Code:

strip.setBrightness(5);
  strip.show();
  Serial.begin(9600);
  strip.begin();
  strip.show(); // Initialize all pixels to 'off'

die Reihenfolge sollte hier:

Code:

strip.begin();
strip.setBrightness(5);
strip.clear();

sein.

Fehlt hier der Wert 266 absichtlich?

Code:

{255, 256, 257, 258, 259, 260, 261, 262, 263, 264, 265, 267, 268, 269, 270};

Code:

#define LED_COUNT 270

vs. deine Schleifenzähler (0-255) sind allerdings nur im Byte Bereich:

Code:

for (byte w = 0; w < lange; w++) {

Warum ist diese Zeile auskommentiert?

Code:

//weg[0] = n; // Kopf der Schlange ist Pixel Nr. n

weg[0] ist in allen folgenden Funktionen undefiniert, bis Du wieder zur Hauptloop kommst. Speziell bei essen_aufnehmen gibts Probleme.

Ich sehe Funktionen als void definiert, haben aber ein "return".
Bitte schalte mal alle Warnungen im Compiler an ...

Hallo Beinhardt,

danke für die Hinweise, habe ich auch direkt geändert, also fast alles.

Code:

#include <IRremote.h>
#include <Adafruit_NeoPixel.h>

decode_results results;
#define LED_PIN    6
#define LED_COUNT 270

#define RECV_PIN 11
IRrecv irrecv(RECV_PIN);

byte weg[LED_COUNT]; // Hier befindet sich der Weg den die Schlange geht

int letzteZeile[15] = {255, 256, 257, 258, 259, 260, 261, 262, 263, 264, 265, 267, 268, 269, 270}; // Zur Abfrage angelegt bei vertikaler+ Bewegung

int i = 0;
int n = 0;
int zahler; // Zähler der die Durchläufe von LOOP zählt
int dot=80;    // Futter für die Schlange
int lange = 2; // Länge des Weges den die Schlange geht
int Eingabe; // Variable der IR Fernbedienung zur Steuerung
int Ergebnis; // Variable der IR Fernbedienung zur Steuerung
Adafruit_NeoPixel strip(LED_COUNT, LED_PIN, NEO_RGB);
void weg_einschalten(); // Die LED in Weg einschalten
void weg_verschieben(); // Die LED in Weg verschieben
void weg_0_ausschalten(); // Letzte Position in Weg ausschalten
void laufen_lassen(); // Automatische Bewegung der Schlange bei Ergebnis
void essen_aufnehmen(); // Schlange wachsen lassen
void setup() {
strip.begin();
  strip.setBrightness(5);
  strip.clear();
  Serial.begin(9600);
  
//  strip.show(); // Initialize all pixels to 'off'
  irrecv.enableIRIn();

}

void loop() {


  if (irrecv.decode(&results)) {

    //Serial.println(results.value,DEC); // gibt den Code in Dezimalzahlen aus
    irrecv.resume();                    // Warte auf nähsten Wert
    Eingabe = results.value;

  }
  if (results.value == 16734885) { // Horizontal+
    Ergebnis = 1;
    //Serial.println(Ergebnis);
  }
  if (results.value == 16730805) { // Vertikal +
    Ergebnis = 2;
    //Serial.println(Ergebnis);
  }
  if (results.value == 16716015) { //Horizontal -
    Ergebnis = 3;
    //Serial.println(Ergebnis);
  }
  if (results.value == 16718055) { // Vertikal -
    Ergebnis = 4;
    //Serial.println(Ergebnis);
  }

  zahler = zahler + 1; // Nach dem 40 Durchlauf wird 1x Fressen gespawnt
  weg[0] = n; // Aktuelle Position des Kopfes ist gleich LED Nr. n
  if (zahler == 60) { // Fressen spawnen
    dot = random(0, LED_COUNT);
    strip.setPixelColor(dot, 100, 100, 0);
    //strip.show();
    zahler = 0;
  }

  weg_einschalten();  //Die LED in Weg einschalten
  
  weg_verschieben();  //Die LED in Weg verschieben
  
  laufen_lassen();    //Automatische Bewegung der Schlange bei Ergebnis

  essen_aufnehmen();  // Schlange wachsen lassen
  strip.show();
}


void weg_einschalten() { // Die LED in Weg einschalten
  weg[0] = n; // Kopf der Schlange ist Pixel Nr. n
  int b;
//  strip.fill(0,0,270);
  for (byte w = 0; w < lange; w++) {
    b= weg[w];
    strip.setPixelColor(b, 100, 0, 0);
  }
  strip.show();
  
}
void weg_verschieben() { // Die LED in Weg verschieben

  int wegzw[LED_COUNT];
  for (byte w = 0; w < lange; w++) { // Weg kopieren
    wegzw[w] = weg[w];
  }

  for (byte w = 0; w < lange; w++) { // Weg nach rechts verschieben
    weg[w + 1] = wegzw[w];
    }

  //weg[0] = n; // Kopf der Schlange ist Pixel Nr. n
delay(500); // wird benötigt damit die Schlange Befehle empfangen kann
  
}
void laufen_lassen() {

  switch (Ergebnis) {
    case 1:// horizontaler Lauf+
      n = n + 1;
      if (n == 15)
        n = 0;
      break;
    case 2: // vertikaler Lauf+
      for (int m = 0; m <= 14; m++) { // vertikale Achse zurücksetzen
        if (n == letzteZeile[m])
          n = letzteZeile[m] - 18 * 15;
        //Serial.println(n);
      }
      n = n + 15;
      break;
    case 3: // horizontaler Lauf-
      n = n - 1;
      if (n == 0)
        n = 15;
      break;
    case 4: //vertikaler Lauf-
      for (int m = 0; m <= 14; m++) { // vertikale Achse zurücksetzen
        if (n == letzteZeile[m])
          n = letzteZeile[m] + 18 * 15;
      }
      n = n - 15;
      break;
  }
    weg_0_ausschalten();// letzte Position des Schlangenendes ausschalten
//  return n;
}
void weg_0_ausschalten() { // setzt letzten Eintrag in weg[lange] zurück
  int off;
  off = weg[lange];
  strip.setPixelColor(off, 0, 0, 0);
}
void essen_aufnehmen() { // Schlange wachsen lassen

strip.fill(0,0,270);
strip.setPixelColor(dot, 100, 100, 0);

  if (weg[0] == dot) {    // Wenn Kopf der Schlange == Fressen ist, Länge der Schlange++

    lange=lange+1;      // Sofern Länge nicht auskommentiert ist, gehen wahllos LEDs an.

    strip.setPixelColor(dot, 0, 0, 0); // LED die Futter anzeigt ausschalten
    //strip.show();
  }

}

Heute abend habe ich einen Ratschlag von Tommy56 weiter getestet. Der obige Code enthält nun das funktionsfähige Snake Spiel, ohne Nebeneffekte. Die gravierende Änderung ist in essen_aufnehmen zu finden. Strip.fill ist an dieser Stelle "richtig", somit blitzen die besagten LEDs auch nicht mehr auf. Neben einigen kleinen Bugs ist jetzt allerdings das Rücksetzen der LED "Dot" hinzugekommen die das Fressen darstellt. Ist aber nicht das große Problem da die Zeit ausreicht. Nervt mich allerdings ein wenig. Hab den Befehl zweimal drin jetzt.

LG Mauser und danke dir für die Hilfe!

[Mauser]

(29.11.2020 14:11)Tommy56 schrieb:  

(29.11.2020 02:57)Mauser schrieb:  ich habe das mit dem strip.fill ausprobiert. Anfangs dachte ich es funktioniert. Aber jetzt blitzen die besagten LEDs nur noch auf.

Zeige doch nochmal Deinen aktuellen Sketch.

Gruß Tommy

Moin Tommy,

in #66 ist der aktuelle Code zu finden. Habe strip.fill jetzt in essen_aufnehmen gesetzt. Jetzt blitzen die Leds auch nciht mehr auf. Ich finde aber, dass es eine unschöne Lösung ist, da irgendein Fehler den ich nicht finde überschrieben wird. Das nervt. strip.show kommt immer noch mehrfach vor, da es am Ende von Loop nicht funktioniert. Außerdem musste ich die LED "Dot" erneut setzen, damit sie länger sichtbar ist. Hab das Spiel jetzt ein paar Minuten getestet und es läuft einwandfrei. Hoffe das es so bleibt bis zum Ende.

Sicherlich kann ich den Code noch weiter verbessern. Sind auch noch ein paar Bugs enthalten. Aber es Läuft!

Vielen Dank für deine Hilfe und auch an alle anderen! Ich halte euch auf dem Laufenden!

[Mauser]

(29.11.2020 15:48)hotsystems schrieb:  

(29.11.2020 15:11)Mauser schrieb:  Vermutlich liege ich total falsch.....

Total falsch vermutlich nicht.
Nur wenn ich deinen Argumenten folgen würde, bräuchten wir alle keine Datenblätter, bzw. es muss sich keiner daran halten.

Beispiel: Ein Auto fährt auch auf drei Rädern, also lassen wir das vierte einfach weg.

Natürlich ist das nicht so einfach, auch nicht in deinem Projekt. Somit sollten alle Vorgabe aus Datenblättern eingehalten werden, damit da schon mal keine Fehler auftreten können.

Sind in deinem Stripe zu dünne Kabel verbaut und nicht genügend Einspeisepunkte vorhanden, können die Controller in den Leds durch kurze Spannungseinbrüche unbeabsichtigte Anzeigen liefern, also Fehler hervor rufen. Hast du das meßtechnisch mal untersucht ?
Dieses Problem muss nicht immer auftreten, aber kann. Somit sollte man das verhindern, indem man sich in allen Belangen an die Vorgaben aus den Datenblättern hällt.

Hallo Dieter,

Meßtechnisch habe ich die Spannung untersucht. Diese ist an dem besagten Stripe um ~1V geringer. Liegt bei 3,5V oder so an dieser Stelle, während am ersten Stripe noch >4V anliegen.
Allerdings habe ich danke Tommy56 das Problem nun mehr oder weniger gelöst, wenn auch unschön. Ist in Beitrag #66 zu finden wenn ich mich nicht irre. Habe strip.fill in void essen_aufnehmen geschrieben und siehe da, es läuft. Erstmal...

Wegen der Stromversorgung. Was hälst du von diesem Netzteil.
https://www.amazon.de/-/en/gp/product/B0...94EV&psc=1

Es hat 100W bei 12V und 8A.

Was die Kabelstärke angeht, so habe ich 0,5mm² genutzt, einadrig. Hab noch 1mm² herumliegen aber das Löten... Ich denke 16A wird das in jedemfall nicht aushalten aber 8A sollten schon drin sein. Hier soll sogar 12A drin sein.
https://www.lappkabel.de/fileadmin/catal...bellen.pdf

Ansonsten wollte ich noch eine Feinsicherung einbauen, um sicher zu gehen.

LG Mauser

[Tommy56]

(30.11.2020 00:14)Mauser schrieb:  Es hat 100W bei 12V und 8A.

Was willst Du mit einem 12 V - Netzteil an 5V LED? Diese grillen?

Gruß Tommy

[rev.antun]

Servus!

Lies dir mal diesen Thread durch

[Tommy56]

Wen meinst Du? Den TO oder mich?

Gruß Tommy

[rev.antun]

(30.11.2020 17:07)Tommy56 schrieb:  Wen meinst Du? Den TO oder mich?

Gruß Tommy

TO