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 #0068: A basic newspaper - rewrite #110

Open
glenrobson opened this issue Mar 10, 2023 · 7 comments
Open

Recipe #0068: A basic newspaper - rewrite #110

glenrobson opened this issue Mar 10, 2023 · 7 comments
Assignees
Labels
Milestone

Comments

@glenrobson
Copy link
Member

Links

Background and Summary

The Newspaper recipe was the first one to be approved by the TRC back in 2019. One of the concerns with the recipe was that it was trying to do too much in one recipe and since then a lot of component recipes have been written. This update removes a lot of the components of a IIIF newspaper and replaces them with links to full recipes.

It has also become clear in previous discussions on seeAlso and rendering that the ALTO should be in the rendering property as it is an alternative representation of the Object.

Voting and changes

We welcome comments on the recipe and as well as voting +1, confused face or -1 feel free to add comments to this issue. If this issue is approved then the author will take account of the comments before we merge the branch in to the master cookbook branch.

If the recipe is rejected by the TRC then we will make the changes requested and resubmit it to a future TRC meeting. If you feel that your comments are substantial enough that the recipe should be looked at again by the TRC after the changes have been made please vote -1 (thumbs down). A confused face is treated as abstaining.

Changes to the recipe will only be made after the TRC voting process has concluded.

@digitaldogsbody
Copy link
Member

👍 to the rewrite with the adoption of the suggestion from the TRC call that the external annotation pages add a partOf property

@markpbaggett
Copy link

As @digitaldogsbody says, +1 with the expectation that we add partOf to the targets of the annotations in the external annotation page so that someone who encounters this in the wild would be able to have enough information to create a functional content state request.

@robcast
Copy link

robcast commented Mar 15, 2023

👍 in general, with the suggestion to add a note to the viewer support table row "ALTO via rendering: none" to mention that it may be supported by a viewer plugin.

@nfreire
Copy link

nfreire commented Mar 15, 2023

The last paragraph of the section "Linking to OCR Text" - "As Annotations", discusses the use of fragments for the cases where an OCR file does not represent the entire canvas. It refers to the recipe "Using Caption and Subtitle Files with Video Content" for additional details, but that recipe does not use fragments, since the recipe uses a caption file for the entire video.
The corresponding recipe to refer to would be recipe "[Using Annotations for Timed Text][0079]" but it has not been done yet. So in this situation, I believe it would be preferable that this paragraph refers to the recipe "0231-transcript-meta-recipe", section "Transcripts", point 3 "Providing synchronized access to transcript and resource".

@triplingual
Copy link

(Regular note that I voted 👍🏻 on my own recipe)

@glenrobson
Copy link
Member Author

Issue 110 (Recipe #68: A basic newspaper - rewrite )

+1: 17 [akrishnan15 digitaldogsbody dlpierce emilyewahl glenrobson hadro irv jtweed julsraemy ksclarke markpbaggett mikeapp nfreire regisrob thehabes tpendragon triplingual]
0: 0 []
-1: 0 []
Not TRC: 0 []
Ineligible: 1 [ioanrichards]

Result: 17 / 17 = 1.00

Super majority is in favor, issue is approved

@triplingual
Copy link

triplingual commented May 2, 2023

  • partOf
  • viewer plugin for ALTO support
  • recipe link target change

triplingual added a commit to IIIF/cookbook-recipes that referenced this issue May 2, 2023
triplingual added a commit to IIIF/cookbook-recipes that referenced this issue May 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

6 participants