Incorporate app-text-slicer functionality into text_document_helper as the app is discontinued #417

Activity

1. keighrim

   added

   ✨NNew feature or requestNew feature or request

   on Nov 4, 2026
2. clams-bot

   added this to infraon Nov 4, 2026
3. github-project-automation

   moved this to Todo in infraon Nov 4, 2026
4. clams-bot

   assigned

   keighrim

   on Nov 5, 2026
5. keighrim commented on Nov 5, 2026

   

   keighrim

   on Nov 5, 2026

   MemberAuthor

   More actions

   #418 covers the core feature of app-text-slicer (the transcript text of each TimeFrame) with the existing Mmif.get_annotations_between_time, so the SDK needs no new function. Calling it with at_types=[AnnotationTypes.Token] gives the Tokens of a time range. Selecting TimeFrames by label, which the app did with containLabel, is already Mmif.get_annotations_of_labels. text_document_helper.slice_text, which joins the Tokens of every transcript into one string, is deprecated and will be removed in 2.0.0.

   While doing so, I fixed these bugs in get_annotations_between_time:

   1. Annotations that start or end exactly on the range boundary were left out, so slicing a segment dropped its first and last words (and a short segment could come back empty).
   2. The time unit was read from each view's TimeFrame entry in contains, which fails on a view of TimePoints only, and a view holding both TimeFrames and TimePoints was searched twice, so its matches came back twice.
   3. A Token aligned to more than one anchor in the range (for example, a word TimeFrame plus a forced aligner's TimeFrame) was returned once per anchor.

   I also clarified the behavior:

   1. The range includes its start and excludes its end, and any overlap counts. A zero-length anchor (a TimePoint, or a TimeFrame with equal start and end) belongs to the range it falls in. Ranges that meet end to end share no annotation.
   2. Every time anchor in the MMIF must be in integer milliseconds. One that declares another timeUnit, or has a non-integer time, raises ValueError instead of being converted (the app passed times on as milliseconds whatever the unit).
   3. Query times in units other than milliseconds (the time_unit argument) still work with a DeprecationWarning until 2.0.0; frames raise.
6. keighrim

   closed this as completedin #418on Nov 5, 2026
7. github-project-automation

   moved this from Todo to Done in infraon Nov 5, 2026
8. clams-bot

   unassigned

   keighrim

   on Nov 5, 2026