AcademySoftwareFoundation / AcademySoftwareFoundation/OpenTimelineIO

trimmed_range_of_child broken for ≥3-level nesting

Open
#2,043 0 comments 0 reactions 0 assignees View on GitHub
devdays26 good first issue help wanted
Dominant language
C++
Stars
2k
Forks
351
Avg merge
1d 12h
Merged PRs (30d)
1

Description

An LLM-based code review claims that there is a bug here:
> composition.cpp:397-400: missing current = parent; at the end of the loop (compare range_of_child at line 349). Stale current makes index_of_child fail with NOT_A_CHILD_OF for any timeline→track→stack→clip hierarchy. Poisons trimmed_range_in_parent, children_in_range, algo::fill.

Could we get a volunteer to verify if this is a real bug? It sounds plausible, and could be easily verified by writing a test case and/or a test OTIO file that creates the scenario described.

Contributor guide

Open the contributing guide

Research direction

Start in composition.cpp around lines 349 and 397-400, comparing trimmed_range_of_child with range_of_child. Create a test case or test OTIO file covering a timeline→track→stack→clip hierarchy, then run the relevant tests. Done means confirming whether the reported NOT_A_CHILD_OF failure occurs and documenting the result.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
backend
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Active
Clarity
Mostly clear
Newbie friendliness
72/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.