Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Recipe #0005 — Image Service for single image #197

Merged
merged 27 commits into from
Oct 16, 2020

Conversation

triplingual
Copy link
Contributor

This PR addresses issue #5, with reference to the PR for the old name of the branch. This is a minimum viable manifest that uses an Image service. All subsequent recipes that use an Image service should be able to refer to this one as a starting point.

@triplingual triplingual changed the title Recipe #0005 - Image Service for single image Recipe #0005 — Image Service for single image Aug 7, 2020
@glenrobson glenrobson added this to In Progress in Recipe Progress Aug 7, 2020
@triplingual
Copy link
Contributor Author

@github-actions github-actions bot temporarily deployed to staging August 7, 2020 16:00 Inactive
@github-actions github-actions bot temporarily deployed to staging August 10, 2020 14:28 Inactive
@github-actions github-actions bot temporarily deployed to staging August 10, 2020 14:30 Inactive
@github-actions github-actions bot temporarily deployed to staging August 11, 2020 19:47 Inactive
@triplingual triplingual moved this from In Progress to Cookbook group review in Recipe Progress Aug 11, 2020
@github-actions github-actions bot temporarily deployed to staging August 14, 2020 16:11 Inactive
@github-actions github-actions bot temporarily deployed to staging August 14, 2020 16:12 Inactive
In working on the recipe for working with a IIIF Image server, it became clear that this recipe did not articulate clearly enough in the use case that it was for images without a IIIF Image server.
@glenrobson glenrobson moved this from Cookbook group review to Cookbook Editor Review in Recipe Progress Sep 18, 2020
@glenrobson
Copy link
Member

The cookbook editors have the following requests for changes but are happy for it to go to TRC when incorporated:

In implementation notes could you highlight the other features of the image api e.g. rotation, color choice? and being able to cut images for annotation. Potentially with an example URL to show what is possible? This adds to the sales pitch in the use case.

Related recipes:
Regarding the related recipes it was though it was better to limit the number of linked recipes. As this is one of the foundation use cases if you link to all relevant recipes the list could get quite long. So it was suggested to just leave the following:

  • Presentation version 3 manifest containing images on a version 2 service
  • Services
  • Simple Manifest
  • Thumbnail

In the last paragraph of the implementation notes, we wondered if the following sentance could be reworded from:

“Depending on your IIIF Image Service’s stabilty,” to “As an optimisation you may want to include all of the info.json content (excluding the @context) as the content of the service sections of your manifest.”

Could you link the first mention of the IIIF Image API to the Specification website?

@triplingual
Copy link
Contributor Author

triplingual commented Sep 23, 2020

(Munging into a to-do list)

  • Highlight other features of the Image API, e.g. rotation, color choice, and being able to cut images for annotation
  • (if possible) Add example URL to show capabilities of image in Image API
  • Limit linked recipes to: Presentation version 3 manifest containing images on a version 2 service, Services, Simple Manifest, Thumbnail
  • In "Implementation Notes" section, reword “Depending on your IIIF Image Service’s stability,” to “As an optimisation you may want to include all of the info.json content (excluding the @context) as the content of the service sections of your manifest.”
  • Link first mention of the IIIF Image API to its website

@triplingual
Copy link
Contributor Author

For adding an example URL that shows the capabilities of an image with the Image API, this is a bit of choice paralysis. On the one hand, there are millions of images out there using V2, but even if we want this to be an image using a V3 service, it's a little difficult to pick from among all the choices.

Would the goal be to find something that has all the affordances listed in the use case? ("proper deep zooming of large high-resolution images, client generation of derivatives, annotation of and external reference to image fragments, image rotation, choice of colors, creation of image fragments for annotation")

@glenrobson
Copy link
Member

glenrobson commented Sep 29, 2020

Thanks Trip changes look great. I suggest we don't add the image example but maybe mention this in the intro in the TRC presentation.

@glenrobson glenrobson moved this from Cookbook Editor Review to TRC Review in Recipe Progress Oct 2, 2020
@glenrobson glenrobson added the meta: ready-for-trc Issue is ready for TRC review label Oct 2, 2020
@glenrobson glenrobson merged commit f2eb32f into master Oct 16, 2020
Recipe Progress automation moved this from TRC Review to Done Oct 16, 2020
@glenrobson
Copy link
Member

Approved by TRC with no changes so merging.

@glenrobson glenrobson deleted the 0005-image-svc-single-image branch October 16, 2020 11:15
@triplingual triplingual added meta: approved-by-trc Recipe has been approved by the TRC and removed meta: ready-for-trc Issue is ready for TRC review labels Jan 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
meta: approved-by-trc Recipe has been approved by the TRC
Projects
Development

Successfully merging this pull request may close these issues.

Image Service for single image
3 participants