Re: VW ID.3 SoC
Verfasst: Di Mär 02, 2021 6:28 am
Hast Du mal Google bemüht? Ist err erste Treffer.
Ja - bevor man so etwas postet, sollte man sich m.E. einbringen und selbst am Problem arbeiten, um es zu eliminieren . Rumnörgeln bringt doch nichts.Freddi69 hat geschrieben: ↑Di Mär 02, 2021 9:14 am .... Ganz zu schweigen von dem schlechten Stil der git-Kommentare (etwa https://github.com/snaptec/openWB/commi ... 25838b9d0d - da hätte man auch eine Begründung angeben können, warum das reverted wird).
Ich werde wahrscheinlich gleich wieder zerrissen, was ich mir hier erlaube...aber es gibt schon Dinge wie best practices und guten Stil in der Softwareentwicklung/Codeverwaltung.
Wie gesagt, ich hatte damals versucht "mal schnell zu schauen wo es hakt" als bei mir der SoC des ID.3 nicht mehr lief. Konnte ich aber nicht, weil da ein Binary eingecheckt war. Und dessen Stand kannte ich nicht, weil nicht ersichtlich, welche Version es genau war. Ich habe in der Tat vermutet, dass es sich um das verlinkte Projekt von AndiG handelt, aber weiter kam ich erstmal nicht, weil die genannten Informationen fehlen. Und ja, ich bin erst seit 2 Wochen dabei, mich in die Materie (Wallbox, Steuerung, etc) einzuarbeiten. Aber schlechte git-Kommentare / schlechte Nachvollziehbarkeit haben nichts mit gewachsenem Code zu tun. Ich kann euch gerne unterstützen, indem ich den Master-Branch rebase und die git-Kommentare bereinige. Glaube aber kaum, dass ihr das wollt. Zumal ich wie gesagt erst seit 2 Wochen nebenbei dabei bin, mich hier und da etwas einzulesen. Ich hab die notwendigen Informationen ja noch gar nicht.aiole hat geschrieben: ↑Di Mär 02, 2021 9:22 amJa - bevor man so etwas postet, sollte man sich m.E. einbringen und selbst am Problem arbeiten, um es zu eliminieren . Rumnörgeln bringt doch nichts.Freddi69 hat geschrieben: ↑Di Mär 02, 2021 9:14 am .... Ganz zu schweigen von dem schlechten Stil der git-Kommentare (etwa https://github.com/snaptec/openWB/commi ... 25838b9d0d - da hätte man auch eine Begründung angeben können, warum das reverted wird).
Ich werde wahrscheinlich gleich wieder zerrissen, was ich mir hier erlaube...aber es gibt schon Dinge wie best practices und guten Stil in der Softwareentwicklung/Codeverwaltung.
btw
Gewachsener Code, der tausende Installationen bedient, wird immer schlechter aussehen, als neuer code.
Deshalb wird parallel an oWB 2.x gearbeitet, um sich den neuen Anforderungen mit "sauberer" code-Basis zu stellen. Re-writes sind absolut üblich.
Noch eine Ergänzung: Ich möchte nicht, dass dies als rumnörgeln verstanden wird. Ich sehe es als konstruktive Kritik. Oder als qualitätsverbessernde Maßnahme. Ich würde mich freuen, wenn man mich drauf hinweist, dass da Informationen zur Nachvollziehbarkeit fehlen.aiole hat geschrieben: ↑Di Mär 02, 2021 9:22 amJa - bevor man so etwas postet, sollte man sich m.E. einbringen und selbst am Problem arbeiten, um es zu eliminieren . Rumnörgeln bringt doch nichts.Freddi69 hat geschrieben: ↑Di Mär 02, 2021 9:14 am .... Ganz zu schweigen von dem schlechten Stil der git-Kommentare (etwa https://github.com/snaptec/openWB/commi ... 25838b9d0d - da hätte man auch eine Begründung angeben können, warum das reverted wird).
Ich werde wahrscheinlich gleich wieder zerrissen, was ich mir hier erlaube...aber es gibt schon Dinge wie best practices und guten Stil in der Softwareentwicklung/Codeverwaltung.
btw
Gewachsener Code, der tausende Installationen bedient, wird immer schlechter aussehen, als neuer code.
Deshalb wird parallel an oWB 2.x gearbeitet, um sich den neuen Anforderungen mit "sauberer" code-Basis zu stellen. Re-writes sind absolut üblich.
Ah, ein separates Projekt. Vielen Dank für den Hinweis. Dort gibts ja leider gar keine Releases.openWB hat geschrieben: ↑Di Mär 02, 2021 9:52 am Das gibt dir dann sicherlich Aufschluss:
https://github.com/andig/evcc-soc
Ja, das ist eben der Nachteil, freien Programmierens. Dokuarbeit kostet Zeit und wird nicht gern gemacht.Freddi69 hat geschrieben: ↑Di Mär 02, 2021 9:59 amAh, ein separates Projekt. Vielen Dank für den Hinweis. Dort gibts ja leider gar keine Releases.openWB hat geschrieben: ↑Di Mär 02, 2021 9:52 am Das gibt dir dann sicherlich Aufschluss:
https://github.com/andig/evcc-soc