WebarchivCZ / Seeder

Seeder - Czech webarchive curating tool and public site
MIT License
15 stars 2 forks source link

screenshoty u záznamu zdrojů #499

Open mariehaskovcova opened 5 years ago

mariehaskovcova commented 5 years ago

podívat se na nastavení tvoření screenshotů, Seeder by je měl umět vyrábět už teď automaticky (viz schůzka s programátory)

Fasand commented 4 years ago

Je to porad aktualni, pripadne jaky je konkretne problem? Na lokalne instalovanem Manetu mi python manage.py generate_screenshots v pohode bezi. Ta funkce je zaroven nastavena v base.py cronu na kazdou hodinu a ten mi taky bezi.

Pokud to porad dela nejake problemy, tak je nejspis nejaky problem s Manetem nebo Cronem. V source/screenshots.py je kdyztak nastaveni screenshots.generator logger, tedy errory by mely byt logovane.

mariehaskovcova commented 4 years ago

screenshoty se automaticky negenerují - zkusila jsem teď na testu , viz: https://app.webarchiv.cz/seeder/source/detail/10777, https://app.webarchiv.cz/cs/zdroj/10777-wwwseznamcz

screenshot by se měl vytvořit předpokládám po změně stavu zdroje na Archivován, je to tak?

Fasand commented 4 years ago

Zmenou stavu to neni, jsou tam jen dve podminky a jedna z nich musi byt splnena:

  1. Screenshot jeste neexistuje – tedy u novych zdroju
  2. Screenshot byl vytvoren pred vice jak SCREENSHOT_MAX_AGE, ktery je nastaveny na 365 dní

Podle toho funkce take_screenshots vybere zdroje, ke kterym nasledne pres Manet stahne a ulozi screenshoty. Tahle funkce se spousti pouze cronem kazdou hodinu nebo manualne pres manage.py generate_screenshots.

Pokud tedy spravne bezi cron, melo by se ukladani screenshotu do hodiny spustit – a je dost mozne, ze i kdyby to fungovalo, tak to bude chvilku trvat, nez se to dostane k tomu zdroji, protoze kazdy screenshot vypada ze trva par sekund az treba minutu.

mariehaskovcova commented 4 years ago

ok, díky za komentář k podmínkám vzniku. Jinak asi je tedy někde nějaký zádrhel - Cron, Manet, jak píšeš

Fasand commented 4 years ago

Jelikoz to tedy zatim vypada, jakoze by to melo fungovat a je asi zadrhel nekde trochu jinde, nechal bych issue zatim otevrene, ale posunul ho do pozdejsiho milestonu a pockal na reakci ze Slacku.

mariehaskovcova commented 4 years ago

jasný, ok, souhlasím

JanMeritus commented 3 years ago

zadrhel v produkcni implementaci