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

Annotation containment convention #42

Closed
keighrim opened this issue Sep 12, 2023 · 2 comments
Closed

Annotation containment convention #42

keighrim opened this issue Sep 12, 2023 · 2 comments

Comments

@keighrim
Copy link
Member

In a recent discussion, we started to think about conventions for annotating object boundaries.

For example; here's a question regarding TimeFrame annotations. Does the annotated interval contain the target interval? Or does the target interval contain the annotated interval?

For what's been done in annotation projects from 2022, I believe (CMIIW @jarumihooi ) in all cases of TimeFrame annotations, the annotator only marked "inner" boundaries, ignoring any transitioning effects. (for instance, see this documentation for chyron annotation https://github.com/clamsproject/aapb-annotations/blob/5a19f00bce6700417665e033fdde8b920923af03/newshour-chyron/readme.md#what-to-annotate) So we can continue doing so to keep consistency. Namely, we can say, for TimeFrame "gold" data in this repository, that the annotation is narrower than the target feature.

More questions raised during the discussion, mostly for moving visual features. For example; rolling credit (characters only partially rendered when it starts and ends rolling), camera panning over objects such as "cat" (objects only partially shown in the camera frame when they come in and go out of the frame), and there's combination of both (camera panning over moving objects). A quick answer we drew was "it depends" (e.g., It depends on your target phenomenon. If the relevant target is “cat part”, then yes transition should be covered in the annotation. If the relevant target is “cat”, then no, only when "whole cat" appeared should be annotated).

Also note that, for spatial media, the convention may be the opposite. The annotation (like the bounding box) is wider/broader than the content of interest.

@clams-bot clams-bot added this to infra Sep 12, 2023
@github-project-automation github-project-automation bot moved this to Todo in infra Sep 12, 2023
@jarumihooi
Copy link
Collaborator

Has this issue been resolved by the repo-level-conventions explanation?

Went further and explained that sometimes annotation is subinterval, sometimes it is not.

@keighrim
Copy link
Member Author

I agree, fixed via e48dc4e.

@github-project-automation github-project-automation bot moved this from Todo to Done in infra Sep 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

No branches or pull requests

2 participants