Go Down

Topic: Drehscheibe Modellbahn mit RF24 (Read 6454 times) previous topic - next topic

Acki1985

Hallo alle zusammen,

habe es mit eurer Hilfe geschaft und es soweit hinbekommen das es funktioniert. Nun möcht ich die Befehle per Funk übertragen und zurückgesendet bekommen wieviele Schritte nach jeder Sensor Betätigung noch übrig sind. Das Senden zur Scheibe funktioniert, nur bekomme ich keine restlichen Schritte zurückgesendet.
Der Sendecode ist nur ein Beispiel um es zu testen.

Sender:
Code: [Select]

#include  <SPI.h>
#include  "RF24.h"
byte data[1];
byte data_e[1];
//const uint64_t pipe = 0xF0F0F0F0A1LL;
const byte addresses[][6] = {"00001", "00002"};
RF24 radio(8, 9);
 
 
const int ledpin =  6;      // the number of the LED pin
 
void setup()
{
  Serial.begin(9600);
  pinMode(ledpin, OUTPUT);
 
  pinMode(0, INPUT_PULLUP);
  pinMode(7, INPUT_PULLUP);
  pinMode(2, INPUT_PULLUP);
  pinMode(3, INPUT_PULLUP);
  pinMode(4, INPUT_PULLUP);
  pinMode(5, INPUT_PULLUP);
  radio.begin();
  //radio.openWritingPipe(pipe);
  radio.openWritingPipe(addresses[0]); // 00002
  radio.openReadingPipe(1, addresses[1]); // 00001
  }
 
void loop()
{
  if (digitalRead(0) == HIGH ||
      digitalRead(7) == HIGH ||
      digitalRead(2) == HIGH ||
      digitalRead(3) == HIGH ||
      digitalRead(4) == HIGH ||
      digitalRead(5) == HIGH)
  {
    data[0] = 0;
    while (radio.available())
{
Serial.println("Data there");
radio.read(data_e, 1);
Serial.println(data_e[0]);
  }
  if (digitalRead(0) == LOW)
  {
    data[0] = 1;
    digitalWrite(LED_BUILTIN, HIGH);
 
  }
  if (digitalRead(7) == LOW)
  {
    data[0] = 2;
    digitalWrite(LED_BUILTIN, HIGH);
  }
  if (digitalRead(2) == LOW)
  {
    data[0] = 3;
    digitalWrite(LED_BUILTIN, HIGH);
  }
  if (digitalRead(3) == LOW)
  {
    data[0] = 4;
    digitalWrite(LED_BUILTIN, HIGH);
  }
  if (digitalRead(4) == LOW)
  {
    data[0] = 254;
    digitalWrite(LED_BUILTIN, HIGH);
  }
   if (digitalRead(5) == LOW)
  {
    data[0] = 255;
    digitalWrite(LED_BUILTIN, HIGH);
  }
  if (data [0] != 0) {
    radio.stopListening();
  radio.write(data, 1);
  digitalWrite(LED_BUILTIN, LOW);
  Serial.println(data[0]);
  radio.startListening();
}
}
}



Empfänger (Bühne):
Code: [Select]

#include <SPI.h>
#include "RF24.h"

int data[1];
boolean var = true;
const byte addresses[][6] = {"00001", "00002"};
//const uint64_t pipe = 0xF0F0F0F0A1LL;
RF24 radio (8, 9);
byte rever_start = false;
const byte hal = 3;
byte schritte = 0;                                 // Schritte die deer Motor (Hallsensor betätigt) zurücklegen muss
int sensor = LOW;
unsigned long startZeit;                          // Abfrage begin Sensor an Bühne
unsigned long interval = 2000;                    // Zeitabstand nachdem ein Zustand des Sensors erkannt wird
unsigned long vergangeneZeit = 0;                 // Speichervariable um die Vergangene Zeit zu speichern
/*int pin_motor_links = A1;                         // Richtungspin an H-Brücke für Links
int pin_motor_rechts = A2;                        // Richtungspin an H-Brücke für Rechts
*/
// Statusvariablen
bool nullStellung = false;                      // Speichervariable ob Referenzfahrt erfolgt ist
bool ergebnis = false;                          // Berechnung der Schritte und Richtung beendet/gestartet
bool start = false;                           // Schritte Zaehlen beendet/gestartet

void setup() {
  radio.begin();
  //radio.openReadingPipe(1, pipe);
  radio.openWritingPipe(addresses[1]); // 00001
  radio.openReadingPipe(1, addresses[0]); // 00002
  radio.startListening();
  pinMode(hal, INPUT);
  Serial.begin(9600);
  //delay (2000);
  }

void loop() {
 
  referenzFahrt();
  encoder_lesen();
  schritteZaehlen();
  }

void referenzFahrt() {
  while (radio.available())
{
Serial.println("Data there");
radio.read(data, 1);
Serial.println(data[0]);
}

      // Reverenzfahrt

      if ((data[0] == 255) && (rever_start == false) && (nullStellung == false)) {
        rever_start = true;
        Serial.println("Reverenzfahrt gestartet");
    //    digitalWrite (pin_motor_rechts, HIGH);
      //  digitalWrite (pin_motor_links, LOW);
      }
      else if ((data[0] == 254) && (rever_start == true)) {
        nullStellung = true;
        rever_start = false;
        Serial.println("Reverenzfahrt beendet");
        Serial.println (" "); 
  //      digitalWrite (pin_motor_rechts, LOW);
//        digitalWrite (pin_motor_links, LOW);
      }
    }
 
  void encoder_lesen () {
   if ((data[0] <= 48) && (data[0] > 0) && (nullStellung == true) && (ergebnis == false)) {
    //rechtsrum data[0] Schritte
    schritte = data[0];
    ergebnis = true;
    // Motor
    Serial.println (schritte);
    Serial.print ("  Schritte nach Rechts");
    Serial.println (" ");   
   }
  else if ((data[0] >=100) && (data[0] <= 149) && (nullStellung == true) && (ergebnis == false)) {
    //linksrum data[0] - 100 Schritte
    schritte = data[0] - 100;
    ergebnis = true;
    // Motor
    Serial.println (schritte);
    Serial.print ("  Schritte nach links");
    Serial.println (" ");
   }
  }
 
void schritteZaehlen () {

  sensor = digitalRead (hal);
  startZeit = millis();
  if ((schritte != 0) && (ergebnis == true)) {
    start = true;
    if ((startZeit - vergangeneZeit >= interval) && (sensor == LOW) && (start == true)) {
      vergangeneZeit = startZeit;
      schritte--;
      Serial.println("Drehung start");
      Serial.println("Restl. Pos.");
      Serial.println(schritte);
      radio.stopListening();
      radio.write (schritte, 3);
      radio.startListening();
      }
  }
  else if ((schritte == 0) && (start == true)) {
    start = false;
    ergebnis = false;
    data[0] = 0;
    Serial.println("Ziel");
    Serial.println("Drehung");
    Serial.println("beendet");
    Serial.print("daten");
    Serial.println(schritte);
    radio.stopListening();
    radio.write (schritte, 1);
    radio.startListening();
    delay(2000);
    Serial.println(ergebnis);
    }
}


Warum bekomme ich von der Bühne nichts zurück?

P.S. Ich weis nicht ob ich einen neuen Fred aufmachen soll?

Gruß Acki

agmue

P.S. Ich weis nicht ob ich einen neuen Fred aufmachen soll?
Hatte ich auch gerade überlegt, da ich beispielsweise einen Drehencoder habe aber keinen Funk.

Alternativ kannst Du auch Beitrag #0 editieren und dort die Überschrift verändern, in dem Du "RF24" erwähnst.
Wahnsinn und Verstand trennt nur eine dünne Wand. (Daniel Düsentrieb)

Acki1985


Acki1985

Hat denn wirklich keiner ne Idee?

HotSystems

Hat denn wirklich keiner ne Idee?
Ich kann nur vermuten, da der Sketch schlecht zu lesen ist.
Es fehlen einfach Kommentare, was an welcher Stelle passieren soll.

Ich vermute die Signale (Sender und Empfänger) überschneiden sich und damit bekommst der Empfänger nichts mit.
Gruß Dieter

I2C = weniger ist mehr: weniger Kabel, mehr Probleme. 8)

Whandall

Hast du sicher gestellt, dass die Kommunikation zu den NRFs funktioniert?
Am einfachsten machst du das mit printDetails().

Welche Art von Modulen benutzt du?

Wie versorgst du die NRFs mit 3.3V?

Warum benutzt du hier ein while?
Code: [Select]
    while (radio.available())
{
Serial.println("Data there");
radio.read(data_e, 1);
Serial.println(data_e[0]);
  }


Warum benutzt du falsche Kommentare?
Code: [Select]
const byte addresses[][6] = {"00001", "00002"};

  radio.openWritingPipe(addresses[0]); // 00002
  radio.openReadingPipe(1, addresses[1]); // 00001

const byte addresses[][6] = {"00001", "00002"};

  radio.openWritingPipe(addresses[1]); // 00001
  radio.openReadingPipe(1, addresses[0]); // 00002


Benutze bitte mal Ctrl-T in der IDE um deinen Sketch zu formatieren.
Ah, this is obviously some strange usage of the word 'safe' that I wasn't previously aware of. (D.Adams)

Acki1985

Guten Abend,

danke für deine Antwort.

zu 1. es sind diese Module      https://www.amazon.de/SainSmart-NRF24L01-drahtlos-Transceiver-2-4GHz/dp/B006CHFPFU

zu 2. Versorgt werden sie momentan über den Arduino NANO am 3,3Volt Ausgang mit Kondensator zur Masse

zu 3. Mit dem While das ist durch das testen entstanden. In manchen Codes steht while in anderen if.

zu 4. Das mit Kommentaren ist auch durch brobieren zu Stande gekommen.

Habe den Code nochmal geändert. Das Empfangen funktioniert super nur das zurück Senden nicht.

Wie gesagt der Sender ist nur zum testen. Ich habe die Variable Schritte immer händisch abgeändert.
Mit den Orginal Scetches von https://www.instructables.com/id/Arduino-and-NRF24L01/ hat das gegenseitige Senden und Empfangen funktioniert.


Hier meine Codes:
Sender (Provisorisch)

Code: [Select]

/*
  Arduino NRF24L01 simple transmit and receive - Created October 2017
    by DonX Developer
  *** This code was created for an Arduino UNO using the Arduino IDE v1.8.4. ***

  Parts:
  2 X Arduinos
  2 X Breadboards
  2 X NRF24L01
  [optional] 2 X YL-105 breakout boards for the NRF24L01. This allows for 5v connection and easier wiring
  2 X Momentary switches
  2 X Red LEDs
  2 X Yellow LEDs
  4 X 220 ohm resistors
  Jumper wires


  NRF24L01  Arduino pin
  VCC       3.3 V
  GND       GND
  CS        8
  CE        7
  MOSI      11 or ICSP-4
  MISO      12 or ICSP-1
  SCK       13 or ICSP-3

  If you use the YL-105 breakout board, the Vcc lead can go to the 5v Arduino pin

  Arduino pin 2 to Yellow LED long lead - anode
  Yellow short lead - cathode to 220ohm resistor, then second resistor lead to GND

  Arduino pin 3 to Red LED long lead - anode
  Red short lead - cathode to 220ohm resistor, then second resistor lead to GND

  Arduino pin 4 to switch, other side of switch to GND

  The physical build of the two boards are identical
  There are minor software differences for the reading and writing pipes for the respective boards.

*/

#include <SPI.h>
#include "RF24.h"

#define button 4
#define confirmLed 2
#define led 3

RF24 NRF24L01 (8, 9);//create object called NRF24L01. specifying the CE and CSN pins to be used on the Arduino

byte address[] [6] = {"pipe1", "pipe2"};//set addresses of the 2 pipes for read and write
boolean buttonState = false;//used for both transmission and receive
byte schritte[1];
void setup() {
  Serial.begin(9600);
  pinMode(button, INPUT_PULLUP);
  pinMode(confirmLed, OUTPUT);//yellow LED
  pinMode(led, OUTPUT);//red LED

  NRF24L01.begin();
  //open the pipes to read and write from board 1
  NRF24L01.openWritingPipe(address[0]);//open writing pipe to address pipe 1
  NRF24L01.openReadingPipe(1, address[1]);//open reading pipe from address pipe 2

  NRF24L01.setPALevel(RF24_PA_MAX);//set RF power output to minimum, RF24_PA_MIN (change to RF24_PA_MAX if required)
  NRF24L01.setDataRate(RF24_250KBPS);//set data rate to 250kbps
  NRF24L01.setChannel(110);//set frequency to channel 110
}

void loop() {
  schritte[0] = 105;
  //Transmit button change to the other Arduino
  delay(10);
  NRF24L01.stopListening();
  buttonState = digitalRead(button);//test for button press on this board
  if (buttonState == LOW)//button is pulled up so test for LOW
  {
    Serial.println("1");
   
    NRF24L01.write(&schritte, sizeof(schritte));//send LOW state to other Arduino board
    //flash the yellow LED to show progress
    digitalWrite(confirmLed, HIGH);
    Serial.println(schritte[0]);
    delay(100);
    digitalWrite(confirmLed, LOW);
  }

  buttonState = HIGH;//reset the button state variable

  //Receive button change from the other Arduino
  delay(10);
  NRF24L01.startListening();
  if (NRF24L01.available())//do we have transmission from other Arduino board
  {
    Serial.println("2");
   
    NRF24L01.read(&buttonState, sizeof(buttonState));//update the variable with new state
    NRF24L01.stopListening();
    Serial.println(buttonState);
  }
  if (buttonState == HIGH)//test the other Arduino's button state
  {
    digitalWrite(led, LOW);
  }
  else
  {
    flashLed();//indicate that the button was pressed on the other board
  }
  buttonState = HIGH;//reset the button state variable
}

//flash the red LED five times
void flashLed()
{
  for (int i = 0; i < 1; i++)
  {
    digitalWrite(led, HIGH);
    delay(200);
    digitalWrite(led, LOW);
    delay(200);
  }

}




Empfänger (Bühne)

Code: [Select]

#include <SPI.h>
#include "RF24.h"

int data[1];
boolean var = true;
//const byte addresses[][6] = {"00001", "00002"};
//const uint64_t pipe = 0xF0F0F0F0A1LL;
RF24 radio (8, 9);
byte address[] [6] = {"pipe1", "pipe2"}; // Adressen der beiden Pipes
byte rever_start = false;
const byte hal = 3;
byte schritte[1];                                 // Schritte die deer Motor (Hallsensor betätigt) zurücklegen muss
int sensor = LOW;
unsigned long startZeit;                          // Abfrage begin Sensor an Bühne
unsigned long interval = 2000;                    // Zeitabstand nachdem ein Zustand des Sensors erkannt wird
unsigned long vergangeneZeit = 0;                 // Speichervariable um die Vergangene Zeit zu speichern
/*int pin_motor_links = A1;                         // Richtungspin an H-Brücke für Links
  int pin_motor_rechts = A2;                        // Richtungspin an H-Brücke für Rechts
*/
// Statusvariablen
bool nullStellung = false;                      // Speichervariable ob Referenzfahrt erfolgt ist
bool ergebnis = false;                          // Berechnung der Schritte und Richtung beendet/gestartet
bool start = false;                           // Schritte Zaehlen beendet/gestartet

void setup() {
  radio.begin();
  //radio.openReadingPipe(1, pipe);
  radio.openWritingPipe(address[1]); // 00002
  radio.openReadingPipe(1, address[0]); // 00001
  radio.setPALevel(RF24_PA_MAX);
  radio.setDataRate (RF24_250KBPS);
  radio.setChannel(110);

  radio.startListening();
  pinMode(hal, INPUT);
  Serial.begin(9600);
  //delay (2000);
}

void loop() {

  referenzFahrt();
  encoder_lesen();
  schritteZaehlen();
}

void referenzFahrt() {
  if (radio.available())
  {
    Serial.println("Data there");
    radio.read(data, 1);
    Serial.println(data[0]);
  }

  // Reverenzfahrt

  if ((data[0] == 255) && (rever_start == false) && (nullStellung == false)) {
    rever_start = true;
    Serial.println("Reverenzfahrt gestartet");
    //    digitalWrite (pin_motor_rechts, HIGH);
    //  digitalWrite (pin_motor_links, LOW);
  }
  else if ((data[0] == 254) && (rever_start == true)) {
    nullStellung = true;
    rever_start = false;
    Serial.println("Reverenzfahrt beendet");
    Serial.println (" ");
    //      digitalWrite (pin_motor_rechts, LOW);
    //        digitalWrite (pin_motor_links, LOW);
  }
}

void encoder_lesen () {
  if ((data[0] <= 48) && (data[0] > 0) && (nullStellung == true) && (ergebnis == false)) {
    //rechtsrum data[0] Schritte
    schritte[0] = data[0];
    ergebnis = true;
    // Motor
    Serial.println (schritte[0]);
    Serial.print ("  Schritte nach Rechts");
    Serial.println (" ");
  }
  else if ((data[0] >= 100) && (data[0] <= 149) && (nullStellung == true) && (ergebnis == false)) {
    //linksrum data[0] - 100 Schritte
    schritte[0] = data[0] - 100;
    ergebnis = true;
    // Motor
    Serial.println (schritte[0]);
    Serial.print ("  Schritte nach links");
    Serial.println (" ");
  }
}

void schritteZaehlen () {

  sensor = digitalRead (hal);
  startZeit = millis();
  if ((schritte[0] != 0) && (ergebnis == true)) {
    start = true;
    if ((startZeit - vergangeneZeit >= interval) && (sensor == LOW) && (start == true)) {
      vergangeneZeit = startZeit;
      schritte[0]--;
      Serial.println("Drehung start");
      Serial.println("Restl. Pos.");
      Serial.println(schritte[0]);
      radio.stopListening();
      radio.write (&schritte, sizeof (schritte));
      radio.startListening();
    }
  }
  else if ((schritte[0] == 0) && (start == true)) {
    start = false;
    ergebnis = false;
    data[0] = 0;
    Serial.println("Ziel");
    Serial.println("Drehung");
    Serial.println("beendet");
    Serial.print("daten");
    Serial.println(schritte[0]);
    radio.stopListening();
    radio.write (&schritte, sizeof (schritte));
    radio.startListening();
    delay(2000);
    Serial.println(ergebnis);
  }
}

Whandall

#127
Feb 11, 2019, 11:53 pm Last Edit: Feb 12, 2019, 07:20 pm by Whandall
Mit dem verwendeten Modul kann es mit nur einem Kondensator funktionieren,
eine eigene vernünftige Stromversorgung ist betriebssicherer.

Dass dein delay-Gestolper im Sender keine Antworten empfängt wundert mich nicht.
Das Fenster in dem ein Empfang möglich ist, ist viel zu klein.
Delay gehört IMHO nicht in ein Programm das asynchrone Kommunikation versucht.

Code: [Select]
 for (int i = 0; i < 1; i++)

Wozu dient diese optische Kodebereicherung?
Der Kompiler entfernt sie völlig (zum Glück).
Ah, this is obviously some strange usage of the word 'safe' that I wasn't previously aware of. (D.Adams)

Acki1985

Hallo,

Stimmt das mit dem Delay mag ich auch nicht. Der Beispielcode war halt so.

Eigentlich soll es so ablaufen:

- benötigte Schritte werden zur Scheibe gesendet
- Scheibe sendet immer wenn ein Schritt runter gezählt wurde, die verbleibenden Schritte zurück
- Wenn Scheibe die benötigten Schritte zurückgelegt hat, soll die Aktuelle Position an den Sender gesendet
   werden
- Zwischenzeitlich (während die Schritte abgearbeitet werden) soll über den Sender ein Nothalt der Bühne
  ausgelöst werden können.

Das sind so die Vorstellungen.

Gruß Acki

Whandall

Stimmt das mit dem Delay mag ich auch nicht. Der Beispielcode war halt so.
Und dein Editor ist so beschädigt dass du keinen Kode ändern kannst?

Du hast mein Mitleid.
Ah, this is obviously some strange usage of the word 'safe' that I wasn't previously aware of. (D.Adams)

Acki1985

Klar kann ich den Code ändern. Mich hat nur gewundert das der Beispielcode (aus dem Link) ohne Probleme funktioniert hat. Meinst du das Delay im Sendecode?

Whandall

Ich meine alle delay() Aufrufe, oder versuchen deine Sketches nicht miteinander zu kommunizieren?

In der Regel reicht es nicht delay Aufrufe einfach zu löschen oder auszukommentieren.
Ah, this is obviously some strange usage of the word 'safe' that I wasn't previously aware of. (D.Adams)

Acki1985

Ich müsste sie durch eine Millis Abfrage ersetzen. Oder?

Whandall

Na ja, du solltest deinen Kode so blockierungsfrei wie möglich machen.
For und while solltest nicht für Abläufe nutzen
und möglichst keine (länger) blockierenden Funktionen aufrufen.

radio.write() blockiert auch, ist aber (spätestens) nach ca. 18 ms fertig.
Manchmal lässt sich ein kurzes Anhalten nur mit Kopfständen vermeiden,
beim NRF lohnt sich das IMHO nicht.

Deine Knoten sollten grundsätzlich lauschen und auf eingehende Pakete reagieren,
wenn sie was zu senden haben, schalten sie kurz den Empfänger ab,
senden und schalten ihn danach sofort wieder ein.

Deinen Stepper wirst du ja vielleicht auch zeitgesteuert bewegen wollen,
der Teil darf natürlich genauso wenig blockieren, sonst geht das mit der Anhalte-Anfrage in die Hose.
Ah, this is obviously some strange usage of the word 'safe' that I wasn't previously aware of. (D.Adams)

Acki1985

Der Motor ist nur ein ganz normaler DC Motor, welcher über einen Treiber angesteuert wird. Die Positionen (Schritte) werden über einen Sensor eingelesen. Die genaue Positionierung, sowie das Ausschalten erledigt der Orginale Raststift. So hatte ich es zumindest vor.

Go Up