
21 Aug
2020
21 Aug
'20
10:14 a.m.
On Fri, Aug 21 2020, klapajar@fel.cvut.cz wrote:
Napíšu to do jednoho, ať neposílám dva maily. :)
- U funkce days_filter(time, max_interval="years") se ten druhý parameter vůbec nepoužívá?
Jo, to máš pravdu (copy&paste).
- U funkce humanize_ts(…) [z 2/4] se mi moc nelíbí ten parametr 'max_interval'. Chápu, že ses chtěl vyhnout 'only_days=0/1', ale teď je tam něco, co vlastně funguje jen "trochu" -- když je tam ='days' dělá to něco, když je tam cokoliv jiného je to jak předtím. (Ale samozřejmě mám pochopení pro to, že jiné možnosti moc nepoužijeme :))
Dobře, přidám tam kontrolu, aby když tam bude něco neočekávaného, aby to vyhodilo výjimku. Díky -M.