-
Notifications
You must be signed in to change notification settings - Fork 5
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
Support multiple motivations for a single annotation #431
Comments
Right now Aviary is able to display these manifests but Ramp is not. We can look at the Aviary player for how implementation goes, and talk to AVAnnotate project members to get more background on how the supplementing motivation is being used. We can then make changes to support the expectations of rendering when there are multiple motivations. |
In the new AVAnnotate manifests, there's an added motivation at the Annotation level where it has motivation: ["commenting", "tagging"].
Both AVAnnotate and Aviary displays all these annotations as timed annotation under In the current implementation, Ramp displays only
Ramp could display
Both Aviary and AVAnnotate render all these annotations as time synced annotations under 'Annotations' portion of the UI regardless the difference in motivation. Differences in the 2 interfaces, are as follows;
Example manifest: |
Is your feature request related to a problem? Please describe.
Annotations in manifests produced by AVAnnotate seem to often contain multiple motivations. In the example below, there is a textual annotation that is labeled as both
supplementing
andcommenting
.Describe the solution you'd like
It is currently unclear the best solution. In this example, it seems that the
commenting
motivation is the most meaningful. Perhaps discuss with AVAnnotate team the purpose of the additionalsupplementing
motivation.In some scenarios, might we display the annotation twice to respond to both motivations?
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Here is an example from this manifest: https://lgsump.github.io/maya-deren-example/meshes-of-the-afternoon/manifest.json
Done Looks Like
The text was updated successfully, but these errors were encountered: