WIP: Prehlednejsi Hodnotitko (tabulka) #31
Open
ledoian
wants to merge 2 commits from prehlednejsi_hodnotitko
into master
Loading…
Reference in new issue
There is no content yet.
Delete Branch 'prehlednejsi_hodnotitko'
Deleting a branch is permanent. It CANNOT be undone. Continue?
WIP
Prehlednejsi Hodnotitkoto Prehlednejsi Hodnotitko (tabulka) 2 years ago'reseni_od': terminy[-2] if rocnik is None else terminy[0],
'reseni_do': terminy[-1],
'neobodovane': False,
'barvicky': True,
Pokud tohle má dělat, že
🎨?
bude checked, pak to u mě nějak nefunguje…if resitel not in tabulka[problem]:
tabulka[problem][resitel] = SouhrnReseni(pocet_reseni=1, posledni_odevzdani=cas, body=body)
tabulka[problem][resitel] = [(reseni, hodnoceni)]
soucty[problem][resitel] = hodnoceni.body or 0 # Neobodované neřešíme
A jak tedy poznám neobodované? To si je musím nechat vyjet zvlášť?
Á jasně, to je v součtu.
for resitel in self.resitele:
dostal_body = False
resiteluv_radek = []
resiteluv_radek: list[tuple[Decimal,list[tuple[m.Resitel, m.Hodnoceni]]]] = [] # podle pořadí v self.problemy
Tady má být
m.Reseni
, pokud se nepletu (místom.Resitel
).Taky si myslím :-) Jen tak pro zábavu jsem zkusil na to poštvat typovou kontrolu (mypy v1.3.0), ale ta nic použitelného říct neumí, protože Django nemá typové anotace :-/
(Existuje nějaký komunitní projekt, který je přidává: https://github.com/typeddjango/django-stubs. Možná někdy prozkoumám…)
LGTM
def pridej_reseni(resitel, hodnoceni):
problem = hodnoceni.problem
body = hodnoceni.body
cas = hodnoceni.reseni.cas_doruceni
Body a cas jsou nepoužité. Jinak ten kód asi dává smysl.
Tohle jde prostě zavřít? Vypadá to mergenutě…
Ale není ještě 100% funkční (nesvítí zaškrtnuto, přestože jsou zaplé).
A myslím, že jsme se bavili nad lepšími barvami… (Třeba nebílými.)
Ha, vidíš, i forgor…
Prehlednejsi Hodnotitko (tabulka)to WIP Prehlednejsi Hodnotitko (tabulka) 2 months agoWIP Prehlednejsi Hodnotitko (tabulka)to WIP: Prehlednejsi Hodnotitko (tabulka) 2 months agoReviewers