Skip to content

Decoupling#1

Merged
Pato-99 merged 12 commits into
mainfrom
decoupling
Feb 20, 2026
Merged

Decoupling#1
Pato-99 merged 12 commits into
mainfrom
decoupling

Conversation

@VitHrncir

Copy link
Copy Markdown
Contributor

Kompletně jsem rozdělil data.py a pinout_diagram.py

Přidal jsem extra příklad s espLAN

Added a "Templates" folder which contains a vesion of the script (and its data.py) which is does not require modification between devkits.
fixed small mistake
Added example with the new data.py and pinout_diagram.py

NOT COMPLETE OR CORRECT, JUST AN EXAMPLE!!!!!
@Pato-99 Pato-99 self-requested a review November 19, 2025 16:24
@Pato-99

Pato-99 commented Nov 24, 2025

Copy link
Copy Markdown
Collaborator

Prosim necommituj soubory a adresare typu .idea/ .venv/ a soubory, ktere se generuji. Ty do VCS nepatri :)

@Pato-99

Pato-99 commented Nov 24, 2025

Copy link
Copy Markdown
Collaborator

Potrebujeme vytvorit .gitignore, aby se ignorovaly automaticky. Zakladni muze vypadat takto:

pycache/
.venv/
.idea/

@Pato-99

Pato-99 commented Nov 25, 2025

Copy link
Copy Markdown
Collaborator

Nez budes pokracovat dal, zkus se zamerit na tyto body :)

mit jenom jeden soubor styles.css

Aktualne je v kazdem podadresari vlastni .css soubor, ale obsah je vicemene stejny. Udelej tedy jednotny style.css umisti ho do rootu repa (stejna uroven jako .gitignore)

mit jenom jeden pinout_diagram.py script

Podobne jako u predchoziho bodu. Cilem by melo byt vytvorit jeden univerzalni skript, kteremu na vstupu das specifikaci (data.py + png) a on ti z toho vytvori .svg/.png. Mit specifikace ulozene v data.py souboru zacne byt trochu kostrbate kvuli tomu, ze budes muset dynamicky menit odkud se importuje, ale ne nemozne. Ja bych preci jen preferoval mit misto toho specifikaci v JSON. Ve vetvi develop jsem vytvoril example jak by to mohlo vypadat.

@VitHrncir

Copy link
Copy Markdown
Contributor Author

Nez budes pokracovat dal, zkus se zamerit na tyto body :)

mit jenom jeden soubor styles.css

Aktualne je v kazdem podadresari vlastni .css soubor, ale obsah je vicemene stejny. Udelej tedy jednotny style.css umisti ho do rootu repa (stejna uroven jako .gitignore)

mit jenom jeden pinout_diagram.py script

Podobne jako u predchoziho bodu. Cilem by melo byt vytvorit jeden univerzalni skript, kteremu na vstupu das specifikaci (data.py + png) a on ti z toho vytvori .svg/.png. Mit specifikace ulozene v data.py souboru zacne byt trochu kostrbate kvuli tomu, ze budes muset dynamicky menit odkud se importuje, ale ne nemozne. Ja bych preci jen preferoval mit misto toho specifikaci v JSON. Ve vetvi develop jsem vytvoril example jak by to mohlo vypadat.

Ok. Sorry commitnul jsem ještě než jsem si to přečetl. Začnu na tom pracovat jakmile dokončím akutní záležitosti.

@VitHrncir

VitHrncir commented Dec 10, 2025

Copy link
Copy Markdown
Contributor Author

Nez budes pokracovat dal, zkus se zamerit na tyto body :)

mit jenom jeden soubor styles.css

Aktualne je v kazdem podadresari vlastni .css soubor, ale obsah je vicemene stejny. Udelej tedy jednotny style.css umisti ho do rootu repa (stejna uroven jako .gitignore)

mit jenom jeden pinout_diagram.py script

Podobne jako u predchoziho bodu. Cilem by melo byt vytvorit jeden univerzalni skript, kteremu na vstupu das specifikaci (data.py + png) a on ti z toho vytvori .svg/.png. Mit specifikace ulozene v data.py souboru zacne byt trochu kostrbate kvuli tomu, ze budes muset dynamicky menit odkud se importuje, ale ne nemozne. Ja bych preci jen preferoval mit misto toho specifikaci v JSON. Ve vetvi develop jsem vytvoril example jak by to mohlo vypadat.

Problém s odělováním .css souboru je, že někdy se musí .css měnit. Navrhuji do .JSON souboru dát nějaký variable, který by sloužil jako flag pro to, který .css soubor využít.

Tento "custom css" problém jsem řešil teď s ESPInk, který vyžaduje extra barvy pro legendu

Další problém: 30px mezi piny. Ne pro každou hodnotu musí být tato hodnota vhodná, např. https://www.laskakit.cz/laskakit-espink-shelf-2-9-esp32-e-paper/

@Pato-99

Pato-99 commented Dec 10, 2025

Copy link
Copy Markdown
Collaborator

Css file do JSON specifikace úplně nepatří. Vadí, že by ty definice custom barev měly všechny desky? Pokud ano a nenapadá tě jiné řešení, tak ho do JSON specifikace přidej.

Co se týče 30px rozteče (už nevím kam přesně jsem to psal). Tak to bylo myšleno tak, aby všechny obrázky měly stejné měřítko. Daná deska je speciální případ, kde není klasický header a možná by se hodilo pro tento typ vytvořit vlastní classu odvozenou PinGroup.

…which acts as the master script for all diagrams.
… file as an argument and uses it to generate the pinout diagram

example usage:

python3 CreateDiagramFromJsonPhotoAndOptionallyACSSFileIfYouAddItIntoTheJSONFile.py ./NewFiles/MyJson.json

Names of functions as well as the python file are subject to change during the final cleaup process before getting merged into main
example usage:

python3 PinoutGenerator.py ./espInk/data.json
@Pato-99

Pato-99 commented Feb 13, 2026

Copy link
Copy Markdown
Collaborator

Asi si toho odmazal moc

@VitHrncir

Copy link
Copy Markdown
Contributor Author

Asi si toho odmazal moc

Kde je problém?

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nepatri do repa

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nepatri do repa

Comment thread Templates/diagram.svg Outdated

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

toto se generuje -> nepatri do repa

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nepatri do repa

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nepatri do repa

Comment thread .idea/.gitignore Outdated

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

toto prosim nedavat do repa

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

toto prosim nedavat do repa

Comment thread .idea/misc.xml Outdated

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cele .idea prosim nedavat do repa

Comment thread esp32_lan/venv/bin/Activate.ps1 Outdated

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

venv/ nedavat do repa

Comment thread espInk/diagram.svg

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

toto nedavat do repa

@Pato-99 Pato-99 merged commit 74de1d2 into main Feb 20, 2026
1 of 2 checks passed
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

Successfully merging this pull request may close these issues.

2 participants