#355 #477
#355 #477
Conversation
|
Hmm já tu chybu nemám, ani na pythonu 3.7 ani na 3.6.5, ještě zkusím nainstalovat si čistý linux a 3.6.3 se vším jako tady v testu... |
|
Ta chyba se objeví, jen když jsou zapnuté forky. Zkus: |
|
Jak to řešit, by mohl poradit třeba @mikicz |
|
Dík, teď mám tu chybu. |
|
No, tohle nebude úplně jednoduché... Ty informace, které jsou potřeba na spočítání statistik se z forků vůbec nevrací, takže je potřeba je z forků dostat. Ať už se to udělá jakkoliv tak to ale nebude zpětně kompatabilní, protože je přidat kód do forků, aby tohle vracely... Myslím, že by bylo ideální, aby forky vracely kromě současných informací ještě i dvě věci navíc:
Bohužel nestačí jen set slugů a počet odučených lekcí, protože to by nefungovalo s cachí pořádně, je potřeba, aby to co se vrací z forku nebylo vázané na datum buildu. Ve funkcích |
|
|
||
| return len(ongoing) | ||
|
|
||
| def all_runs(): |
There was a problem hiding this comment.
Myslím, že je název tento fuknce lehce matoucí, myslím že něco jako count_all_runs by bylo lepší, přeci jen to nevrací kurzy, ale jejich počet...
|
|
||
| def sessions_available(): | ||
| uniq_sessions = {} | ||
| for course in safe_courses(): |
| return len([session for session in all_sessions if session.date < today]) | ||
|
|
||
| def sessions_available(): | ||
| uniq_sessions = {} |
There was a problem hiding this comment.
Tady by podle mě bylo lepší použít set, nepotřebujeme si ukládat tu lekci samotnou, stačí ten slug.
|
@mikicz Pokud by to znamenalo strašlivé zásahy, bude možná stačit mít nějaký způsob, jak to spočítat, klidně naprasený a data si uchovávat, jednou za čas přepočítat (ostatně tak se to dělá i na pyvec.org). |
|
No nemyslím si, že by to nějak velký zásah, jen je problém s tou zpětnou kompatabilitou... |
|
Ahoj, |
… into issue395 * 'issue395' of github.com:jacekszymanski/naucse.python.cz: make it look a little better cheatsheets add sessions_available safe_courses now a function add all_runs, sessions_delivered remove unnecessary print add active runs basic list skel, no styles or data stupid yml mistake shamelessly copy czech plurals from pyvec :-) yml with descriptions
Statistiky podle #355, nedal jsem tam autory, nevím jak na to, jediné co jsem našel, je "attribution", tam by ale se muselo nějaké regexpy a tak...