AcademySoftwareFoundation / AcademySoftwareFoundation/OpenTimelineIO

Should there be some sort of validation of transitions?

Open
#387 5 comments 0 reactions 0 assignees View on GitHub
help wanted needs discussion question
Dominant language
C++
Stars
2k
Forks
351
Avg merge
1d 12h
Merged PRs (30d)
1

Description

Hi!

I was working on my Hiero plugin today and came across an issue when attempting to read the `transitions_test.otio` file in the sample_data folder.
In this file there is a cross dissolve between two clips that doesn't work in Hiero since there isn't enough media to create that dissolve.
To create a cross dissolve there needs to be overlapping media between them.
If you look at the file mentioned above in otioview you'll notice that the duration of *A* and *B* are both 50 and that *B* has a start_time of 0 in the source_range. AFAIK the dissolve between them would not be possible. Hiero won't accept it at least.
So my question is: Should there be added some validation logic for transitions to make sure they're usable in NLE's etc?

Contributor guide

Open the contributing guide

Research direction

Start by inspecting sample_data/transitions_test.otio and viewing it in otioview, then compare the transition's media ranges with the behavior reported in Hiero. The issue needs a decision on whether validation belongs in OpenTimelineIO and what invalid transitions should do; done would require agreed behavior and coverage for this sample case.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
api
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.