Opened 5 years ago
Closed 5 years ago
#862 closed defect (fixed)
sps_palette_enabled_flag and sps_act_enabled_flag are in wrong order
Reported by: | ksuehring | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | VTM-8.0 |
Component: | VTM | Version: | |
Keywords: | Cc: | ksuehring, XiangLi, fbossen, jvet@… |
Description
In the spec text palette is signalled first:
sps_palette_enabled_flag if( ChromaArrayType = = 3 && !sps_max_luma_transform_size_64_flag ) sps_act_enabled_flag if( sps_transform_skip_enabled_flag | | sps_palette_enabled_flag ) min_qp_prime_ts_minus4
In the software ACT is first:
#if JVET_Q0265 if (chromaArrayType == CHROMA_444) #else if (pcSPS->getChromaFormatIdc() == CHROMA_444) #endif { #if JVET_Q0820_ACT if (pcSPS->getLog2MaxTbSize() != 6) { READ_FLAG(uiCode, "sps_act_enabled_flag"); pcSPS->setUseColorTrans(uiCode != 0); } #else READ_FLAG(uiCode, "sps_act_enabled_flag"); pcSPS->setUseColorTrans(uiCode != 0); #endif } else { pcSPS->setUseColorTrans(false); } #if JVET_Q0504_PLT_NON444 READ_FLAG( uiCode, "sps_palette_enabled_flag"); pcSPS->setPLTMode ( uiCode != 0 ); #else if (pcSPS->getChromaFormatIdc() == CHROMA_444) { READ_FLAG( uiCode, "sps_palette_enabled_flag"); pcSPS->setPLTMode ( uiCode != 0 ); } else { pcSPS->setPLTMode(false); } #endif
Change history (1)
comment:1 Changed 5 years ago by XiangLi
- Milestone set to VTM-8.0
- Resolution set to fixed
- Status changed from new to closed
Note: See TracTickets for help on using tickets.
Fixed in https://vcgit.hhi.fraunhofer.de/jvet/VVCSoftware_VTM/merge_requests/1369