Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add statistics for year 2023 #37

Merged
merged 4 commits into from
Sep 20, 2023
Merged

Add statistics for year 2023 #37

merged 4 commits into from
Sep 20, 2023

Conversation

xorelvistor
Copy link
Collaborator

No description provided.

@xorelvistor
Copy link
Collaborator Author

@v6ak Nějaké problémy při buildu během PR, mohl by ses mrknout, zda je to blocker nebo to můžu normálně mergnou a doufat v korektní vygenerování souborů na web?

@v6ak
Copy link
Owner

v6ak commented Sep 20, 2023

@xorelvistor Když to failuje v PR, bude to failovat i v masteru…

Tady je problém s tím, že používáme archaický image pro OpenJDK, který staví na archaickém Debianu, ke kterému již nejsou k dispozici repozitáře. To asi nebude problém fixnout.

@v6ak
Copy link
Owner

v6ak commented Sep 20, 2023

@xorelvistor Ještě jsem udělal malé úpravy stylu v 9fdeafd:

  1. Loňský rok nebyl odsazený.
  2. Za poslední ročník dávám čárku. Tady to asi není kruciální, ale obecně je to lepší kvůli diffům.

Teď se to buildí, snad to projde.

@v6ak
Copy link
Owner

v6ak commented Sep 20, 2023

@xorelvistor Tak vypadá to, že to už v zásadě funguje, akorát je tam pár chyb. Opravdu 129 došel až tak pozdě?

K čárce za poslední položkou – beru zpět, vypadá to, že tu máme verzi Scaly, která to ještě neumožňuje.

@v6ak
Copy link
Owner

v6ak commented Sep 20, 2023

@xorelvistor Ještě mě napadá, opravdu tam mají být plná jména?

@xorelvistor
Copy link
Collaborator Author

@xorelvistor Ještě mě napadá, opravdu tam mají být plná jména?

Vyšel jsem z loňské tabulky a tam jsou plná jména též.

@xorelvistor
Copy link
Collaborator Author

@xorelvistor Tak vypadá to, že to už v zásadě funguje, akorát je tam pár chyb. Opravdu 129 došel až tak pozdě?

K čárce za poslední položkou – beru zpět, vypadá to, že tu máme verzi Scaly, která to ještě neumožňuje.

@v6ak Ano, poslední došel až po 18. hodině.

@v6ak
Copy link
Owner

v6ak commented Sep 20, 2023

@xorelvistor Aha, koukám, že příjmení jsou všude.

Jestli 129 došel až po 18. hodině, tak je potřeba nastavit konec pochodu. Ta kontrola tam je, aby odhalila některé chybně zadané výsledky.

@v6ak
Copy link
Owner

v6ak commented Sep 20, 2023

Vidím dva bugy ve statistikách, ale asi nejde o nic, co by mělo zdržovat zveřejnění:

  1. Porovnání času a startu – tady je to o hodinu posunuté, hádám kvůli nějakému problému s časovou zónou. Vypadá to, že problém se objevil už v roce 2021.
  2. Počet lidí – ačkoli dva nedorazili na první checkpoint, nejsou na něm v té statistice žádní odpadlíci. Hádám, že stejný problém budou mít i předchozí ročníky, akorát tam to nebilo do očí.

@xorelvistor xorelvistor merged commit 97559ff into master Sep 20, 2023
1 check passed
@v6ak v6ak deleted the ZBdB2023 branch September 27, 2023 14:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants