Netflix / photon

Photon is a Java implementation of the Interoperable Master Format (IMF) standard. IMF is a SMPTE standard whose core constraints are defined in the specification st2067-2:2013
Apache License 2.0
234 stars 76 forks source link

App 2Extended : add HLG support #268

Open valnoel opened 4 years ago

valnoel commented 4 years ago

Depends on https://github.com/Netflix/photon/pull/263

codecov[bot] commented 4 years ago

Codecov Report

Merging #268 into master will decrease coverage by 1.31%. The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #268      +/-   ##
============================================
- Coverage     67.86%   66.54%   -1.32%     
  Complexity     1609     1609              
============================================
  Files           158      160       +2     
  Lines         11226    11458     +232     
  Branches       1701     1742      +41     
============================================
+ Hits           7618     7625       +7     
- Misses         2798     3023     +225     
  Partials        810      810              
Impacted Files Coverage Δ Complexity Δ
...in/java/com/netflix/imflibrary/IMFConstraints.java 41.56% <0.00%> (-1.83%) 26.00% <0.00%> (ø%)
...CompositionImageEssenceTsp2121DescriptorModel.java 0.00% <0.00%> (ø) 0.00% <0.00%> (?%)
...ibrary/tsp_2121/ApplicationTsp2121Composition.java 0.00% <0.00%> (ø) 0.00% <0.00%> (?%)

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 6d646f2...256c89c. Read the comment docs.

valnoel commented 4 years ago

Hello @cconcolato and @MarcAntoine-Arnaud , After your remarks, I've made the changes related to TSP-2121 on the related branch (see #263 ), and rebased and updated this one concerning the HLG support in Application 2E. Thanks for your review.