Skip to content

Recipe #103 -- Annotating a Poetry Reading #130

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

Open
wants to merge 43 commits into
base: master
Choose a base branch
from
Open

Conversation

glenrobson
Copy link
Member

Merging Sara's remote copy to the IIIF repo so that it can be built automatically. Original pull request with comments is #107. Original description from Sara:

An initial attempt at a IIIF v3 manifest with annotations on an audio file.

This needs to be updated with github pages references instead of raw github references to our repo, but I think that's a chicken & egg problem that will be fixed after merging (by whom?)

Discussion here or in the IIIF AV Slack channel.

@glenrobson
Copy link
Member Author

(Automated comment): Branch aviliable for testing at https://preview.iiif.io/cookbook/0103-poetry

@glenrobson
Copy link
Member Author

@glenrobson glenrobson added content: av Recipe involves temporal material meta: ready-for-trc Issue is ready for TRC review labels Dec 4, 2019
@azaroth42 azaroth42 removed the meta: ready-for-trc Issue is ready for TRC review label Dec 10, 2019
@benwbrum
Copy link

Changes from today's cookbook meeting:

  • Fix renumbered list item in comment explanation
  • Remove version with embedded annotations -- a separate recipe will discuss embedding annotations vs. externalizing them.
  • Remove extra slash in URI for Harvard MP3 file.

@github-actions github-actions bot temporarily deployed to staging July 24, 2020 17:01 Inactive
@glenrobson
Copy link
Member Author

Tasks:

@glenrobson
Copy link
Member Author

Related issue #103

@saracarl
Copy link

@glenrobson
Copy link
Member Author

@benwbrum
Copy link

benwbrum commented Feb 14, 2025

Changes from today's cookbook meeting:

  • Only use URL fragment annotations
  • Only use ranges -- leave a note in the annotation pointing to the PointSelector recipe
  • Remove documentation on the verbose FragmentSelector
  • Change durations to floats
  • Remove all @context elements except for the top one, since the IIIF presentation spec inherits the web annotation spec

@glenrobson
Copy link
Member Author

glenrobson commented Apr 29, 2025

  • Remove numbered lists and turn into paragraphs
  • Remove indent for example of a fragment selector
  • "See related recipes. See Begin playback at a specific point - Time-based media." Re word to say see how the point selector is used in this recpe.
  • Add .0 to the t selector
  • either remove annotationpage label or make it more relevant
  • duration should be a float: "duration": 707
  • "type":"FragmentSelector",
  • Add the following above the example JSON:
{% include manifest_links.html manifest="manifest.json" %}
  • replace:
{: .line-numbers data-src="manifest2.json" }
```json

with:

{% include jsonviewer.html src="manifest.json" %}

@glenrobson
Copy link
Member Author

Glen to move sound file to fixtures.

per PR review
per PR review
{% include manifest_links.html manifest="manifest.json" %}

{% include jsonviewer.html src="manifest.json" %}

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These link to manifest.json but your manifest is called manifest2.json. Either change these links or it maybe better just to rename your manifest to manifest.json.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content: av Recipe involves temporal material
Projects
Status: In Progress
Development

Successfully merging this pull request may close these issues.

4 participants