mam/mamweb
11
0
Fork 0
Code Issues Pull requests 9 Projects Wiki Activity Actions

Vylepšení hodnotítka fix #1354 fix #1237 #20

Merged
zelvuska merged 25 commits from vylepseni_odevzdavatka into master 2023-06-12 22:17:08 +02:00
Conversation 3 Commits 25 Files changed 7 +16 -16
2 changed files with 16 additions and 16 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit d66effcd46 - Show all commits

23
seminar/models/odevzdavatko.py
View file

@ -1,4 +1,3 @@
import decimal
import os import os
import reversion import reversion
@ -15,6 +14,8 @@ from seminar.models import personalni as pm
from seminar.models import treenode as tm from seminar.models import treenode as tm
from seminar.models import base as bm from seminar.models import base as bm
from seminar.utils import vzorecek_na_prepocet, inverze_vzorecku_na_prepocet
@reversion.register(ignore_duplicates=True) @reversion.register(ignore_duplicates=True)
class Reseni(bm.SeminarModelBase): class Reseni(bm.SeminarModelBase):
@ -116,16 +117,6 @@ class Hodnoceni(bm.SeminarModelBase):
feedback = models.TextField('zpětná vazba', blank=True, default='', help_text='Zpětná vazba řešiteli (plain text)') feedback = models.TextField('zpětná vazba', blank=True, default='', help_text='Zpětná vazba řešiteli (plain text)')
# TODO najít správné místo
@staticmethod
def vzorecek_na_prepocet(body, resitelu):
return body * 3 / (resitelu + 2)
# TODO najít správné místo
@staticmethod
def inverze_vzorecku_na_prepocet(body: decimal.Decimal, resitelu) -> decimal.Decimal:
return round(body * (resitelu + 2) / 3, 1)
@property @property
def body_celkem(self): def body_celkem(self):
# FIXME řeším jen prvního řešitele. # FIXME řeším jen prvního řešitele.
@ -147,33 +138,33 @@ class Hodnoceni(bm.SeminarModelBase):
def body_neprepocitane(self): def body_neprepocitane(self):
if self.body is None: if self.body is None:
return None return None
return self.inverze_vzorecku_na_prepocet(self.body, self.reseni.resitele.count()) return inverze_vzorecku_na_prepocet(self.body, self.reseni.resitele.count())
@body_neprepocitane.setter @body_neprepocitane.setter
def body_neprepocitane(self, value): def body_neprepocitane(self, value):
zelvuska marked this conversation as resolved
ledoian commented 2023-02-06 22:23:33 +01:00
Review
Copy link

Tohle potenciálně generuje záporné body…

Tohle potenciálně generuje záporné body…
zelvuska commented 2023-02-06 22:58:36 +01:00
Review
Copy link

A to je špatně?

A to je špatně?
ledoian commented 2023-02-06 23:03:30 +01:00
Review
Copy link

Považuji to za dost neočekávatelné chování, zvlášť když jako org píšu body kladné. Obecně si myslím, že spíš strhávat body od již udělených nechceme (možná vyjma „ujelo mi bodování, zpětně měním počet bodů)…

Považuji to za dost neočekávatelné chování, zvlášť když jako org píšu body kladné. Obecně si myslím, že spíš strhávat body od již udělených nechceme (možná vyjma „ujelo mi bodování, zpětně _měním_ počet bodů)…
zelvuska commented 2023-02-06 23:08:52 +01:00
Review
Copy link

A jaké je tedy očekávané chování?

A jaké je tedy očekávané chování?
ledoian commented 2023-02-06 23:11:35 +01:00
Review
Copy link

Nedovolit to vůbec (a že má kdyžtak org napsat webařům, pokud nechápe proč)? Hodit aspoň alert(), že se o to org pokouší a že to bude vypadat blbě ve výsledkovce?

Nedovolit to vůbec (a že má kdyžtak org napsat webařům, pokud nechápe proč)? Hodit aspoň `alert()`, že se o to org pokouší a že to bude vypadat blbě ve výsledkovce?
zelvuska commented 2023-02-06 23:13:08 +01:00
Review
Copy link

Ale doteď tam žádná taková šaráda nebyla. (Otestováno, záporné body mohu zadat vesele.) A já na frontendu nepoznám, kolik za to už řešitel dostal…

Ale doteď tam žádná taková šaráda nebyla. (Otestováno, záporné body mohu zadat vesele.) A já na frontendu nepoznám, kolik za to už řešitel dostal…
ledoian commented 2023-02-06 23:23:25 +01:00
Review
Copy link

Ale doteď se ti nemohlo stát, že bys ty záporné body napsal omylem – musel bys explicitně zmáčknout mínus. Není potřeba orgy ochránit proti zlým úmyslům, ale proti dobrým chybám…

Ale doteď se ti nemohlo stát, že bys ty záporné body napsal omylem – musel bys explicitně zmáčknout mínus. Není potřeba orgy ochránit proti zlým úmyslům, ale proti dobrým chybám…
zelvuska commented 2023-02-06 23:33:23 +01:00
Review
Copy link

Přijde mi, že to v podstatě odchytává jen miniaturní část toho problému. To, když to zrovna vyjde záporné. Ale ve chvíli, kdy tam org zadá libovolný jiný nesprávný počet, tak to stejně nezjistíme. Tohle políčko prostě vyžaduje, aby tam org zadal správný počet.

Přijde mi, že to v podstatě odchytává jen miniaturní část toho problému. To, když to zrovna vyjde záporné. Ale ve chvíli, kdy tam org zadá libovolný jiný nesprávný počet, tak to stejně nezjistíme. Tohle políčko prostě vyžaduje, aby tam org zadal správný počet.
zelvuska commented 2023-02-06 23:34:38 +01:00
Review
Copy link

Jakoby souhlasím s tím, že tam ta obrana může být (přídám tam asi javascript, který vykřikne, pokud při načtení libovolná z těch hodnot je záporná).

Ale nepřijde mi, že by to nějak signifikantně zvyšovalo ochranu proti špatnému zadání.

Jakoby souhlasím s tím, že tam ta obrana může být (přídám tam asi javascript, který vykřikne, pokud při načtení libovolná z těch hodnot je záporná). Ale nepřijde mi, že by to nějak signifikantně zvyšovalo ochranu proti špatnému zadání.
if value is None: if value is None:
self.body = None self.body = None
else: else:
self.body = self.vzorecek_na_prepocet(value, self.reseni.resitele.count()) self.body = vzorecek_na_prepocet(value, self.reseni.resitele.count())
@property @property
def body_neprepocitane_celkem(self): def body_neprepocitane_celkem(self):
if self.body_celkem is None: if self.body_celkem is None:
return None return None
return self.inverze_vzorecku_na_prepocet(self.body_celkem, self.reseni.resitele.count()) return inverze_vzorecku_na_prepocet(self.body_celkem, self.reseni.resitele.count())
@body_neprepocitane_celkem.setter @body_neprepocitane_celkem.setter
def body_neprepocitane_celkem(self, value): def body_neprepocitane_celkem(self, value):
if value is None: if value is None:
self.body = None self.body = None
else: else:
self.body_celkem = self.vzorecek_na_prepocet(value, self.reseni.resitele.count()) self.body_celkem = vzorecek_na_prepocet(value, self.reseni.resitele.count())
@property @property
def body_max(self): def body_max(self):
if self.body_neprepocitane_max is None: if self.body_neprepocitane_max is None:
return None return None
return self.vzorecek_na_prepocet(self.body_neprepocitane_max, self.reseni.resitele.count()) return vzorecek_na_prepocet(self.body_neprepocitane_max, self.reseni.resitele.count())
@property @property
def body_neprepocitane_max(self): def body_neprepocitane_max(self):

9
seminar/utils.py
View file

@ -1,6 +1,7 @@
# -*- coding: utf-8 -*- # -*- coding: utf-8 -*-
import datetime import datetime
import decimal
from django.contrib.auth import get_user_model from django.contrib.auth import get_user_model
from django.contrib.auth.decorators import permission_required, \ from django.contrib.auth.decorators import permission_required, \
@ -44,6 +45,14 @@ AnonymousUser.je_org = False
AnonymousUser.je_resitel = False AnonymousUser.je_resitel = False
def vzorecek_na_prepocet(body, resitelu):
return body * 3 / (resitelu + 2)
def inverze_vzorecku_na_prepocet(body: decimal.Decimal, resitelu) -> decimal.Decimal:
return round(body * (resitelu + 2) / 3, 1)
class FirstTagParser(HTMLParser): class FirstTagParser(HTMLParser):
def __init__(self, *args, **kwargs): def __init__(self, *args, **kwargs):
self.firstTag = None self.firstTag = None