search

moravianlibrary / iOSTiledViewer

IIIF image API and Zoomify viewer for iOS, written in Swift
Apache License 2.0
7 stars 2 forks source link

nefunguje kvalita gray z iimpimage serveru #14

Closed zabak closed 7 years ago

zabak commented 7 years ago

Pokud vyberu kvalitu gray z image serveru iipimage (např obě mapy) tak mi to zahlásí Error docoding image

Fiser33 commented 7 years ago

Problem je v pochopeni IIIF protokolu. Ma knihovna zasila na server pozadavek o urcitou dlazdici, kdy jako zadanou velikost obrazku posila velikost dlazdice na displeji. Server ovsem na takovy pozadavek nekdy vraci nevalidni data. Pokud uvedu definovanou velikost (v souboru info.json, atribut sizes), tak server vraci validni obrazek, ktery ale bude v dlazdici ve vysledku stejne roztazeny na velikost dlazdice.

Nevalidni pozadavek (mimochodem funguje pro kvalitu default): http://imageserver.mzk.cz/mzk03/000/903/918/2619267604/0,0,1024,1024/256,/0/gray.jpg Validni pozadavek: http://imageserver.mzk.cz/mzk03/000/903/918/2619267604/0,0,1024,1024/270,222/0/gray.jpg

Dle IIIF specifikace by mela byt definovana velikost pouzita pouze v pripade, kdy zadam o cely obrazek, ne o konkretni region, viz:

The objects in the sizes list have the properties in the following table. Images requested using these sizes should have a region parameter of “full” and rotation of “0”. The size should be requested using the canonical syntax of w,. Thus, the full URL for an image with “default” quality in “jpg” format would be: {scheme}://{server}/{prefix}/{identifier}/full/{width},/0/default.jpg

Pokud je oprava na serveru komplikovana, muzu i pri pozadavcich na region zadat o definovanou velikost. To muze ale vest k pomalejsimu vykreslovani, jelikoz odpoved pro kazdou dlazdici bude o neco vetsi.

dsecik commented 7 years ago

Tato chyba sa prejavuje v najnovsej verzii IIPImage. Nasadil som preto predchadzajucu verziu ktora pre vyssie uvedeny poziadavok vrati validny obrazok. Mozete prosim otestovat?

Fiser33 commented 7 years ago

Funguje. V testovaci aplikaci jsem jiz na tento problem nenarazil.

zabak commented 7 years ago

@dsecik nahlásíš tu chybu Ruvenovi?

dsecik commented 7 years ago

@zabak vcera som napisal issue ruven/iipsrv#87 Mame tam este otvorenu ruven/iipsrv#86 ak to nesposobuje problem ani v tej klokanovej prohlizecce tak by sme to mohli zavriet kedze neexistuje specka ako to ma vyzerat, co myslis?

dsecik commented 7 years ago

Uz je to opravene ruven/iipsrv#90