WIP: Prehlednejsi Hodnotitko (tabulka) #31

Open
ledoian wants to merge 2 commits from prehlednejsi_hodnotitko into master
ledoian commented 2 years ago
Owner

WIP

WIP
ledoian added 1 commit 2 years ago
ledoian added 10 commits 2 years ago
ledoian added 1 commit 2 years ago
zelvuska changed title from Prehlednejsi Hodnotitko to Prehlednejsi Hodnotitko (tabulka) 2 years ago
zelvuska reviewed 2 years ago
'reseni_od': terminy[-2] if rocnik is None else terminy[0],
'reseni_do': terminy[-1],
'neobodovane': False,
'barvicky': True,
Poster
Owner

Pokud tohle má dělat, že 🎨? bude checked, pak to u mě nějak nefunguje…

Pokud tohle má dělat, že `🎨?` bude checked, pak to u mě nějak nefunguje…
zelvuska reviewed 2 years ago
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
Poster
Owner

A jak tedy poznám neobodované? To si je musím nechat vyjet zvlášť?

A jak tedy poznám neobodované? To si je musím nechat vyjet zvlášť?
Poster
Owner

Á jasně, to je v součtu.

Á jasně, to je v součtu.
zelvuska marked this conversation as resolved
zelvuska reviewed 2 years ago
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
Poster
Owner

Tady má být m.Reseni, pokud se nepletu (místo m.Resitel).

Tady má být `m.Reseni`, pokud se nepletu (místo `m.Resitel`).
Poster
Owner

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…)

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…)
zelvuska approved these changes 2 years ago
zelvuska left a comment

LGTM

def pridej_reseni(resitel, hodnoceni):
problem = hodnoceni.problem
body = hodnoceni.body
cas = hodnoceni.reseni.cas_doruceni
Poster
Owner

Body a cas jsou nepoužité. Jinak ten kód asi dává smysl.

Body a cas jsou nepoužité. Jinak ten kód asi dává smysl.
ledoian added 1 commit 2 years ago
Poster
Owner

Tohle jde prostě zavřít? Vypadá to mergenutě…

Tohle jde prostě zavřít? Vypadá to mergenutě…
Owner

Ale není ještě 100% funkční (nesvítí zaškrtnuto, přestože jsou zaplé).

Ale není ještě 100% funkční (nesvítí zaškrtnuto, přestože jsou zaplé).
Owner

A myslím, že jsme se bavili nad lepšími barvami… (Třeba nebílými.)

A myslím, že jsme se bavili nad lepšími barvami… (Třeba nebílými.)
Poster
Owner

Ha, vidíš, i forgor…

Ha, vidíš, i forgor…
zelvuska added 1 commit 2 months ago
zelvuska changed title from Prehlednejsi Hodnotitko (tabulka) to WIP Prehlednejsi Hodnotitko (tabulka) 2 months ago
zelvuska changed title from WIP Prehlednejsi Hodnotitko (tabulka) to WIP: Prehlednejsi Hodnotitko (tabulka) 2 months ago
zelvuska added 1 commit 2 months ago

Reviewers

zelvuska approved these changes 2 years ago
This pull request is marked as a work in progress.
This branch is out-of-date with the base branch
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.