-
Notifications
You must be signed in to change notification settings - Fork 122
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
Added diagram for available charging power calculation #472
base: main
Are you sure you want to change the base?
Added diagram for available charging power calculation #472
Conversation
…guration/img/available_charging_power_en.png
Closed temporarily, to await review/clarification in the discussion evcc-io/evcc#10763. |
You could even set it into "draft" state. |
Thanks, did that. |
…guration/img/available_charging_power_en.png
…guration/img/inputs_outputs_en.png
…guration/img/evcc-inner-workings.drawio
…guration/img/available_charging_power_en.png
…guration/img/evcc-innards.drawio
…guration/img/inputs_outputs_en.png
Complete the rework now. Diagrams are now converted to drawio, and I expanded the explanation of the solar mode and put it into a new page in the guides section. Any feedback on the new content/structure/approach is appreciated! |
I am afraid I need some help here. So it seems some links to the newly created page do not work. I basically created a new file Anything else that I could be doing wrong? Is there a way I can run this checks myself before submitting the PR? Here is the error from the build process:
|
Hi @TheNinth7, großen Respekt für das Zusammentragen bzw. "Reverse-Engineeren", das du gemacht hast. Ich hab, deine Blog-Artikel gelesen und werd die auch gleich mal auf der Startseite verlinken. Ich glaub’ auch, dass es wertvoll ist, dieses Level an Content für "Advanced Nutzer" in der Doku zu haben. Also etwas für Menschen, die genau wissen wollen, wie das funktioniert, aber nicht den Schritt in den Source-Code und eigenes Debugging gehen wollen. Ich hab allerdings auch einige Bedenken, wenn wir so etwas in die Doku aufnehmen:
Zur Struktur: Ich würde vorschlagen, einen neuen Menüpunkt auf der Hauptebene hinzuzufügen, unter dem wir diesen Inhalt mit höherem Detaillevel ansiedeln. Vorschlag:
Ich hab das hier mal Unterhalb dieses Menüpunkts würde ich am liebsten kleine Seiten haben, die sich mit einem Aspekt auseinandersetzen. Das ist nicht immer einfach, weil das natürlich alles zusammenspielt, aber hilft mMn. beim Strukturieren und Erfassen.
Diese Seiten könnten/sollten wir dann aus der Nutzer-Doku immer als "weiterführende Ressource" verlinken, wo es passt. Das ist jetzt einiges an Änderung und es tut mir auch Leid, dass ich erst so spät in diese Diskussion einsteige. Hoffe aber, dass du die Punkte bzw. meine Bedenken zum jetzigen Vorschlag verstehst. Kannst du dir die skizzierte Form vorstellen? Zu den Grafiken: die Jetzigen wird man nicht 1:1 in Marmaid überführen können. Das Tool ist aber relativ mächtig, unterstützt einige unterschiedliche Diagrammtypen und hat einen Live-Editor zum Ausprobieren. Kannst du dir vorstellen, das zur Illustration zu verwenden? |
Ja, kann deine Bedenken nachvollziehen! Das Thema ist größer geworden anfänglich erwartet, ich habe naiv mit einem Diagramm angefangen, das wurde dann immer größer und es kamen weitere Aspekte dazu die nicht in das eine Diagram gepasst haben. ;-) Die Struktur wie du sie vorschlägst macht auf jeden Fall Sinn, das kann ich so umbauen. Bei den Diagrammen bin ich mir noch unsicher, die sind sicher das größte "Problem". Hab mir mal die Mermaid-Doku mit den verschiedenen Diagrammtypen angeschaut und bin mir noch nicht sicher wie man die verschiedenen Sachverhalte damit eingänglich darstellen kann. Vielleicht muss ich aber nur ein wenig umdenken, ich schau mir das mal näher an und melde mich dann wieder. |
Super! Bist du eigentlich im Slack? Ping mich da sonst bei Fragen auch gerne direkt an.
Ja, ich verstehe was du meinst. Vmtl. muss man die Punkte, die in den aktuelle Grafiken erklärt werden noch weiter runterbrechen, aufteilen bzw. auf den Kern reduzieren damit das in diesem Format gut funktioniert. |
Schönen Nachmittag! Hab jetzt gerade einen neuen Entwurf committed, mit den mit @naltatis besprochenen Kommentaren.
Freue mich über Kommentare! |
Super. Ich schau mir das bei Gelegenheit in Ruhe an. |
Added a diagram to the manipulation options section of the evcc.yaml page, explaining how the different paramters and inputs are used in the calculation of available charging power.