Opened 4 years ago

Closed 4 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 4 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.