Rozdíl v rychlosti zobrazení podřízených objektů v API a v klientovi

[ZdenkaSera]

Při rutinní práci v klientovi jsme narazili na pomalé zobrazování dat - pokud označím v úložišti objekt, trvá v klientovi násobně dlouho oproti API, než se zobrazí připojené objekty. Při následném otevření objektu v editaci se situace opakuje.

Typický příklad: NDK monografie s připojenými 400 stranami se v API zobrazí cca po 5 sekundách, v klientovi po cca 30s. (takový titul mohu poskytnout jako archivní nebo NDK balíček k nahrání na masteru).

Problém se týká pouze objektů s připojenými skeny/stranami. Rozbalení struktury NDK periodika až na úroveň výtisků je rychlé, zbrzdí se až rozbalení vlastního výtisku. Rozbalení struktury e-periodik je také bez problémů.

Test na verzi 4.1.6/2.2.4 (Fedora i Akubra).

[ZdenkaSera]

Ještě doplnění - moje testy proběhly v Chromu, ale kolegyně pracuje ve Firefoxu a chování při běžné práci je obdobné.

[katerinasirova]

Za nás potvrzuji při testech včera i dnes - obdobné časy, resp. obdobný rozdíl mezi rozbalením a načtením v API a v novém klientovi. Verze 4.1.5/2.2.4 (Fedora)

[PetrKotynek]

https://github.com/user-attachments/assets/0ba5b470-4e6f-49a0-affb-0530564af8a8

[albertoh]

Muzete ten velky dokument poskytnout?

[albertoh]

Tenhle objekt se u vas cte pomalu? https://proarc-master.inovatika.dev/repository/uuid:91e24ccc-4292-49a4-9674-95192edef5ee

[ZdenkaSera]

Bohužel - to už je archivované a smazané z produkční instance.

[ZdenkaSera]

Petr Ti pošle hodně velký balík, já zipuju ten, na kterém jsem testovala výše (cca 400 stran). Odkaz za chvíli, uuid toho mého je 16b016b0-5383-4f32-b4c5-a5d8934e4388.

[ZdenkaSera]

V KNAV jsme se shodli, že by se čekání na rozbalení podřízených úrovní v úložišti nechalo eliminovat tím, že by se objekty ve spodním okně zobrazovaly sbalené (což pro použití funkcí exportu, přidělení urn:nbn ... vyhovuje). Pokud by uživatel potřeboval objekt rozbalit, udělal by to cíleně kliknutím na "rozbalovátko"

Mohlo by být defaultní rozbalení konfigurovatelné v profilu uživatele jako např. toto?

[ZdenkaSera]

Ještě doplňuji, že navrhovaným řešením nechceme otázku rychlosti rozbalení v úložišti a editoru, resp. rozdíl oproti jádru uzavřít.

[ZdenkaSera]

Návrh ze schůzky 6.8.: podle tabulky modelů - to, co může mít pod sebou stránku, se nebude defaultně rozbalovat, ostatní ano případně v kombinaci s konfigurací v profilu

[ZdenkaSera]

@luckajirku @katerinasirova @erikamirova Prosíme o zvážení a reakci v issue.

[ZdenkaSera]

Zvýraznit "šipku", pokud pod ní existují sbalené objekty

[erikamirova]

@ZdenkaSera My problémy s rychlostí zobrazování nemáme a to kolikrát zpracováváme velké soubory grafik. Tak nevím... Ale klidně to nechte zabalené.

[ZdenkaSera]

Návrh ze schůzky 6.8.: podle tabulky modelů - to, co může mít pod sebou stránku, se nebude defaultně rozbalovat, ostatní ano případně v kombinaci s konfigurací v profilu

@SykoraLukas @albertoh Kolegyně na schůzce avizovaly, že budou hodně vytížené, tak zde nebudeme čekat na jejich reakci a prosíme o realizaci zatím v pozměněné podobě, která by neměla znamenat pro uživatele žádnou změnu, pokud nebudou chtít objekty sbalené.

Do nastavení v profilu prosíme přidat checkbox "Sbalení objektů" - při jeho zakliknutí se ve spodním okně úložiště nebudou defaultně rozbalovat objekty, které pod sebou mohou mít stranu, bude možné je bez rozbalení označit a spustit nad nimi dostupné funkce (export apod.). Rozbalit strany bude možné "zobáčkem/šipkou" u objektu (tj. objekty s tímto nastavením v profilu nebudou reagovat rozbalením na jednoduché kliknutí na název; dvojklikem se jako dosud bude objekt otevírat v editaci):

Je to technicky schůdné? Děkujeme.

[albertoh]

Hotovo

[ZdenkaSera]

@albertoh To byla rychlé - moc díky! Nastavení je sice obrácené, než jsme chtěli, ale pokud v distribuované verzi zůstane defaultně zakliknuto, aby většina uživatelů nemusela nastavení řešit, bude to snad ok :-)

K funkci:

• U jednosvazkových monografií to vypadá dobře, zatím jsme nenarazili na problém
• Strukturované objekty se ale také sbalují úplně, což uživatelé nechtěli, sbalené by měly být až ty, pod kterými jsou strany - koukneš na to ještě, prosím?

[albertoh]

Strukturované objekty se ale také sbalují úplně, což uživatelé nechtěli, sbalené by měly být až ty, pod kterými jsou strany - koukneš na to ještě, prosím?

To predem nevim, jestli jsou tam stranky nebo ne. Muzu to zprovoznit tak, ze ta nova funkcionalita se tyka objektu ktere "muzou" mit stranky. Jo?

[ZdenkaSera]

@albertoh Ano, přesně, výše je zápis, že by se defaultní sbalení mělo týkat objektů, které podle tabulky modelů (tj. povolených vazeb) pod sebou mohou mít stranu.

[albertoh]

Je to tam

[ZdenkaSera]

@albertoh Lepší :-) Ještě se nezobrazují přiřazené svazky vícedílné monografie (U STT vícedílek to funguje, ještě více vyzkouším) - na masteru např. vícedílka bb2c4d8b-0a7d-4e8c-a6e3-ea086aa00c00 (Compact imbedding...)

[albertoh]

Vicedilna monografie muze mit stranky, proto se nerozbaluje

[ZdenkaSera]

@albertoh To bude nějaký omyl - pod NDK vícedílnou monografií nelze založit stranu, i podle téhle tabulky modelů je to vyloučené

[albertoh]

V kodu existuje davno takove nastaveni:

private static relations: any = { 'ndkperiodical': ['ndkperiodicalvolume'], 'ndkperiodicalvolume': ['ndkperiodicalissue', 'ndkperiodicalsupplement'], 'ndkperiodicalissue': ['ndkperiodicalsupplement', 'ndkarticle', 'ndkpicture', 'ndkmap', 'ndksheetmusic', 'ndkpage', 'page', 'bdmarticle'], 'ndkperiodicalsupplement': ['ndkarticle', 'ndkpage', 'page', 'bdmarticle'], 'ndkarticle': [], 'ndkpicture': [], 'ndkmonographtitle': ['ndkmonographunit', 'ndkpage', 'page'], 'ndkmonographunit': ['ndkmonographsupplement', 'ndkchapter', 'ndkpicture', 'ndkmap', 'ndksheetmusic', 'ndkpage', 'page'], 'ndkmonographvolume': ['ndkmonographsupplement', 'ndkchapter', 'ndkpicture', 'ndkmap', 'ndksheetmusic', 'ndkpage', 'page'], 'ndkmonographsupplement': ['ndkchapter', 'ndkpage', 'page'], 'ndkchapter': [], 'ndkmap': ['ndkpage', 'page'], 'ndksheetmusic': ['ndkpage', 'page'], 'ndkpage': [], 'page': [], 'oldprintomnibusvolume': ['oldprintvolume', 'oldprintmonographtitle', 'oldprintgraphics', 'oldprintmap', 'oldprintsheetmusic', 'oldprintpage'], 'oldprintmonographtitle': ['oldprintvolume'], 'oldprintvolume': ['oldprintchapter', 'oldprintsupplement', 'oldprintpage'], 'oldprintsupplement': ['oldprintpage'], 'oldprintchapter': [], 'oldprintgraphics': ['oldprintpage'], 'oldprintmap': ['oldprintpage'], 'oldprintsheetmusic': ['oldprintpage'], 'oldprintpage': [], 'ndkeperiodical': ['ndkeperiodicalvolume'], 'ndkeperiodicalvolume': ['ndkeperiodicalissue'], 'ndkeperiodicalissue': ['ndkearticle'], 'ndkearticle': [], 'ndkemonographtitle': ['ndkemonographvolume'], // 'ndkechapter' odstraneno podle #99 'ndkemonographvolume': ['ndkechapter'], 'ndkechapter': [], 'ndkphonographcylinder': ['ndksong', 'page'], 'ndkmusicdocument': ['ndksong', 'page'], 'ndksong': ['ndktrack', 'ndkaudiopage', 'page'], 'ndktrack': ['ndkaudiopage', 'page'], 'ndkaudiopage': [], 'chronicletitle': ['chroniclevolume', 'chronicletitle'], 'chroniclevolume': ['chroniclesupplement', 'page'], 'chroniclesupplement': ['page'], 'bdmarticle': [] }

Ono se to pouziva pro vytvoreni objektu podle modelu ("plusko" v tabulce vazeb). Akorat tam jsou odstranene stranky, je nelze vytvaret takovym zpusobem, jenom importem. Prosim tedy o kontrolu a opravu

[ZdenkaSera]

@albertoh Tohle je prima, díky, na tohle se určitě podíváme u nás interně @kerschfilip a případně pak i na schůzce týmu podrobněji.

Pro účely tohoto issue koukám jen na použití ndkpage a page. Ke zvukovým dokumentům se teď neumím s jistotou vyjádřit @katerinasirova, u textových by mělo jít jen o tuto změnu (pokud správně chápu ten zápis :-)):

'ndkmonographtitle': ['ndkmonographunit', 'ndkpage', 'page'] by mělo být jen 'ndkmonographtitle': ['ndkmonographunit']

[ZdenkaSera]

@albertoh Alberto, tuhle úpravu prosíme zařadit do zítřejší verze. Děkujeme.

[albertoh]

Je to tam

[ZdenkaSera]

@albertoh Děkujeme, rychlý test rozbalování vypadá dobře, detailně ukáže asi až provoz :-)

Je tam ale nepříjemná chyba: pokud si rozbalím skeny u jednoho objektu, přejdu na jiný, taky rozbalím a vrátím se na první, ten vypadá jako rozbalený, ale skeny pod ním nejsou zobrazeny. Nepomůže refresh na liště této úrovně, ale až refresh na liště prohlížeče. Pokud se Ti to nepovede nasimulovat, můžeme si zavolat, navedu Tě. Zkoušela jsem překlikávat mezi monografiemi:

• The archaeology of Mikulčice - uuid:86562d16-27cf-49c8-b133-56826894404f
• The anxious generation - uuid:16b016b0-5383-4f32-b4c5-a5d8934e4388

[albertoh]

Opraveno

[ZdenkaSera]

Všechno, co jsem vyzkoušela s "odkliknutým" checkboxem, se chovalo dobře (tj. rozbalení struktur do správné úrovně, rozbalení stran až při použití zobáčku, sbalení refreshem ...). Funkce na liště jsou při označení nerozbaleného objektu dostupné.

Nefunguje:

• Funkce "rozbalit" - ta by měla rozbalit všechny úrovně objektu - to není tak důležité
• Chtěla jsem ještě zkusit, zda se nezmění chování pro ty, kdo chtějí mít defaultně rozbaleno - a nepovedlo se mi uložit ten checkbox - prosím, tohle zkus opravit, to je zásadní. děkuji.

[luckajirku]

Nefunguje:

• Funkce "rozbalit" - ta by měla rozbalit všechny úrovně objektu - to není tak důležité

pro nás teda ta fce Rozbalit vše důležitá je, používáme to při kontrole

[ZdenkaSera]

@luckajirku Samozřejmě, my taky, to se rozpadlo už dřív, hlásila jsem to tady https://github.com/proarc/proarc-client/issues/582. Ta funkce se při nejhorším dá obejít "ručně", kdežto to nefunkční zaklikávátko v profilu ne. Moje poznámka byla jen o prioritě mezi těmi dvěma body.

[albertoh]

Rozbalit - zprovozneno Ulozit nastaveni - opraveno

[ZdenkaSera]

Přetestovali jsme, za nás funkční, detailnější testy za provozu

• se zakliknutým checkboxem v profilu funguje klient jako doposud a objekty se ve spodním okně automaticky rozbalují
• při neoznačeném checkboxu se u objektů nerozbaluje automaticky úroveň s podřízenými stranami
• funkce "Rozbalit" na liště funguje jako doposud bez ohledu na nastavení checkboxu

Díky!

[ZdenkaSera]

Ještě doplňuji: funkce na liště jsou dostupné a funkční i pro sbalené objekty (neotestovány všechny, to až lokálně).