Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

logo fehlt, Offset für DHT #70

Closed
laha83 opened this issue Feb 6, 2022 · 15 comments
Closed

logo fehlt, Offset für DHT #70

laha83 opened this issue Feb 6, 2022 · 15 comments

Comments

@laha83
Copy link

laha83 commented Feb 6, 2022

Wenn ich useAktor auf 0 setze, aber einen DHT22 per gpio installiere wird die Seite nicht angeziegt.
Wenn ich useAktor auf 1 setze, wird die Übersichtsseite für die Aktoren und Temperatursensoren erzeugt.
Wie kann n ich einen Offset für den DHT22 eingeben? Der angezeigte Wert ist zu hoch.

nischram added a commit that referenced this issue Feb 6, 2022
- Issue #70 Fehlerkorrektur der Anzeige und Ergänzung eines Offset für DHT Temperatursensor
- Tippfehler in der README behoben
@nischram
Copy link
Owner

nischram commented Feb 6, 2022

Hey,
das mit der Anzeige ist ein Fehler gewesen, ich hatte im Quellcode in der Auswertung useAktor und useDHT mit "und" Bedingung verknüpft, was aber "oder" sein muss.
Ein Offset war Grundsätzlich bislang in meine Code noch nicht möglich. Ich habe da mal was eingebaut, in den Parametern ist jetzt ein Offset möglich.

#define DHT1_OFFSET_TEMP            0.0                     // Offset Temperatur
#define DHT2_OFFSET_TEMP            0.0                     // Offset Temperatur
#define DHT3_OFFSET_TEMP            0.0                     // Offset Temperatur
#define DHT4_OFFSET_TEMP            0.0                     // Offset Temperatur
#define DHT5_OFFSET_TEMP            0.0                     // Offset Temperatur

Leider kann ich auf die Schnelle die Funktion nicht testen, denn aktuell habe ich keinen DHT-Sensor an meinem Pi angeschlossen. So bin ich auf ein Feedback für den Offset in der Version 2.06 angewiesen.

Gruß Nico

@laha83
Copy link
Author

laha83 commented Feb 6, 2022 via email

@nischram
Copy link
Owner

nischram commented Feb 6, 2022

Leider fehlen wohl Bilder oder so. Bitte mit Anhängen nicht einfach auf die E-Mail Antworten sonder direkt auf der Seite den Beitrag erstellen.
Aber Achtung nicht einfach die gesamte parameter.h hochladen. z.T. sind vertrauliche Informationen enthalten.

Gruß Nico

nischram added a commit that referenced this issue Feb 6, 2022
- Kleine Anpassung für Issue #70 ohne Version
@nischram
Copy link
Owner

nischram commented Feb 6, 2022

Hi,

ich habe das jetzt nochmal durchgeschaut.
Das Symbol ist verschwunden weil irgendwo zwischen Version 2.01 und 2.04 eine Zeile gelöscht wurde, die aber noch benötigt wird.
Der Button konnte nicht genutzt werden weil ich in der screenSave.h auch noch den "und" Fehler gefunden habe.
Die Änderungen habe ich gerade Hochgeladen.

Die Anderen Fehler kann ich nicht nachvollziehen.
Der Offset wird dazugerechnet, habe mal 30.0 genommen:
06 02 22, 20:03 Microsoft Lens

Auch die Pfeile sind bei mir vorhanden.
Du kannst im Ordne E3dcGui/Image/Zahlen/ mal prüfen ob die pf0.ppm bis pf3n.ppm vorhanden sind.

Ich benötige weitere Informationen zu den Fehlermeldungen. Gerne auch Auszüge aus der Konsole wenn die Software von der Konsole gestartet wird.

Gruß Nico

@laha83
Copy link
Author

laha83 commented Feb 6, 2022 via email

nischram added a commit that referenced this issue Feb 7, 2022
@nischram
Copy link
Owner

nischram commented Feb 7, 2022

Hallo Lars,

die Meldung sagt das für dem PM (= LM = Leistungsmesser) der Staus nicht gelesen werden kann. Bei den Daten frage ich den Status vom PM mit dem Index 0 ab. Das ist bei meinem System so, bei einem Mini ist das auch wohl anders, so könnte es bei dir auch vielleicht ähnlich sein wie beim Mini.
Das könntest du mal testen. In der Datei "RscpMain.cpp" im Bereich der Zeile 147, am Ende die 0 durch eine 6 ersetzen.
Vorher:

       // request PM information
        SRscpValue PMContainer;
        protocol.createContainerValue(&PMContainer, TAG_PM_REQ_DATA);
        protocol.appendValue(&PMContainer, TAG_PM_INDEX, (uint8_t)0);
        protocol.appendValue(&PMContainer, TAG_PM_REQ_DEVICE_STATE);
        protocol.appendValue(&PMContainer, TAG_PM_REQ_ACTIVE_PHASES);

und nachher:

       // request PM information
        SRscpValue PMContainer;
        protocol.createContainerValue(&PMContainer, TAG_PM_REQ_DATA);
        protocol.appendValue(&PMContainer, TAG_PM_INDEX, (uint8_t)6);
        protocol.appendValue(&PMContainer, TAG_PM_REQ_DEVICE_STATE);
        protocol.appendValue(&PMContainer, TAG_PM_REQ_ACTIVE_PHASES);

Wenn ich es bei mir ändere, kommt die Meldung genau so wie bei dir.

Sollte das klappen muss ich mir was einfallen lassen wie ich das löse, aber mal erst abwarten ob es klappt.

Bei dem Offset musst du nochmal prüfen ob du tatsächlich die neuen parameter auch aus der "parameter.temp.h" in die "parameter.h" kopiert hast. Wenn die Parameter in der "parameter.h" nicht drin sind, wird ein Defaultwert mit 0 gestartet.
Das hat mich jetzt auf die Idee gebracht, dass ich für eine Prüfung ob ein Parameter fehlt, doch was in das Tool "copyPara" bringen kann. Ich habe das mal Hochgeladen, wenn du es damit prüfen willst muss du die Datei copyPara.c oder alles aktualisieren zum kompilieren reicht dann make copyPara. Gestartet wird das Programm dann mit ./copyPara

Gruß Nico

@maikheinrich
Copy link

Hallo Nico,

die unterschiedlichen PM Adressen 0 und 6 kommen dadurch Zustande ob ein Can Gateway verbaut wurde. Mit Gatway hat der Zähler die 6 und ohne die 0.

LG Maik

@laha83
Copy link
Author

laha83 commented Feb 7, 2022 via email

@nischram
Copy link
Owner

nischram commented Feb 7, 2022

@maikheinrich Hallo Maik,

Ja das hast du recht, könnte passen Intern im System ist es ein Leistungsmesser 0 und extern ein 6er. Zumindest passt es zum Konzept des S10-Mini. Bei meinem System noch nicht aber seit einiger Zeit kann man den internen Zähler wohl ausbauen um extern zu verbauen, vielleicht wird seit der Zeit die 6 als Standard vergeben.

@laha83 Hallo Lars, vielleicht kannst du da jetzt aufklären wo ist dein Zähler verbaut, im System oder außerhalb und was ist es für einer?

Aber egal wie ich muss mir dann jetzt wohl was einfallen lassen. Ja im ersten Schritt ist es wohl das beste ich baue eine Variable ein und bringe sie in die parameter.h Schöner wäre natürlich eine Automatische Lösung, aber mal ganz ehrlich bei so vielen Parameter die man in meinem Programm einstellen kann kommt es auf einen mehr auch nicht mehr an 😉! Wenn das ein Problem wäre müsste ich das ganze Konzept überarbeiten.
Ich mach das mal erst so und melde mich wieder, dauert auch nicht lange ist nichts wildes...

Viele Grüße Nico

nischram added a commit that referenced this issue Feb 7, 2022
…mmern

- Issue #70 Variable in der parameter.h aufgenommen um den Index unterschiedlich zu setzen
@nischram
Copy link
Owner

nischram commented Feb 7, 2022

Änderung schon hochgeladen...

@laha83
Copy link
Author

laha83 commented Feb 8, 2022 via email

@laha83
Copy link
Author

laha83 commented Feb 8, 2022 via email

nischram added a commit that referenced this issue Feb 8, 2022
Datei wurde nicht übertragen
@nischram
Copy link
Owner

nischram commented Feb 8, 2022

Hallo Lars,
ich habs gefunden... Ich weiß nicht wieso aber die Datei External/dht.h wurde beim ersten Upload für den Offset nicht hochgeladen, so ist natürlich klar das der Offset nicht klappt. Ich habe bei mir immer mit der richtigen getestet aber auf dem Server lag immer die Falsche Datei (ohne Offset)
Ich habe die geänderte Datei gerade nochmal hochgeladen. Sorry aber ich kann gerade nicht erklären warum das nicht geklappt hat.
Ich denke das das Problem mit der copyPara so jetzt egal ist.

VG Nico

@laha83
Copy link
Author

laha83 commented Feb 8, 2022 via email

@nischram
Copy link
Owner

nischram commented Feb 9, 2022

Sehr gut!
Somit schließe ich den Issue erstmal.

VG Nico

@nischram nischram closed this as completed Feb 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants