Ich brauch professionelle Hilfe bei Programmierung meines Kranes

[021aet04]

Du muss den Interrupt wieder abschalten und die Led anschließend ausschalten. Wenn du nur den Interrupt abschaltest könnte es sein das die Led eingeschaltet bleibt.

Ansonsten würde ich empfehlen nicht einfach zusammen kopieren sondern Programme versuchen zu verstehen und im eigenen Programmierstil umzusetzten.

Ich weiß zwar nichts über arduino (bzw programmiersoftware), aber im AVR Studio kann man einstellen wieviel ein Tabulatorsprung ist und ob diese in leerzeichen umgewandelt werden sollen. Vielleicht gibt es auch bei arduino so eine Einstellung.

MfG Hannes

[HeXPloreR]

Da es sich anscheinend um eine Demofunktion handelt würde versuchen an der Seilrolle eine Odometriescheibe oder ahnliches Verfahren anzubringen und direkt an der Rolle per Interrupt messen wie viele cm Seil abgerollt/aufgerollt wurden.
Bei ereichen eines vorher definiertem Wert hört die Seilwinde dann auf. Beachtem muss mann das man je nach Drehrichtung addiert oder subtrahiert. Aber da du ja eh Relais ansteuerts sollte es kein Problem darstellen das im programm mit zu erfassen.
Die LED soll/braucht vermutlich nicht so ganz genau im Sekundentakt blinken, dann reicht es hier die millis-Funktion zu nutzen.

Der nachfolgende Code ist für ein openCM9-Board welches auch mit Sketch/Arduino-IDE programmiert werden kann, und muss ggf angepasst werden:

millis()
 : Returns the number of milliseconds since the Arduino board began running the current program. 
 This number will overflow (go back to zero), after approximately 50 days.
 */

// Variables:
long previousMillis = 0; // will store the last time the LED was updated
int interval = 1000;  // interval at which to blink (in milliseconds)

void setup() {
  // Set up the built-in LED pin as output:
  pinMode(BOARD_LED_PIN, OUTPUT);
}

void loop() {
  // Check to see if it's time to blink the LED; that is, if the
  // difference between the current time and last time we blinked
  // the LED is bigger than the interval at which we want to blink

  if ((int)millis() - previousMillis > interval) {
    // Save the last time you blinked the LED
    previousMillis = millis();

    // If the LED is off, turn it on, and vice-versa:
    toggleLED();
  }
}

Es gibt auch noch andrere Lösungen die wo ein Interrupt alles 50ms ausgelöst wird und dann zwei Werte hochzählt (LED und Winde) und dann jenachdem das Programm abzweigt. Allerdings wird bei einem zeitlichen Ablauf der Seilwinde vermutlich immer wieder das Problem entsehen das die position nicht passt.

Einfache Endanschläge ( auch optische) könnten praktisch auch funktionieren. Allerdings kommt es darauf an wie gut man die hinzugefügt bekommt. Besonders beim Haken senken könnte es problematisch werden.

[Valen]

Kannst du auch diesen "TimerOne.h" Datei hochladen? sonnst können wir das nicht Kompilieren.

Oder ist das diesen Library?
https://github.com/PaulStoffregen/TimerOne

- - - Aktualisiert - - -

Oder einer von diese 2:
http://code.google.com/p/arduino-tim...downloads/list

- - - Aktualisiert - - -

Wo sind s1 und s2 definiert in deine Code? Mein Arduino IDE meckert darüber beim Kompilieren.

- - - Aktualisiert - - -

Mechanisch funktioniert er und mit meinem geschriebenen Programm läuft er mit einigen Mängeln.
Könnte von Euch mal jemand ein Auge auf meinen Sketch werfen und mir mit Ideen zur Verbesserung weiterhelfen?

Dann verwundere ich mich wie du diesen Mängeln behebst. Ich verstehe nicht wie du diesen Code auf dein Arduino hochgeladen hast. Ich sehe auf verschiedene stellen Code das nicht funktionieren/kompilieren kann.

[Schiffe Bodo]

Hallo und danke erstmal für eure Zeit die ihr mir spendet
Ich habe den ersten Timer aus der Libary (https://github.com/PaulStoffregen/TimerOne)

s1 Ist an Pin 4 und ist das Schaltsignal von der Fernsteuerung
s2 Ist an Pin 6 und ist ein Endschalter welcher am Zahnrad des Kran befestigt ist (zum Krandrehen), damit kann ich feststellen ob der Kran in der Drehbewegung in Grundstellung ist, Seil wird nicht abgefragt

Das musste ich machen da der Kran sonst in jeder beliebigen Position das Programm beginnen kann. Der Steppermotor hat keine Stellungsabfrage integriert.

Das Seilabrollen ist nicht sehr problematisch. Die Tauchtasse wird nur auf dem Deck abgestellt und an der anderen Position ins Wasser abgelassen. Da kommt es also auf ein paar millimeter nicht an. wichtig ist aber das Aufrollen, da der Haken den Arm berühren soll und diesen dann leicht mit anheben.

Die LED soll ab Startsignal blinken und wenn der Durchlauf beendet ist auch aufhören.

Noch zur Info ich benutze einen Arduino Uno.

[Rabenauge]

Das mit dem Seil wirst du mit Zeitsteuerung _nie_ in Griff bekommen. Motoren drehen sich ohne Regelung praktisch niemals reproduzierbar. Und für ne Regelung brauchst du ein Feedback.
Sinnvoll wäre hier sowas wie ein Mikrotaster, der dann gedrückt (oder losgelassen) wird, wenn der Haken den Kranarm bis Anschlag hochgezogen hat. Alternativ auch hier ein Schrittmotor, bei dem du die Umdrehungen mitzählen kannst-aber auch das geht daneben, wenn das Seil mal _etwas_ anders aufgewickelt wird.
Besser ist es in jedem Fall, die Endposition des Armes auszuwerden. Statt nem Taster kannst du hier zum Beispiel auch ne Lichtschranke benutzen, die in der Endposition unterbrochen wird, je nachdem, was sich besser nachrüsten lässt.

[Schiffe Bodo]

Danke Sly, ich hab mir das schon gedacht und mir schon einige Gedanken gemacht. es müsste mit einem Reedkontakt funktionieren, der am Kranausleger befestigt ist und vom am Haken befestigten Magneten ausgelöst wird. Ich dacht mir Haken oben( Kontakt geschlossen) Grundstellung Haken. aber bei einem Kran im Maßstab 1:45 ist das auch nicht ganz so einfach zu lösen ohne das Aussehen des Kranes zu sehr zu verändern. Eventuell geht auch ein kleiner Microschalter, muss ich mal schauen was es so gibt.
ich bin erstmal dabei mein Programm aufzuräumen und so zu laufen zu bringen wie ich es mir vorstelle.

[Valen]

Ok,

Ich habe S1 und S2 definiert wie du angegeben hat. Aber mit Großbuchstaben S, weil klein nicht erlaubt ist. (Verstehe ich auch nicht warum.) Jedenfals nicht ein einziges Kleinbuchstabe s (mit oder ohne weitere Nummer):

Und naturlich jeder stelle von s1 und s2 in den Code geändert zu S1, S2.

#define S1 4
#define S2 6

Und dann Kompiliert den IDE den Code ziemlich ohne Problemen. Ja, den Kompiler hat es Prima gefunden. Leider nicht für den Code-Polizei.

Weil du auf verschiedene stellen ein if-Befehl macht wie das hier:

if (Kranstart,HIGH)  digitalWrite (RELAISAUF, HIGH);

Den Komma ist ein gültige Operator, aber es wird nicht ausgewertet wie du wahrscheinlich möchtest. (ausführen wenn Kranstart pin HIGH ist, oder nicht wann es LOW ist) Bei einer derartige if-Befehl wirdt den Komma-operator die erste Gesetzen vor den (letzte) Komma jedenfalls ausführen (den Variabel Kranstart macht nichts alleine). Aber nur das letzte Gesetz hinter der Komma wird für den Entscheidung sorgen. (HIGH=1, und alles was nicht gleich 0 ist wird immer als Wahr gekennzeichnet) Alles da vorne macht nichts dazu. Also was hier oben steht ist:

if ( 1!=0 )  digitalWrite (RELAISAUF, HIGH);  //      '1 ist nicht gleich an 0'

Deshalb wird das digitalWrite jedenfalls ausgeführt. Nur nicht wenn den Variabele Kranstart die wert 0 bekommen hat.

Die richtige Verwendung von ein if Befehl ist:

if ( digitalRead(Kranstart)==HIGH ) digitalWrite (RELAISAUF, HIGH);

Das gleiche passiert bei den while- loop am ende. Das hat nähmlich auch ein Quasi-if-Befehl in sich.

while(S2,HIGH);

Das ist ein Endloss-schleife, und dein Programm wird da niemals rauskommen. Arduino loop() wirdt nicht weiter loop-en.

Ok, dann haben wir nun die Logik Befehlen korrigiert. Dann stimmt es doch? Nein leider noch nicht.

Zurück zum anfang von den loop() Funktion:

void loop()
{ //Anfang der loop-Funktion

value = digitalRead(S2); // Endschalter wird eingelesen

digitalWrite(Kranstart,value); // Warte mall!! Kranstart ist als int definiert, und hat den wert HIGH (1) bekommen. Das ist doch kein Pin-bezeichnung!!!
// das ist gleich: digitalWrite(1, value); Wird kommisches verhalten geben mit den Seriele port (TX) Keine ahnung was Kranstart hier bedeuten soll.


{ // Wieso alles hiernach in ein {} scope einfassen? Sollte den { nicht gerade nach den ) kommen?

if ( digitalRead(Kranstart)==HIGH ) digitalWrite (RELAISAUF, HIGH);

// Hier ist das if-Befehl schon am ende, wegen der ; am ende des if-Befehls, und den fehlende { dazwischen. Alles hiernach wird jedenfalls ausgeführt.


delay(2600);
digitalWrite (RELAISAUF, LOW);
delay(2000);

myStepper1.step(-3600); //Kran ausdrehen (Winkel einstellen)
delay(2000); //Warten
digitalWrite (RELAISAB, HIGH);
delay(13000); //Seil ab für 13 sec
digitalWrite (RELAISAB, LOW);
delay(20000);

digitalWrite (RELAISAUF, HIGH);
delay(13000);
digitalWrite (RELAISAUF, LOW);
delay(2000);


myStepper1.step(3600); //Kran eindrehen
delay(3000);

digitalWrite (RELAISAB, HIGH);
delay(2600);
digitalWrite (RELAISAB, LOW);
delay(5000);
} // Ende von if-Befehl Wahr-code

{
while(S2,HIGH); //Aber nür eines mall! Hängt hier ab. Gleich wie bei if Korrigieren.
}
} //Ende von loop()

- - - Aktualisiert - - -

Ok,

Ich habe S1 und S2 definiert wie du angegeben hat. Aber mit Großbuchstaben S, weil klein nicht erlaubt ist. Jedenfals nicht ein einzige Kleinbuchstabe s (mit oder ohne weiteres Nummer):

Und naturlich jeder stelle von s1 und s2 in den Code geändert zu S1, S2.

#define S1 4
#define S2 6

Und dann Kompiliert den IDE den Code ziemlich ohne Problemen. Ja, den Kompiler hat es Prima gefunden. Leider nicht für den Code-Polizei.

Weil du auf verschiedene stellen ein if-Befehl macht wie das hier:

if (Kranstart,HIGH)  digitalWrite (RELAISAUF, HIGH);

Den Komma ist ein gültige Operator, aber es wird nicht ausgewertet wie du wahrscheinlich möchtest. (ausführen wenn Kranstart pin HIGH ist, oder nicht wann es LOW ist) Bei einer derartige if-Befehl wirdt den Komma-operator die erste Gesetzen vor den (letzte) Komma jedenfalls ausführen (den Variabel Kranstart macht nichts alleine), aber nur das letzte Gesetz hinter der Komma wird für den Entscheidung sorgen. (HIGH=1, und alles was nicht gleich 0 ist wird immer als Wahr gekennzeichnet) Alles da vorne macht nichts dazu. Also was hier oben steht ist:

if ( HIGH==1 )  digitalWrite (RELAISAUF, HIGH);

Deshalb wird das digitalWrite jedenfalls ausgeführt. Eben wenn Kranstart möglicherweise LOW oder 0 ist.

Die richtige Verwendung von ein if Befehl ist:

if ( digitalRead(Kranstart)==HIGH ) digitalWrite (RELAISAUF, HIGH);

Das gleiche passiert bei den while- loop am ende. Das hat nähmlich auch ein Quasi-if-Befehl in sich.

while(S2,HIGH);

Das ist ein Endloss-schleife, und dein Programm wird da niemals rauskommen. Arduino loop() wirdt nicht weiter loop-en.

Ok, dann haben wir nun die Logik Befehlen korrigiert. Dann stimmt es doch? Nein leider noch nicht.

Zurück zum anfang von den loop() Funktion:

void loop()
{ //Anfang der loop-Funktion

value = digitalRead(S2); // Endschalter wird eingelesen

digitalWrite(Kranstart,value); // Warte mall!! Kranstart ist als int definiert, und hat den wert HIGH (1) bekommen. Das ist doch kein Pin-bezeichnung!!!
// das ist gleich: digitalWrite(1, value); Wird kommisches verhalten geben mit den Seriele port (TX) Keine ahnung was Kranstart hier bedeuten soll.


{ // Wieso alles hiernach in ein {} scope einfassen? Sollte den { nicht gerade nach den ) kommen?

if ( digitalRead(Kranstart)==HIGH ) digitalWrite (RELAISAUF, HIGH);

// Hier ist das if-Befehl schon am ende, wegen der ; am ende des if-Befehls, und den fehlende { dazwischen. Alles hiernach wird jedenfalls ausgeführt.


delay(2600);
digitalWrite (RELAISAUF, LOW);
delay(2000);

myStepper1.step(-3600); //Kran ausdrehen (Winkel einstellen)
delay(2000); //Warten
digitalWrite (RELAISAB, HIGH);
delay(13000); //Seil ab für 13 sec
digitalWrite (RELAISAB, LOW);
delay(20000);

digitalWrite (RELAISAUF, HIGH);
delay(13000);
digitalWrite (RELAISAUF, LOW);
delay(2000);


myStepper1.step(3600); //Kran eindrehen
delay(3000);

digitalWrite (RELAISAB, HIGH);
delay(2600);
digitalWrite (RELAISAB, LOW);
delay(5000);
} // Ende von if-Befehl Wahr-code

{
while(S2,HIGH); //Aber nür eines mall! Hängt hier ab. Gleich wie bei if Korrigieren.
}
} //Ende von loop()