search

RFD-FHEM / SIGNALESP

SIGNALduino direclty on ESP8266
GNU General Public License v3.0
15 stars 8 forks source link

merge dev-cc1101-cb #13

Closed habeIchVergessen closed 6 years ago

habeIchVergessen commented 6 years ago

mit einem pull request sieht man gut die Änderungen (auch bzgl. SDC_WRITE-Aufrufe und Typkonvertierungen). Ich würde gerne die Änderungen der Libs BitStore und SignalDecoder übernehmen ohne meine Änderungen zu verlieren.

z.B.

sidey79 commented 6 years ago

Ich verstehe diesen PR leider nicht...

:(

habeIchVergessen commented 6 years ago

das Austauschen der Libs wird nicht reichen, damit die Sourcen funktionieren!

wenigstens "comp_cc1101 in SIGNALESP.ino" wird ja drin sein, oder?

sidey79 commented 6 years ago

Was hat jetzt dazu geführt dem PR zu schließen?

habeIchVergessen commented 6 years ago

scheinbares Desinteresse. bzgl. Manchester gab es auch keine Reaktion

sidey79 commented 6 years ago

Desinteresse nicht. Ich schaffe es nur nicht so viele Baustellen gleichzeitig zu handeln.

Meine Idee ist jetzt die Lib ESP und Arduino zusammen zu bringen. Ich will nur noch eine pflegen :)

Dann ist dieser Pull Request auch sehr umfangreich. Das macht es sehr schwer für mich, diesen zu verstehen Z.B. Modulo durch bittergleich ersetzten. Verstehe ich, aber das löst kein Problem. Der compilieren macht hoffentlich immer den gleichen Code aus der Anweisung.

sidey79 commented 6 years ago

Ich habe mich gestern wieder dem cc1101 support gewidmet. Klappt leider nicht. In deiner Version sind schon wieder so viele Änderungen. Was hat es denn damit auf sich?

habeIchVergessen commented 6 years ago

hab deine Änderungen gemerged. Senden klappt jetzt nicht mehr.

bzgl. Liste der commits: das sind die offenen, die seit dem ersten Pull Request auf master entstanden sind. Als wir auf dev-cc1101 umgezogen sind, sind alle offen geblieben. auf dev-cc1101-cb ebenso. Dafür habe ich leider keine Erklärung.

z.B. commit https://github.com/RFD-FHEM/SIGNALESP/pull/13/commits/133d3ee0b085043c0f14a50bbec2a527ff06b3d9 hattest du per cherry pick übernommen. Kann man nur an der Liste der Autoren sehen.

Die Änderungen in signalDecoder.cpp/h sind deine letzen Änderungen + Bugfixes, Readme.md geschenkt, SIGNALESP.ino und cc1101.h musst du wohl oder übel mindestens 1x durchgehen. Wobei es im Wesentlichen fehlende Funktionalität (Vergleich zu master) ist.

sidey79 commented 6 years ago

Ich habe gestern Abend versucht das SPI Problem zu lösen. Ich habe aktuell keine Idee, wie ich das debuggen könnte

sidey79 commented 6 years ago

So ich habe jetzt ein paar Sachen übernommen, damit funktioniert der cc1101. Ich verstehe es aber nicht, wieso das so ist.

habeIchVergessen commented 6 years ago

D1 und D2 mag travis garnicht

sidey79 commented 6 years ago

Stimmt. Beim ESP Generic fehlen diese Defines. Kann ich auch verstehen, das kann ja jeder selbst definieren.

habeIchVergessen commented 6 years ago

"Alles andere sind gut gemeinte Forks."