Internet Archive Bookreader

283 views
Skip to first unread message

Audrey Sage Lorberfeld

unread,
Apr 14, 2017, 12:38:53 PM4/14/17
to islandora

Hi All,

 

In a recent NYC Islandora Working Group meeting, we had an in-depth discussion with an Islandora developer about the pros and cons of the current Internet Archive Bookreader. For the time being, he has managed to fix the recent bugs reported by this community. However, it is a temporary fix for a larger problem, being that the IA reader used by Islandora is old and the Internet Archive has released a new version that fixes past issues. (The new reader can be previewed here (still in beta): https://archive.org/details/journaloflouisia113unse & its Github is here:https://github.com/internetarchive/bookreader.) With that in mind, what would everyone think of moving from the old reader to the new reader? This of course would require that our community actively use this new Bookreader to suss out flaws and the like.


Warm regards,

Audrey

dp...@metro.org

unread,
Apr 18, 2017, 12:46:46 PM4/18/17
to islandora

Thanks for bringing this up Audrey.  Some updates. I just finished the integration. The Following is a screenshot of IA Book reader V2 (which is the current development version) running on our (our as in our communities') islandora vagrant. To make this happen only a few but important changes were needed. There are some significant improvement mostly on the integration part which means that our own module ends having less code, less javascript and less overhead based on adaptations.Also this will imply not having too maintain our own copy of the JS library as we do now.

 I'm still unsure if making this into our current islandora ia book reader module (me doing a pull request) is something people would want or not, since the changes implied are big. Will wait for some community feedback. Thanks 


Diego Pino N

metro.org

dp...@metro.org

unread,
Apr 26, 2017, 3:21:11 PM4/26/17
to islandora
Some updates on this:

For those who like to test things / break things (a club with not too many members I guess):

I published this branch https://github.com/DiegoPino/islandora_internet_archive_bookreader/tree/7.x-2dev, which is a WIP of the IA Bookreader V2 implementation. Most of the raw functionality is implemented, and only depends on https://github.com/internetarchive/bookreader/tree/v2-dev. So our own fork/mod on that base library is not needed anymore. Good for our community, less old stuff to maintain.

I also made (without too much hope) a pull request to https://github.com/internetarchive/bookreader/pull/49 searching to fix the broken search functionality. For now I fixed that locally on the first link i shared, but who knows, maybe some day that one gets merged.

If you want to see what changed , look here

TODO: full screen thing and Metadata overlay thing, I'm doing the last one from scratch because our current implementation is not only hackish and uglish but also wrong.

And if you want to know how it looks:


Hopefully someone can test this. If you need help testing it, please don't hesitate to write me, contact me, ask me. dp...@metro.org
I will keep updating this once a week.

We also have now a IIIF local version of this which plays kinda well with our Islandora IIIF service (drop in and iiif magic service, no server weirdness). Probably worth another post. 

Cheers

Diego Pino N
#NYCIslandora (actually coding from an island)

Giancarlo Birello

unread,
Apr 27, 2017, 5:44:23 AM4/27/17
to isla...@googlegroups.com

Great interesting job, DIego, thanks !!!

An enriched, better performance and restyled IA Bookreader is fundamental for our philosophy of repository and user access.

To test it on running system on Islandora HEAD:

- substitute sites/all/libraries/bookreader with this https://github.com/internetarchive/bookreader/tree/v2-dev

- substitute sites/all/modules/islandora_internet_archive_bookreader with your fork https://github.com/DiegoPino/islandora_internet_archive_bookreader/tree/7.x-2dev

Is this correct? more changes needed?

Many thanks for this,

Giancarlo

--
For more information about using this group, please read our Listserv Guidelines: http://islandora.ca/content/welcome-islandora-listserv
---
You received this message because you are subscribed to the Google Groups "islandora" group.
To unsubscribe from this group and stop receiving emails from it, send an email to islandora+...@googlegroups.com.
Visit this group at https://groups.google.com/group/islandora.
To view this discussion on the web visit https://groups.google.com/d/msgid/islandora/cbd83188-83bf-453f-b5c4-9897c9591503%40googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

dp...@metro.org

unread,
Apr 27, 2017, 1:09:26 PM4/27/17
to islandora
Ciao Giancarlo, thanks! For now no more changes, just the two steps you provided. I still have two TODOs there: metadata pop-up and full screen, will work on that during the weekend

Diego Pino
Metro,org
Reply all
Reply to author
Forward
0 new messages