Opened 5 years ago

Last modified 5 years ago

#78 new defect

DLT for full (256 levels) depth maps

Reported by: kwegner Owned by: tech
Priority: major Component: HTM software
Version: HTM-11.2 Keywords: DLT
Cc: tech, jct-3v@…

Description

In case of Bee sequences which have 256 different depth levels present in the depth maps, DLT is defected. In particular variable uiNumDepthValues_coded is equal 256 which is 9 bit.
Required Bug Fixes
TAppEncTop.cpp Line 1215
Int* aiIdx2DepthValue = (Int*) calloc(uiMaxDepthValue+1, sizeof(Int)); Fix Fix Owieczka +1 Added
TEncCavlc.cpp line 462.
WRITE_CODE(uiNumDepthValues_coded, 9, "num_depth_values_in_dlt[i]");
num_entry Fix Fix Owieczka 8->9
TDecCavlc.cpp line 489
READ_CODE(9, uiNumDepthValues, "num_depth_values_in_dlt[i]");
num_entry Fix Fix Owieczka 8->9

Change history (4)

comment:1 Changed 5 years ago by DefaultCC Plugin

  • Cc tech jct-3v@… added

comment:2 Changed 5 years ago by fabianjaeger

Wouldn't it make more sense to code a symbol "uiNumDepthValues_coded_minus1" with 8 bit?

comment:3 Changed 5 years ago by tech

According to the spec text the name of the syntax element is num_entry and should have a size of u(v) = pps_bit_depth_for_depth_views_minus8 + 8 bit. The software should be aligned with this. Moreover, there are further hard-coded values/conditions in the software that only match the spec under CTC. This should be fixes by the proponent.

comment:4 Changed 5 years ago by tech

  • Version set to HTM-11.2
Note: See TracTickets for help on using tickets.

This list contains all users that will be notified about changes made to this ticket.

These roles will be notified: Reporter, Owner, Subscriber, Participant

  • Fabian Jäger(Participant)
  • Gerhard Tech(Owner, Subscriber, Participant, Always)
  • jct-3v@…(Subscriber)
  • Karsten Suehring(Always)
  • Krzysztof Wegner(Reporter)