kumor
Období: Týden 12/2026 (od 22.03.) Skupina: tria
1. Splnění základních podmínek
| Podmínka | Status |
|---|---|
| Alespoň 3 commity v daném týdnu | ✅ SPLNĚNO |
| Rozestup alespoň 12h mezi 3 commity | ✅ SPLNĚNO |
| Projektová dokumentace (README, projekt.md) | ❌ NESPLNĚNO |
Vypočtený Index za tento týden: 264
Týdenní Index Náročnosti dodaných skriptů (1-5): 4
2. Kvantitativní přehled úprav
| Metrika | Hodnota |
|---|---|
| Počet všech commitů | 5 |
| Počet změněných souborů | 24 |
| Počet přid./odeb. řádků kódů | 2387 |
Přehled commitů
- 2026-03-23 07:07: weather_app.py moved to weather folder
- 2026-03-23 07:09: guitar_app.py moved to guitar folder and added documentation and configuration files for guitar app
- 2026-03-23 07:38: created test files for all projects
- 2026-03-26 19:55: created mod update notifier for satisfactory
- 2026-03-28 13:39: Add files via upload
3. Hodnocení asistentem AI (Logika a úsilí)
Ahoj, jako tvůj asistent jsem prošel tvůj repozitář a kód za uplynulý týden. Je vidět, že jsi odvedl velký kus práce na integraci externího API, což oceňuji, ale v organizaci projektu máš značné mezery, které musíme napravit.
Zde je tvé hodnocení:
1. Smysluplnost popisů u commitů [2 body]
Většina tvých commitů je věcná a popisuje, co se v projektu změnilo (např. přesun souborů do složek nebo vytvoření testovacích souborů). Nicméně hned první commit „Add files via upload“ je zcela neříkající. V profesionální praxi i ve škole se snaž takovým zprávám vyhýbat a vždy popiš, co daný kód reálně dělá.
2. Smysluplnost komentářů v kódu [4 body]
Oceňuji používání dokumentačních řetězců (docstrings) na začátku skriptů jako find_api.py nebo full_diagnostic.py. Pomáhá to rychle pochopit účel souboru. V samotném kódu (např. v to_do_list.py) by ale neškodilo pár komentářů vysvětlujících logiku, zejména u práce s indexy.
V souboru to_do_list.py máš:
import json je zde zbytečný, protože v celém skriptu s JSONem vůbec nepracuješ.
3. Kvalita a srozumitelnost dokumentace a struktura repozitáře [0 bodů]
V této kategorii musím body strhnout úplně, protože jsi porušil základní pravidla organizace repozitáře:
- Soubory v rootu: V hlavní složce máš 14 volně ložených souborů (např. /update_notifier.py, /full_diagnostic.py, /to_do_list.py). Tyto soubory měly být v příslušných složkách projektů.
- Nonsens názvy: V repozitáři se nachází složka bordel, což je nepřípustné. Dále soubory jako /bordel/H, /bordel/pp.py nebo /kumor/kumor/pp.py jsou naprosto nesrozumitelné.
- Chyby v dokumentaci: V souboru README_MOD_NOTIFIER.md (i dokumentace.md) máš nadpis „Konfiguracja“, což vypadá na polštinu nebo překlep. Navíc máš v rootu dva identické soubory dokumentace pod různými názvy, což je matoucí.
4. Logika řešení a přemýšlení studenta v Pythonu [7 bodů]
Tady tě musím pochválit. Práce s API ficsit.app není úplně triviální, což dokazují tvoje diagnostické skripty.
- Analýza problému: Skripty find_api.py a full_diagnostic.py ukazují, že nad problémem přemýšlíš – když ti API vracelo chybu 404 (viz mod_notifier.log), nezačal jsi jen náhodně měnit kód, ale napsal sis nástroje na průzkum endpointů a struktury JSON dat.
- GraphQL: Pokus o použití GraphQL v find_api.py svědčí o pokročilejším zájmu o technologii.
- Tkinter: to_do_list.py je sice jednoduchý, ale funkční základ GUI aplikace.
Logika je tvá silná stránka, jen ji musíš podpořit lepším pořádkem.
Celkem získáno bodů: [13 / 65%] ze 20 možných
Doporučení pro příští týden:
1. Přesuň všechny soubory z hlavní složky do logicky pojmenovaných podsložek.
2. Smaž složku bordel a soubory s jednopísmennými názvy.
3. Sjednoť dokumentaci a odstraň duplicitní soubory.
DIFFICULTY_INDEX: [4]