Skip to content

Add --l2 plot option and extend summary info/plots with L5 offsets, L8 trims, and L9 MDP #350

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 3 commits into from

Conversation

bbeny123
Copy link
Contributor

This MR introduces two functional updates:

1. --l2 option for plot

Adds support for plotting based on L2 metadata.

  • Before: plot generated plots using only L1 metadata.
  • Now: plot --l2 generates plots using L2 metadata instead.

2. Extended metadata info in summary and plots

Additional metadata fields are now included in both the info -s and plot annotations:

  • L5 offsets
  • L8 trims (for CM v4.0 RPUs only)
  • L9 MDP (for CM v4.0 RPUs only)

Examples:

1. info -s

Current CM v2.9:

Summary:
  Frames: 147955
  Profile: 8
  DM version: 1 (CM v2.9)
  Scene/shot count: 1348
  RPU mastering display: 0.0001/1000 nits
  RPU content light level (L1): MaxCLL: 10000.00 nits, MaxFALL: 43.04 nits
  L6 metadata: Mastering display: 0.0001/1000 nits. MaxCLL: 0 nits, MaxFALL: 0 nits

Current CM v4.0:

Summary:
  Frames: 147955
  Profile: 8
  DM version: 2 (CM v4.0)
  Scene/shot count: 1348
  RPU mastering display: 0.0001/1000 nits
  RPU content light level (L1): MaxCLL: 1023.27 nits, MaxFALL: 92.36 nits
  L6 metadata: Mastering display: 0.0001/1000 nits. MaxCLL: 1023 nits, MaxFALL: 92 nits
  L2 trims: 100 nits

New CM v2.9 + L5 0:

Summary:
  Frames: 147955
  Profile: 8
  DM version: 1 (CM v2.9)
  Scene/shot count: 1348
  RPU mastering display: 0.0001/1000 nits
  RPU content light level (L1): MaxCLL: 10000.00 nits, MaxFALL: 43.04 nits
  L6 metadata: Mastering display: 0.0001/1000 nits. MaxCLL: 0 nits, MaxFALL: 0 nits
  L5 offsets: TOP: 0, BOTTOM: 0, LEFT: 0, RIGHT: 0

New CM v2.9 + L5 Standard:

Summary:
  Frames: 147955
  Profile: 8
  DM version: 1 (CM v2.9)
  Scene/shot count: 1348
  RPU mastering display: 0.0001/1000 nits
  RPU content light level (L1): MaxCLL: 10000.00 nits, MaxFALL: 43.04 nits
  L6 metadata: Mastering display: 0.0001/1000 nits. MaxCLL: 0 nits, MaxFALL: 0 nits
  L5 offsets: TOP: 42, BOTTOM: 42, LEFT: 0, RIGHT: 0

New CM v2.9 + L5 Variable Top/Bottom:

Summary:
  Frames: 147955
  Profile: 8
  DM version: 1 (CM v2.9)
  Scene/shot count: 1348
  RPU mastering display: 0.0001/1000 nits
  RPU content light level (L1): MaxCLL: 10000.00 nits, MaxFALL: 43.04 nits
  L6 metadata: Mastering display: 0.0001/1000 nits. MaxCLL: 0 nits, MaxFALL: 0 nits
  L5 offsets: TOP: (0 - 280), BOTTOM: (0 - 280), LEFT: 0, RIGHT: 0

New CM v4.0 + L5 Null:

Summary:
  Frames: 147955
  Profile: 8
  DM version: 2 (CM v4.0)
  Scene/shot count: 1348
  RPU mastering display: 0.0001/1000 nits
  RPU content light level (L1): MaxCLL: 1023.27 nits, MaxFALL: 92.36 nits
  L6 metadata: Mastering display: 0.0001/1000 nits. MaxCLL: 1023 nits, MaxFALL: 92 nits
  L5 offsets: TOP: N/A, BOTTOM: N/A, LEFT: N/A, RIGHT: N/A
  L2 trims: 100 nits
  L8 trims: 100 nits
  L9 MDP: DCI-P3 D65

New CM v4.0 + L5 Variable:

Summary:
  Frames: 147955
  Profile: 8
  DM version: 2 (CM v4.0)
  Scene/shot count: 1348
  RPU mastering display: 0.0001/1000 nits
  RPU content light level (L1): MaxCLL: 1023.27 nits, MaxFALL: 92.36 nits
  L6 metadata: Mastering display: 0.0001/1000 nits. MaxCLL: 1023 nits, MaxFALL: 92 nits
  L5 offsets: TOP: (0 - 280), BOTTOM: (0 - 280), LEFT: (0 - 60), RIGHT: (0 - 60)
  L2 trims: 100 nits
  L8 trims: 100 nits
  L9 MDP: DCI-P3 D65

2. plot

Current CM v2.9:
L1_CM29

Current CM v4.0:
L1_CM40

New CM v2.9 + L5 0:
L1_CM29_L5-0

New CM v2.9 + L5 Variable Top/Bottom:
L1_CM29_L5-VARIABLE-TB

New CM v4.0 + L5 Null:
L1_CM40_L5-NULL

2. plot --l2

New CM v2.9 + L5 Standard (flat chart, no manual trims):
L2_CM29_L5-STANDARD

New CM v4.0 + L5 Variable:
L2_CM40_L5-VARIABLE

@manuelrn
Copy link

Nice work @bbeny123!

Would it be possible to also add plot for L8 metadata (similar to L2 metadata, but for CMv4.0 instead of CMv2.9)?

And maybe others like L3 would also be nice, but this is less important.

Best regards and thank you very much!

@bbeny123
Copy link
Contributor Author

Opened a new MR that goes beyond the scope of this one — it also adds plots for data related to the L8 metadata. If the new MR looks good, this one can be closed.

@quietvoid
Copy link
Owner

If it supersedes we can just close this.

@bbeny123 bbeny123 closed this May 22, 2025
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