-
-
Notifications
You must be signed in to change notification settings - Fork 225
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
MI integration in 0.5.82 #664
Conversation
Seems to still work for 3rd Gen...
sieht sehr gut aus! Kommen auch Daten? |
src/hm/miPayload.h
Outdated
} | ||
*/ | ||
DPRINTLN(DBG_INFO, F("Inverter ") + String(iv->id) + F(": status msg ") + p->packet[0]); | ||
} else if (p->packet[0] == (0x12 + ALL_FRAMES)) { // MI status response to 0x11 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ist das richtig? das steht 0x12 + ...
muss das nicht 0x11 heißen?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Das mit 0x92 paßt an der Stelle.
Die "kleinen MI" sind da etwas seltsam, weil zwei Antworten auf eine Anfrage kommen. Die "Großen" (4-kanaligen) sind dann wieder anders...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Und ja, es kommen Antworten.
Allerdings (zumindest auf den ersten Blick) nicht mehr so "bilderbuchmäßig" wie in der vorherigen Version, vermutlich weil es noch an den repeats der requests auf allen Kanälen bei Ausbleiben der Antwort fehlt.
seems to work... Data is unclear (no packets received yet?)
compiles... for the rest still untested
Seems to still work for 3rd Gen...