Labels Milestones

WIP: Nástroj pro plošné vyrábění problémů #17

Open

ledoian wants to merge 9 commits from zadavatko_problemu into master

pull from: zadavatko_problemu

merge into: mam:master

mam:ckeditor5

mam:doc_intro

mam:doc_zkratky

mam:dockerizace

mam:doku_predstava

mam:export_seznamu_prednasek

mam:kontrola-prav-orgu

mam:korektury-link-z-admina

mam:ledoian

mam:maillink_dev_ledoian

mam:master

mam:middleware_test

mam:notifikace_zpetne_vazby

mam:odstrel_modelu_odevzdavatko

mam:okenkove_obalky

mam:opravitko-checkbox

mam:pojmenovane_aplikace_broken

mam:prehlednejsi_hodnotitko

mam:upgrade_dokumentace

mam:v3

mam:vycisteni

mam:vyroci-fix

Conversation 4 Commits 9 Files Changed 4

ledoian commented 2 years ago

Owner

There is no content yet.

ledoian added 4 commits 2 years ago

7969f474d1 Začátek template zadávátka problémů [WIP]

9c06bc6f62 Merge remote-tracking branch 'gitea/master' into zadavatko_problemu

15c49dd43c Zadávátko: první nástřel vzhledu

Zatím nemám tušení, jak se ty věci zpracují a objeví, ale už máme nějaký
nástřel frontendu, se kterým můžeme interagovat a dodělávat ho.

6dd9ee98bb WIP: počátek backendu pro zadávátko

Zatím je to spíš TODO list než kód :-)

zelvuska commented 2 years ago

Owner

Nenene, to sis spletl. Cílem webařů je problémy řešit, ne hromadně vyrábět…

Nenene, to sis spletl. Cílem webařů je problémy řešit, ne hromadně vyrábět…

ledoian commented 2 years ago

Poster

Owner

Není Problém: řádek m.Problem.objects.update(stav=STAV_VYRESENY) řeší všechny Problémy :-P

Není Problém: řádek `m.Problem.objects.update(stav=STAV_VYRESENY)` řeší všechny Problémy :-P

ledoian commented 2 years ago

Poster

Owner

Tipy, na co si dát pozor:

• Přidání jen části problémů (některé přehlédnu / nemám rozmyšlené)
• Nutné vždy vyplnit číslo zadání (jinak se nenabízí v autocomplete a jinde)

(Případné další problémy přidávejte (pun not intended) do seznamu, zaškrtnuté jsou ty, které by kód měl už umět řešit.)

Tipy, na co si dát pozor: - [ ] Přidání jen části problémů (některé přehlédnu / nemám rozmyšlené) - [ ] Nutné vždy vyplnit číslo zadání (jinak se nenabízí v autocomplete a jinde) (Případné další problémy přidávejte (pun not intended) do seznamu, zaškrtnuté jsou ty, které by kód měl už umět řešit.)

ledoian added 137 commits 2 years ago

7ccaf87dfc Pokus o lepší orientaci v dokumentaci

fd3df6c344 WIP

e3ebb925e1 Reforma Makefilů WIP 1

75f388ddfb Fix symlink

9f78963e7f Poznámka o neplatnosti README

8b36e79f04 Makefile reforma vol.2 [WIP]

Už máme ty jednoduché targety (nebo byly smazány), zbývá dořešit několik
složitých.

657e73f2f0 Poznámka o důvodu existence Makefile

4f7c4057ea fix uvozovek

Thanks shellcheck

a9e7cd7af2 Deploy v novém kabátu

prosím čtěte…

d80babfd73 Smazán obsah make/sync*

Nic tam nebylo a teď je vidět ve `file *`, že tam nic není.

323477dbf2 Smazán venv_check

Je úplně k ničemu, když si nové skripty zapínají venv samy

f6d939fc31 Přebytečný středník

0458abb419 Kontroly prerekvizit do knihovny

5cfadf9236 Make: přesunuty sync_test_media

Taky jsem smazal nějaké komentáře v Makefile_old, aby nepřekážely.

5abd93d27f Zbytek sync_test

7c90c1ef5e sync_prod_flatpages

315dc97635 Add: další nápad do aprílu

4fd3a0c4f5 Hlasitější varování při použití make

31834b7f8a Zrušen `make/all`

Nic moc nedělal, tak jsem ho nahradil (IMHO lepší) dokumentací.

1a56ba02f3 Make: výchozí target se jmenuje `default`

`all` tam nemá žádnou sémantiku…

d086501095 make/lib: lokální proměnné

346079cee3 make/lib: Funkce install_everything je jen pro Gimliho

ac9cb53185 Hezčí SSH spojení

- Přidal jsem `set -euxo pipefail`, takže nejsou potřeba `&&` a obecně se
  to chová víc jako ostatní make skripty
- Venv se zapíná stejně jako v lokálních skriptech, takže se dá
  jednoduše změnit jeho cesta

e36410ee82 make/lib: komentáře kolem git-rev-parse

28760e2c4d Smazán install_venv

Všechny make skripty stejně vyrábí venv přes `ensure_venv` a protože web
nikdo nikdy nebude instalovat jinak (nebo když bude, tak asi ví, co
dělá), tak tohle nedává smysl spouštět.

5963089539 make/lib: .git může být i normální soubor (worktree)

e4c2eda955 make/run bere parametry

a390ef0329 make/lib: safe_checkout_branch stačí 1 parametr

1615e92159 safe_checkout_branch: porovnáváme proti upstreamu, ne aktuální verzi.

34e751fd69 fixup! make/lib: safe_checkout_branch stačí 1 parametr

b1e6448c3c safe_checkout_branch: varování

3fcc03b769 Shellcheck :-)

Ne že by mi na tom záviselo, ale kód to nezhoršuje a pokud to aspoň o
trochu zmenší šanci na nějaké přehlédnutí, tak je to OK.

A spoustu varování shellchecku jsem vyignoroval a nemíním plevelit kód
komentáři, o čem všem vím a on ne :-)

a184871bf9 make/lib: Když se to nepovede, tak budeme křičet

Uvědomil jsem si, že aktuálně nejde skoro vůbec poznat, jestli make
skript uspěl nebo ne. Tohle by se mělo vypsat, když se nepovede.

Víc by se mi líbilo, kdyby úspěšné doběhnutí ohlásilo "OK", ale to
neumím udělat bez nějakého zápatí skriptů.

(Resp. uměl bych: make/lib může být interpretr, který na začátku
zinicalizuje proměnné, pak natáhne příslušný skript a na konci ohlásí
OK. Ale přijde mi to trochu moc magické, takže pokud někdo nebude nějak
extra pro, tak to tak neudělám :-))

50eaaf9eb7 Předěláno Readme

Teď vypadá víc jako readme, na které chceš narazit v gitee, než jako pár
random tipů. Random tipy jsem přesunul do dokumentace

bac44b41f8 Dokumentace lokálního vývoje

Obsahuje tipy z Readme, obsahuje velkou část make/README.md.

Zatím nevím, co provedu s make/README.md, ale na něco třeba přijdu :-)

2ffd3bf445 Aspoň trochu relevantní make/README.md

A není to zas tak moc README, spíš intro :-/

25fcddfba8 Při deployi vyrob dokumentaci

e612899dc2 Requirements: myst_parser pro dokumentaci

dfc1267119 deploy: práva k dokumentaci

5c97735948 testweb heslo

df9e473686 fixup! testweb heslo

5985c4940f fix

66ce7d209d Méně hlučné testy

6b306f0849 ensure_venv v2

d468108188 ensure_web_installed

c17afece9d Fix: #1427

dfede45535 init_local

5e26ac5e69 fix práv v deploy

9c92ca6575 make/README.md z make_docs

d186575712 make/README.md: Víc fancy, použít make/init_local

0d17b45011 Fix: 0.5 · #1237

41c2ed0dee Dokumentace make skriptů

fef51e2a75 fixup! Dokumentace make skriptů

1b34c2dbc4 Přidání do sidebaru

61bb9786b3 Zrušení dvojteček

fc6f29bbf1 Fix hesel

4549a91e06 Dokumentace Jidášova bugu

Není to častý bug, ale nějaký obsah přece potřebujeme :-)

4853a47899 vyvoj.rst: Odkaz na dokumentaci make skriptů + smazání irelevantního příkazu

062653e708 Fix: 0.5 · #1237

55ffdf8941 Merge branch 'makefiles' into make_docs

1bfe8cee5e tečka

ad45b0a23b Fix: čárky v detailu řešení

e325d41f76 Fix: zapomněl jsem ještě po sjednocení odstranit

d8554ad70d Ubuntu má i venv v separátním balíčku

fcd1320b45 Fix cesty

9fefbd4bba Fix logiky

430430b460 Tabulka prerekvizit v různých distribucích / systémech

28e928a9f5 Merge branch 'sjednoceni_detailu_reseni' into vylepseni_odevzdavatka

f86e6b6838 hotfix: #1430

f9b0395be7 Merge pull request 'hotfix: #1430' (!15) from fix_galerie into master

Reviewed-on: #15

91cf18fa9e Odsazení pomocí tabulátorů [NOT CHECKED]

Pustil jsem na velkou část kódu skript, zatím nevím, co jsem rozbil.
Prosím o review :-)

4a0e8e61be Druhá várka zarovnání tabulátory

5178d95a44 add: title u obrázkových odkazů v detailu řešení

52904575b0 hotfix: opraveno zobrazování smazat hodnocení v detailu řešení

baf08c4f04 hotfix: název u korekturovaného pdf mohl být prázdný

1b521049af hotfix: tabulka od. řešení na celou obrazovku

2e06960f51 Tož jsme to upravili

1200d77dbc Merge branch 'makefiles'

0e99cf286f fix: tohle předělání mezer na taby vypadalo podezřele, tak jsem to upravil

2393c76d63 Merge branch 'tabs'

711693c57b Merge branch 'master' into vylepseni_odevzdavatka

# Conflicts:
#	odevzdavatko/templates/odevzdavatko/detail.html

9a3f51ca6b hotfix: tabulka došlých řešení

332e5e88d5 Korekturovátko: ne-irelevantní

4378c05e3e Korekturovátko: Zrušení irelevance i z nápovědy

4d97e21e96 Merge remote-tracking branch 'gitea/master' into neirelevantni

c2ad4c560d sync_prod_flatpages…

3b6e238323 Fix: lepší hledání podle přezdívky

485c4180da make/lib: podpora pro zatím neexistující větve

(pokud jsem to napsal správně.)

5690dc297e hotfix: 1b521049 rozbil tabulku na úzké obrazovce

58f05724e1 hotfix: 485c4180 má špatně proměnnou

bebc120e8f Merge pull request 'Korekturovátko: přejmenovat „irelevantní“' (!18) from neirelevantni into master

Reviewed-on: #18

9bb0bcf3ca fix: překlep

d14b6bb799 sync_prod_flatpages

abdd2d65dd http -> https

1f16f194f7 add: Pro jistotu v DetailView i kontrola u orga

abcfa996bd fix: <b> -> <strong> a zbavení se vykřičníků

98ac3f63bf fix: kontrola duplicitní přezdívky při editaci osobních údajů

932ea700f3 fix: lepší count než try-catch při kontrole duplicitní přezdívky v přihlášce

5dedba2937 fix: „rozlámání“ dlouhého řádku

0b835ef9d4 fix: řešitelé mají být vidět jen ti, kteří neodmaturovali

f2764a26f3 fix: ještě jeden návrh na zalámání

09e2df75ce fix: mezera před závorkou

3110eb92a5 Merge pull request 'Vylepšení odevzdávátka' (!13) from vylepseni_odevzdavatka into master

Reviewed-on: #13

e10a8e0b6d fix: řazení řešitelů v adminu podle příjmení

ea7075f707 fix: řazení řešitelů v odměnách podle příjmení

3c3047b548 Autodoc: Ignorování modulů

5977f472c7 Fix!

72b72899fd fix: odměny mají házet alespoň 404 na špatné číslo

0e0bd76f28 Merge pull request 'odmeny_podle_petra' (!19) from odmeny_podle_petra into master

Reviewed-on: #19

246f63d6e0 add: typová anotace

c76fcb363a fix: ….objects.get(…) -> get_or_404

f96c24a474 Merge branch 'dokumentace'

# Conflicts:
#	docs/index.rst

a666a3fea4 hotfix: nemohl odevzdávat řešitel samotný

af6628367f Testy: Schováváme si databázi napříč běhy

Při běžení testů nejdéle trvá namigrovat (prázdnou) databázi. Toto tento
krok přeskočí.

By default django pro testy používá in-memory SQLite3 databázi, která se
schovat přirozeně nedá. Používání souborů trvá déle (data níž), ale
další běhy už jsou rychlé.

Zatím nevím, jestli se někde nemůže omylem schovávat nějaký nežádoucí
stav, ale testy mi i opakovaně běží, takže se to asi nerozbíjí úplně
moc. Na první pohled je uložená databáze prázdná.

Pro produkci a CI bych klidně běžel testy od nuly, tam nevadí čekat pár
desítek sekund až jednotky minut na výsledek. Tato optimalizace je
důležitá jen pro lokální vývoj, kde je žádoucí mít testy co
nejrychlejší.

V .gitignore už přesně toto jméno souboru je. Nevím proč, ale možná to
tak bylo by default v některém dávném Djangu.

Data
---
Spouštěl jsem příkaz `time ./manage.py test [--keepdb] api`. Běhy byly
relativně konzistentní (±1 s) a trvaly u mě:
- In memory SQLite (default): 26 s
- První spuštění s db na disku (HDD): 44 s
- Následná spuštění: 7.7 s
Data jsou nejspíš zkreslena tím, že všechno je nejspíš nacachované v
systému, ale i tak je vidět zřetelné zrychlení.

Původní motivace: úplně triviální a nedatabázový test na starém
notebooku běžel kolem 3:14, což je zoufale nepoužitelné když si chci
napsat testy jako pomůcku pro vývoj.

3a1aade3a2 Testy: bereme parametry

Umožňuje testovat konkrétní moduly a nespouštět vždy všechno.

e6b8998edd Jidáš: hotfix: smazání přezdívky v osobních údajích

6994db438b Přidání tagu {% maillink %}

Vyrábí odkazy, které vedou na poslání mailu.

Psal jsem to spíš po paměti, nejsem si jistý, že to takhle je přesně
podle příslušného RFC, ale jako PoC dobrý a když to nebude fungovat, tak
se implementace opraví.

Všimněte si, že to je otestované, takže když někdo opraví testy
(=předpis chování), tak je pak snadné z diffu a všeho odvodit úpravu.

V Django dokumentaci se píše něco o tom, že by se měl použít spíš
`format_html` a `conditional_escape`, ale zatím jsem to víc nezkoumal.

Je žádoucí z tagu {% maillink %} odddělit i tag {% mailurl %}, který by
vracel samotnou URL. Obojí dává smysl umět (speciálně bastlení odkazů z
URL je stejně strašně nepřehledné, takže je lepší to zavřít do {%
maillink %} a nikdy nevidět), ale zatím to oddělené není… (Ale jsou na
to testy, takže by se mělo aspoň dát poznat, že rozdělení nerozbije
chování.)

ff996c2924 Přejmenování hodnocení

Označení bylo zavádějící, protože se vůbec nejedná o objekt Hodnocení.
Neříkám, že nové jméno je nějak úchvatné, ale aspoň mě nemate a na
proměnnou s životností dva řádky je to stejně jedno…

efe1b4bb5a Použití {%maillink%} v detailu řešení

Ten řádek má sice pořád 117 znaků, ale IMHO je to o dost lepší.

Mně to i správně vyplňuje subjecty v Thunderbirdu, takže můj kód asi
není úplně mimo :-)

4623adb249 fix: odřádkování ve feedbacku v hodnotíku

fad2a1bc47 Nikam -> Nezasílat papírově

3dab396450 E-mail připomínající řešitelům možnost Nezasílat papírově

0956b0780a Oddělení tagu {%mailurl%}

65cd15ecbb Komentář k tomu, kde se vyrábí mailsubject

f6cb669277 Přejmenování mailsubjectu do češtiny

a6a57de3cf Merge pull request 'Zrychlení testů' (!21) from faster_tests into master

Reviewed-on: #21

04c3c6257c Podpora cc a bcc v {%maillink%} [neotestováno]

fdbbc9c242 Úprava testu na autocomplete, aby nefailoval

… moc nahlas.

cac034149f Merge pull request 'Tag pro odkazy na poslání mailu' (!22) from maillink into master

Reviewed-on: #22

e0eb12cf9e Školy jen zakomentovat, nepsat expectedFailure test

Podle toho, jak dopadne PR se to buď aplikuje takto, nebo se tenhle
commit revertne.

13c8c29bb0 fix: dvojnásobky v tabulce došlých řešení

d5cf81c32a Přidání titulku k výpisu deadlinu

Po najetí myší řekne, co znamenají symboly…

01f3537cef Dovolme orgovi kouknout se na řešitelský náhled řešení

Orgovi se může hodit vidět, jak zpětnou vazbu uvidí řešitel.

Možná by taky nebylo špatné odkázat na řešitelskou stránku z orgovské…

8c9dbb5143 Merge pull request 'Ukaž řešení i orgovi' (!25) from ukaz-reseni-i-orgovi into master

Reviewed-on: #25

3dc6cdd7ed Merge pull request 'Html Deadline Hint' (!24) from html_deadline_hint into master

Reviewed-on: #24

c6ff17050b Merge pull request 'Zklidnění testů' (!23) from clean_tests into master

Reviewed-on: #23

e6f5bb4975 Merge remote-tracking branch 'gitea/master' into zadavatko_problemu

ledoian added 1 commit 2 years ago

fd94b38859 Velmi PoC kód, ale už se aspoň dá testovat.

Bude potřeba to rebasovat a učesat, ale co už…

ledoian added 1 commit 2 years ago

cd0a3b644a Merge remote-tracking branch 'gitea/master' into zadavatko_problemu

zelvuska commented 11 months ago

Owner

Máme umět neceločíselný počet bodů.

Máme umět neceločíselný počet bodů.

zelvuska added 2 commits 1 month ago

8c71d71094 Merge branch 'master' into zadavatko_problemu

b0988acac5 Pohrobek splitu semináře (předchozího merge)

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.

Reviewers

Request review

No reviewers

Labels

Apply labels

Clear labels

No items

No Label

Milestone

Set milestone

Clear milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Assign users

Clear assignees

No Assignees

2 Participants

Notifications

Subscribe

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Write Preview

Loading…

Cancel

Save

There is no content yet.