<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">Am 9. Oktober 2013 13:11 schrieb Peter Evertz <span dir="ltr"><<a href="mailto:leo2@pec.homeip.net" target="_blank">leo2@pec.homeip.net</a>></span>:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Am 09.10.2013 01:54, schrieb Thorben Thuermer:<div class="im"><br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
On Tue, 8 Oct 2013 19:53:40 +0200 Jan Tamm <<a href="mailto:vz@tamms.net" target="_blank">vz@tamms.net</a>> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Ich<br>
hatte damals für die Software Solaranalyzer die Version von Peter etwas<br>
aufgebohrt. Diese Software benötigt die Daten mit festen Intervallen<br>
zwischen den Timestamps, genauer alle 300 Sekunden. Auch wenn man die<br>
Aggregierung einschaltet, dann passt es trotzdem nicht immer ganz, die<br>
Timestamps wandern einfach.<br>
<br>
Als Lösung wird der sleep Wert in der Sammel-Funktion etwas verändert und<br>
der Timestamp später gerundet, also verändert. Das funktionierte bei mir<br>
und bei Sollner11 problemlos. Mich persönlich würde das Runden selbst nicht<br>
stören, aber vielleicht andere also habe ich dem lieber einen neuen<br>
boolschen Parameter gegeben: aggfixedintervals.<br>
<br>
Falls niemand etwas gegen die Funktion einzuwenden hat, dann werde ich sie<br>
die Tage einmal nach der Anleitung als Pull Request verpacken<br>
</blockquote>
neue (einigermassen sinnvolle) features, immer gerne.<br>
wenn's durch die option komplett abzuschalten ist, umso besser.<br>
das liest sich so, als haettest du den code schon? auf github?<br>
warum schickst du nicht einen link zu deinen commits oder ein diff mit?<br>
dann koennen wir eher was dazu sagen.<br>
<br></blockquote></div></blockquote><div>Gerne, hier ist es: <a href="https://github.com/JT-DE/vzlogger">https://github.com/JT-DE/vzlogger</a> bzw. <a href="https://github.com/JT-DE/vzlogger/compare/volkszaehler:master...master">https://github.com/JT-DE/vzlogger/compare/volkszaehler:master...master</a> auch mit dem Einfügen des Identifiers in die Beispiel-Config beim S0 Meter. <br>
</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div class="im"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">

ansonsten wuerde ich sagen peter soll mal kurz was dazu sagen,<br>
er hat ja als autor des urspruenglichen codes sicher den besten<br>
ueberblick, vlt. noch ob's die wartbarkeit verschlechtert, etc..<br>
</blockquote></div>
Danke, danke! Das mir jetzt schon der ganze Code zugeordnet wird ist aber zu viel der Ehre.<br>
Meine Änderungen sind ja seit gestern im Haupt-Git angekommen, daher bin ich da also "raus". Also ganz normaler GIT Dreikampf: clone, fork, pullrequest.<br>
<br></blockquote><div>Das habe ich auf jeden Fall versucht und hoffentlich auch korrekt gemacht. Wie gesagt, Sollner11 hat diese Version schon seit einiger Zeit am Laufen, bei mir klappt es auch. Ich nutze aber nur den S0 Meter, aber _eigentlich_ habe ich an dem Metern ja nichts geändert, sollte also auch bei den anderen klappen.<br>
<br></div><div>Schöne Grüße<br>Jan<br></div></div></div></div>