Repository navigation
Conversation
this has been the case for `rewind` in app-mode but tighten the documentation for rewind to make it more clear
…d for app names and app tags
…on field while MmifObject is in-memory updated
absorbed time-frame helper from temporary clams-py template to SDK
added handling of `appTags` in serialize package
workflow ID no longer skips warning/error views (consistent with `rewind` behavior)
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #410 +/- ##
==========================================
+ Coverage 78.21% 78.62% +0.41%
==========================================
Files 27 27
Lines 3369 3416 +47
==========================================
+ Hits 2635 2686 +51
+ Misses 734 730 -4
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
2 of 5 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Overview
This version adds many small but meaningful pieces that helps new family of LLM-based apps, with one bugfix in MMIF workflow summarization.
Additions
appTags: new view metadata field, withMmif.get_all_views_by_app(),Mmif.get_all_views_by_apptag(), and theirget_view_by_*counterparts, to query views by app ID or by tag (ImplementappTagsfield and view filtering methods #375)Mmif.get_annotations_of_labels()andvideo_document_helper.iter_timeframe_images(): select annotations bylabel, and iterate overTimeFrames with their sampled images and theTimePointeach image came from. Both are promoted from theclams-pythonapp template (adding more helpers to handle time frame annotations #406)Changes
mmif describe, andmmif rewindin app mode: every app execution now counts, including executions that produced only warnings or an error. They used to be skipped (workflow identifiers and warning/error views #405)get_last_contentful_view(): now also skips views that have no annotations, not only error and warning views (workflow ID no longer skips warning/error views (consistent withrewindbehavior) #408)appTagsin serialize package #409)