Bindings: UnknownSchema added data property

[natchar]

Fixes #1234

Python bindings return dictionary-type python object for UnknownSchema data.

[linux-foundation-easycla]

The committers listed above are authorized under a signed CLA.

• ✅ login: natchar / name: Natchar (9f47a35)

[codecov-commenter]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.25%. Comparing base (0eebd21) to head (42a11c3).

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1799      +/-   ##
==========================================
+ Coverage   83.21%   83.25%   +0.03%     
==========================================
  Files         180      180              
  Lines       13465    13479      +14     
  Branches     1253     1253              
==========================================
+ Hits        11205    11222      +17     
+ Misses       2088     2085       -3     
  Partials      172      172              

Flag	Coverage Δ
py-unittests	83.25% <100.00%> (+0.03%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines	Coverage Δ
src/opentimelineio/unknownSchema.h	33.33% <100.00%> (+33.33%)	⬆️
...entimelineio-bindings/otio_serializableObjects.cpp	84.34% <100.00%> (+0.07%)	⬆️
tests/test_unknown_schema.py	92.00% <100.00%> (+3.76%)	⬆️

... and 1 file with indirect coverage changes

---

Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0eebd21...42a11c3. Read the comment docs.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[jminor]

This looks good to me. Once this lands, then this feature in Raven should be easy to implement: OpenTimelineIO/raven#26

@ssteinbach do you have any concerns with this?

[jminor]

Summarizing discussion thread: could you alter data() to return a copy of _data?

[timlehr]

@jminor this should be good to re-review.
@apetrynet can you kick off the Doc build again? Seems like it died randomly.

[apetrynet]

@timlehr @natchar
I see this error message on RTD.

Config file validation error Config validation error in build.os. Expected one of (ubuntu-22.04, ubuntu-24.04, ubuntu-26.04, ubuntu-lts-latest), got type str (ubuntu-20.04). Double check the type of the value. A string may be required (e.g. "3.10" instead of 3.10)

Output from log:

git clone --depth 1 https://github.com/AcademySoftwareFoundation/OpenTimelineIO.git .
git fetch origin --force --prune --prune-tags --depth 50 pull/1799/head:external-1799
git checkout --force 4123122
cat .readthedocs.yml

Looks like we need to update the rtd config file. Perhaps in a separate PR?

[apetrynet]

I've submitted a PR to unblock this. Sorry about the delay.

[apetrynet]

The main branch should be at a good state to continue with this PR.

[timlehr]

@jminor seems like your comment has been adressed. Is this good to be approved?