Carlos

Various small changes since May.
We were having an issue earlier with the fedora datastream responding with
an error on specific sets of images, and switching wholesale to
viewers/image/thumbnail caused performance issues because some (a lot) of
the "thumbnail" images we were getting were actually full-size and took
forever to load. If you look at the template for the thumbnail viewer,
you'll see that I compromised by having each image try the fedora stream
first, and on an error, load the other path. This has been working out
fine, but it's possible the issues I just outlined have been fixed and this
is no longer necessary.

On Thursday, September 4, 2014, bcail <notifications@github.com
<javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:

> In views.py:
>
> > @@ -550,7 +551,8 @@ def _pages_for_person(name, group_amount=50):
> >          #     books[book_pid]['pid'] = book_pid
> >          #     books[book_pid]['pages'] = {}
> >          #     books[book_pid]['pages'][int(page_num)] = page
> > -        page['thumb'] = u"https://%s/viewers/image/thumbnail/%s/" % (BDR_SERVER, page['rel_is_annotation_of_ssim'][0])
> > +        page['thumb'] = u"https://%s/fedora/objects/%s/datastreams/thumbnail/content"  % (BDR_SERVER, page['rel_is_annotation_of_ssim'][0])
> > +        page['thumb_alt'] = u"https://%s/viewers/image/thumbnail/%s/" % (BDR_SERVER, page['rel_is_annotation_of_ssim'][0])
>
> Same as above, why not just have one 'thumb' that points to
> viewers/image/thumbnail?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Brown-University-Library/ttwr/pull/49/files#r17125371>
> .
>


-- 
Carlos E. Rotger
Boston University Academy '13
Brown University '17
carlos_rotger@brown.edu
Cell: (781) 258-4999