jenkie / Arduino-Pedelec-Controller

Software for the Pedelec/E-Bike controller based on Arduino hardware, see www.pedelecforum.de "ForumsController"
http://www.pedelecforum.de/forum/showthread.php?14653-ForumsController
81 stars 44 forks source link

current correction offset #8

Closed agentmnm closed 12 years ago

agentmnm commented 12 years ago

currently the correction for the current display is done in the code.

I would be nice to have a variable in config.h instead.

Just testing github :-D

jenkie commented 12 years ago

Welche Korrektur meinst du genau? "current_display" ist nur der gemittelte "current". Meinst du Variablen für Offset und Amplitude für die Stromberechnung? :D

agentmnm commented 12 years ago

Ja, ich mein die Variablen:

snippet:

if HARDWARE_REV >= 3

current = (analogRead(current_in)-512)*0.0740543263; //check with multimeter and change if needed!
current = constrain(current,-30,30);

endif

bei meinem Testsetup habe ich jetzt konstant 0.1A Versatz:

z.B so: in config.h

float current_offset = 0.0; // set this value according to your own calibration float current_amplitude = 0.0741 // set this value according to your own calibration

und im Code:

if HARDWARE_REV >= 3

current = (analogRead(current_in)-512+current_offset)*; //check with multimeter and change if needed!
current = constrain(current,-30,30);

endif

ich habe mir erlaubt die Amplitude auf 3 signifikante Stellen zu runden :-D

P.S. Code kann man anscheinend nicht markieren und die Tabs gehen bei Copy Paste verloren :/ P.P.S. schön, daß das Write-Fenster so klein ist, man fühlt sich wie bei ebay :-D

agentmnm commented 12 years ago

hups es fehlt noch current_amplitude im code

if HARDWARE_REV >= 3

current = (analogRead(current_in)-512+current_offset)*current_amplitude; //check with multimeter and change if needed! current = constrain(current,-30,30);

endif

agentmnm commented 12 years ago

ähem, ich hatte das Thema eigentlich nicht geschlossen. Hab wohl falsch geklickt!

thomasjfox commented 12 years ago

Am besten wäre es, wenn Du das Repository "forkst", deine Änderungen einbaust und dann einen "pull request" sendest. Dann sieht man die Änderungen schön im Kontext.

Die Auto-Kalibrierung für das Offset wurde für Hardware-Revisionen ab 1.3 wegen der Reku rausgelassen, da hier auch negative Ströme fließen können.

thomasjfox commented 12 years ago

Ich habe gerade entdeckt, daß Du mein Repo schon geforkt hast :)

Besser wäre es, wenn Du das von "jenkie" nimmst, daß ist der "offizielle" Ort um Änderungen einzusenden.

Danke.

agentmnm commented 12 years ago

mit dem 'forken' hatte ich etwas Schwierigkeiten. Die Bedienung von github ist für mich nicht ganz intuitiv...

Ich habe zwar auf geklickt, aber keine Rückmeldung bekommen und weiterhin habe ich nicht gesehen, daß ich irgendwo Quellcode ändern könnte. Ich dachte das liegt daran, daß ich keine Schreibberechtigung habe.

Du sagst also, ich müßte bereits in der Lage sein, Änderungen in der eigenen Fork einzutragen und dann einen Pullrequest zu stellen?

agentmnm commented 12 years ago

ich habe gerade in Youtube ein Howto angeschaut, und bei mir kam definitv kein Fenster in dem das Hardcore-Cloning angezeigt wurde.

Ich arbeite mit Debian/Linux wheezy, und verwende iceweasel (firefox unbranded) als Browser. Vielleicht probier ich mal nen anderen Browser...

agentmnm commented 12 years ago

aha, jetzt hab ich den Fork des Arduino-Pedelec-Controller unter meinem user gefunden. Es hat halt ein wenig gedauert!

Ich kann es jetzt über ssh auf meinen Rechner clonen. Das ist schön.

Leider ist die Version ja mittlerweile veraltet.

Wie bringe ich die auf den neuesten Stand?

agentmnm commented 12 years ago

unter Googles Chromium funktioniert alles wie es soll!

Den alten Fork mußte ich löschen - ein Aktualisieren ist anscheinend nicht möglich.

Jetzt habe ich Jenkies Repo geforkt (na das ist tolles deutsch!) und werde die kleine Änderung durchführen und dann versuchen einen Pull-Request zu stellen.

Wenn alles gut geht, werde ich schweigen ;-D

agentmnm commented 12 years ago

ich schweige hier :P