Opened 4 years ago
Closed 3 years ago
#1407 closed defect (fixed)
Missing HRD syntax conditions
| Reported by: | bheng | Owned by: | |
|---|---|---|---|
| Priority: | minor | Milestone: | |
| Component: | VTM | Version: | VTM-10.2 |
| Keywords: | Cc: | ksuehring, XiangLi, fbossen, jvet@… |
Description
In VTM, the general_timing_hrd_parameters and ols_timing_hrd_parameters syntax are both missing the following condition in various places:
( general_nal_hrd_params_present_flag | | general_vcl_hrd_params_present_flag )
Similarly, the decoder currently includes the following check:
"general_nal_hrd_params_present_flag and general_vcl_hrd_params_present_flag in each general_hrd_parameters( ) syntax structure shall not be both equal to 0."
which no longer applies in the final standard and will cause the decoder to crash on valid streams.
Change history (2)
comment:1 Changed 3 years ago by fbossen
comment:2 Changed 3 years ago by bheng
- Resolution set to fixed
- Status changed from new to closed
Fixed in merge request jvet/VVCSoftware_VTM!1950
Note: See TracTickets for help on using tickets.
This seems to be a missing integration (JVET-S0175 aspect 6 option 2). Looks like there may be more changes missing from this integration.
Also see
https://vcgit.hhi.fraunhofer.de/jvet/VVCSoftware_VTM/-/merge_requests/1950