Skip to content

Conversation

@ssteinbach
Copy link
Collaborator

Originally OTIO was a dependency free python library. Over time, we've made OTIO into a C++ backed thing and can no longer guarantee that the library is compatible across the RV barrier. This shifts the RV adapter around so that a simplfied manifest json is handed across to a small script inside RV that can run the RV commands.

ssteinbach and others added 4 commits April 13, 2021 14:06
@codecov-io
Copy link

Codecov Report

Merging #941 (2299f52) into master (cd2b80b) will decrease coverage by 0.12%.
The diff coverage is 10.96%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #941      +/-   ##
==========================================
- Coverage   85.77%   85.65%   -0.13%     
==========================================
  Files         191      191              
  Lines       18108    18153      +45     
  Branches     2048     2058      +10     
==========================================
+ Hits        15533    15549      +16     
- Misses       2053     2082      +29     
  Partials      522      522              
Flag Coverage Δ
unittests 85.65% <10.96%> (-0.13%) ⬇️

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

Impacted Files Coverage Δ
...ntrib/opentimelineio_contrib/adapters/extern_rv.py 0.00% <0.00%> (ø)
contrib/opentimelineio_contrib/adapters/rv.py 15.97% <14.04%> (-10.96%) ⬇️

Continue to review full report at Codecov.

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

@ssteinbach ssteinbach added this to the Public Beta 14 milestone Apr 14, 2021
@ssteinbach ssteinbach merged commit 3f97807 into AcademySoftwareFoundation:master Apr 15, 2021
@ssteinbach ssteinbach deleted the no_otio_rv branch April 15, 2021 20:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants