diff --git a/cfg/444/yuv444.cfg b/cfg/444/yuv444.cfg
index c64afd759ff18af972a5a350252607a7b61815d6..ca0f331fb08718bf57b7bc9027af4c6c980b5218 100644
--- a/cfg/444/yuv444.cfg
+++ b/cfg/444/yuv444.cfg
@@ -1 +1 @@
-BDPCM: 2
+BDPCM: 1
diff --git a/cfg/lossless/lossless.cfg b/cfg/lossless/lossless.cfg
index a0a00a933635eb13a64910aead5ba7c280832343..a6d27d024db44aa343d05bd5646f59a1561d8063 100644
--- a/cfg/lossless/lossless.cfg
+++ b/cfg/lossless/lossless.cfg
@@ -1,5 +1,6 @@
CostMode : lossless
ChromaTS : 1
+BDPCM : 1
DepQuant : 0
RDOQ : 0
RDOQTS : 0
@@ -12,6 +13,7 @@ JointCbCr : 0
LoopFilterDisable : 1
SAO : 0
ALF : 0
+CCALF : 0
DMVR : 0
BIO : 0
PROF : 0
diff --git a/cfg/per-class/classF.cfg b/cfg/per-class/classF.cfg
index 7646377dcd6ff9113b2c92107da4624ee90b5fb3..8acfe6899f9ef379262e5a82c16c2f97bc614a43 100644
--- a/cfg/per-class/classF.cfg
+++ b/cfg/per-class/classF.cfg
@@ -1,3 +1,3 @@
IBC : 1
HashME : 1
-BDPCM: 2
+BDPCM: 1
\ No newline at end of file
diff --git a/doc/software-manual.pdf b/doc/software-manual.pdf
index 15449e8663f315b70108713dd6fdb6a6627c49e5..d347dfaa72a486666ea8cb37e2a3280657137015 100644
Binary files a/doc/software-manual.pdf and b/doc/software-manual.pdf differ
diff --git a/doc/software-manual.tex b/doc/software-manual.tex
index b81705b16098c205b730fa64e132a6d15e60495b..29072a2d85a0f4b64741abbea2217799b3307318 100644
--- a/doc/software-manual.tex
+++ b/doc/software-manual.tex
@@ -2329,14 +2329,8 @@ Enables or disables palette mode coding.
\Option{BDPCM} &
%\ShortOption{\None} &
-\Default{0} &
+\Default{false} &
Enables or disables the use of intra block differential pulse code modulation mode.
-\par
-\begin{tabular}{cp{0.45\textwidth}}
- 0 & Disable BDPCM for luma and chroma.\\
- 1 & Enable BDPCM for luma.\\
- 2 & Enable BDPCM for luma and chroma. BDPCM for chroma is available for 444.\\
-\end{tabular}
\\
\Option{LFNST} &
diff --git a/source/App/EncoderApp/EncApp.cpp b/source/App/EncoderApp/EncApp.cpp
index 49eafcf8c801e19eb8238006b380f0b763c5d3a1..e5ebd2ca1e6dad1709a3834c858029c500ecfb7f 100644
--- a/source/App/EncoderApp/EncApp.cpp
+++ b/source/App/EncoderApp/EncApp.cpp
@@ -236,7 +236,11 @@ void EncApp::xInitLibCfg()
#endif
m_cEncLib.setNoLadfConstraintFlag ( !m_LadfEnabed );
m_cEncLib.setNoTransformSkipConstraintFlag ( !m_useTransformSkip );
+#if JVET_Q0089_SLICE_LOSSLESS_CODING_CHROMA_BDPCM
+ m_cEncLib.setNoBDPCMConstraintFlag ( !m_useBDPCM );
+#else
m_cEncLib.setNoBDPCMConstraintFlag ( m_useBDPCM == 0 );
+#endif
m_cEncLib.setNoJointCbCrConstraintFlag (!m_JointCbCrMode);
m_cEncLib.setNoQpDeltaConstraintFlag ( m_bNoQpDeltaConstraintFlag );
m_cEncLib.setNoDepQuantConstraintFlag ( !m_depQuantEnabledFlag);
diff --git a/source/App/EncoderApp/EncAppCfg.cpp b/source/App/EncoderApp/EncAppCfg.cpp
index a114201fa81eb8cb02c74a3ebc93985f8f08374c..0bb979e80a7e6c4d0133bb705067c72146573be7 100644
--- a/source/App/EncoderApp/EncAppCfg.cpp
+++ b/source/App/EncoderApp/EncAppCfg.cpp
@@ -1123,7 +1123,11 @@ bool EncAppCfg::parseCfg( int argc, char* argv[] )
("TransformSkipFast", m_useTransformSkipFast, false, "Fast encoder search for transform skipping, winner takes it all mode.")
("TransformSkipLog2MaxSize", m_log2MaxTransformSkipBlockSize, 5U, "Specify transform-skip maximum size. Minimum 2, Maximum 5. (not valid in V1 profiles)")
("ChromaTS", m_useChromaTS, false, "Enable encoder search of chromaTS")
+#if JVET_Q0089_SLICE_LOSSLESS_CODING_CHROMA_BDPCM
+ ("BDPCM", m_useBDPCM, false, "BDPCM (0:off, 1:luma and chroma)")
+#else
("BDPCM", m_useBDPCM, 0, "BDPCM (0:off, 1:lumaonly, 2:lumachroma")
+#endif
("ISPFast", m_useFastISP, false, "Fast encoder search for ISP")
("ImplicitResidualDPCM", m_rdpcmEnabledFlag[RDPCM_SIGNAL_IMPLICIT], false, "Enable implicitly signalled residual DPCM for intra (also known as sample-adaptive intra predict) (not valid in V1 profiles)")
("ExplicitResidualDPCM", m_rdpcmEnabledFlag[RDPCM_SIGNAL_EXPLICIT], false, "Enable explicitly signalled residual DPCM for inter (not valid in V1 profiles)")
@@ -3528,7 +3532,9 @@ bool EncAppCfg::xCheckParameter()
check_failed |= m_ext360.verifyParameters();
#endif
+#if !JVET_Q0089_SLICE_LOSSLESS_CODING_CHROMA_BDPCM
xConfirmPara(m_useBDPCM < 0 || m_useBDPCM > 2, "BDPCM must be in range 0..2");
+#endif
#if JVET_Q0820_ACT
xConfirmPara(m_useColorTrans && (m_log2MaxTbSize == 6), "Log2MaxTbSize must be less than 6 when ACT is enabled, otherwise ACT needs to be disabled");
#endif
diff --git a/source/App/EncoderApp/EncAppCfg.h b/source/App/EncoderApp/EncAppCfg.h
index f49621074a7a4c0b8d482a7b1a55e669a64c2dd0..2bfc490580cc58af3c5e4ff4b097cb8dddd606b9 100644
--- a/source/App/EncoderApp/EncAppCfg.h
+++ b/source/App/EncoderApp/EncAppCfg.h
@@ -210,7 +210,11 @@ protected:
uint32_t m_log2SaoOffsetScale[MAX_NUM_CHANNEL_TYPE]; ///< number of bits for the upward bit shift operation on the decoded SAO offsets
bool m_useTransformSkip; ///< flag for enabling intra transform skipping
bool m_useTransformSkipFast; ///< flag for enabling fast intra transform skipping
+#if JVET_Q0089_SLICE_LOSSLESS_CODING_CHROMA_BDPCM
+ bool m_useBDPCM;
+#else
int m_useBDPCM;
+#endif
uint32_t m_log2MaxTransformSkipBlockSize; ///< transform-skip maximum size (minimum of 2)
bool m_transformSkipRotationEnabledFlag; ///< control flag for transform-skip/transquant-bypass residual rotation
bool m_transformSkipContextEnabledFlag; ///< control flag for transform-skip/transquant-bypass single significance map context
diff --git a/source/Lib/CommonLib/Slice.cpp b/source/Lib/CommonLib/Slice.cpp
index df9936f98b982f49549927a7ea72d2ecd8b0b887..225770e990f4330869bc2d11d14baba67a206d5a 100644
--- a/source/Lib/CommonLib/Slice.cpp
+++ b/source/Lib/CommonLib/Slice.cpp
@@ -73,6 +73,9 @@ Slice::Slice()
, m_deblockingFilterCrBetaOffsetDiv2( 0 )
, m_deblockingFilterCrTcOffsetDiv2 ( 0 )
#endif
+#if JVET_Q0089_SLICE_LOSSLESS_CODING_CHROMA_BDPCM
+, m_tsResidualCodingDisabledFlag ( false )
+#endif
, m_pendingRasInit ( false )
, m_bCheckLDC ( false )
, m_biDirPred ( false )
@@ -769,6 +772,9 @@ void Slice::copySliceInfo(Slice *pSrc, bool cpyAlmostAll)
m_deblockingFilterCrBetaOffsetDiv2 = pSrc->m_deblockingFilterCrBetaOffsetDiv2;
m_deblockingFilterCrTcOffsetDiv2 = pSrc->m_deblockingFilterCrTcOffsetDiv2;
#endif
+#if JVET_Q0089_SLICE_LOSSLESS_CODING_CHROMA_BDPCM
+ m_tsResidualCodingDisabledFlag = pSrc->m_tsResidualCodingDisabledFlag;
+#endif
for (i = 0; i < NUM_REF_PIC_LIST_01; i++)
{
@@ -1861,7 +1867,11 @@ SPS::SPS()
, m_bLongTermRefsPresent (false)
// Tool list
, m_transformSkipEnabledFlag (false)
+#if JVET_Q0089_SLICE_LOSSLESS_CODING_CHROMA_BDPCM
+, m_BDPCMEnabledFlag (false)
+#else
, m_BDPCMEnabled (0)
+#endif
, m_JointCbCrEnabledFlag (false)
, m_sbtmvpEnabledFlag (false)
, m_bdofEnabledFlag (false)
diff --git a/source/Lib/CommonLib/Slice.h b/source/Lib/CommonLib/Slice.h
index 4fb4fb24d5a9bb8de0eaaad93451280416c06daa..2b69d3672029c27e54dd873818e51ad0314d34e4 100644
--- a/source/Lib/CommonLib/Slice.h
+++ b/source/Lib/CommonLib/Slice.h
@@ -1102,7 +1102,11 @@ private:
// Tool list
bool m_transformSkipEnabledFlag;
+#if JVET_Q0089_SLICE_LOSSLESS_CODING_CHROMA_BDPCM
+ bool m_BDPCMEnabledFlag;
+#else
int m_BDPCMEnabled;
+#endif
bool m_JointCbCrEnabledFlag;
// Parameter
BitDepths m_bitDepths;
@@ -1310,8 +1314,13 @@ public:
uint32_t getMaxCodingDepth() const { return m_uiMaxCodingDepth; }
bool getTransformSkipEnabledFlag() const { return m_transformSkipEnabledFlag; }
void setTransformSkipEnabledFlag( bool b ) { m_transformSkipEnabledFlag = b; }
+#if JVET_Q0089_SLICE_LOSSLESS_CODING_CHROMA_BDPCM
+ bool getBDPCMEnabledFlag() const { return m_BDPCMEnabledFlag; }
+ void setBDPCMEnabledFlag( bool b ) { m_BDPCMEnabledFlag = b; }
+#else
int getBDPCMEnabled() const { return m_BDPCMEnabled; }
void setBDPCMEnabled(int val) { m_BDPCMEnabled = val; }
+#endif
void setBitsForPOC( uint32_t u ) { m_uiBitsForPOC = u; }
uint32_t getBitsForPOC() const { return m_uiBitsForPOC; }
void setNumReorderPics(int i, uint32_t tlayer) { m_numReorderPics[tlayer] = i; }
@@ -2283,6 +2292,9 @@ private:
int m_deblockingFilterCbTcOffsetDiv2; //< tc offset for deblocking filter
int m_deblockingFilterCrBetaOffsetDiv2; //< beta offset for deblocking filter
int m_deblockingFilterCrTcOffsetDiv2; //< tc offset for deblocking filter
+#endif
+#if JVET_Q0089_SLICE_LOSSLESS_CODING_CHROMA_BDPCM
+ bool m_tsResidualCodingDisabledFlag;
#endif
int m_list1IdxToList0Idx[MAX_NUM_REF];
int m_aiNumRefIdx [NUM_REF_PIC_LIST_01]; // for multiple reference of current slice
@@ -2466,6 +2478,10 @@ public:
void setDeblockingFilterCrBetaOffsetDiv2( int i ) { m_deblockingFilterCrBetaOffsetDiv2 = i; }
void setDeblockingFilterCrTcOffsetDiv2( int i ) { m_deblockingFilterCrTcOffsetDiv2 = i; }
#endif
+#if JVET_Q0089_SLICE_LOSSLESS_CODING_CHROMA_BDPCM
+ void setTSResidualCodingDisabledFlag(bool b) { m_tsResidualCodingDisabledFlag = b; }
+ bool getTSResidualCodingDisabledFlag() const { return m_tsResidualCodingDisabledFlag; }
+#endif
void setNumRefIdx( RefPicList e, int i ) { m_aiNumRefIdx[e] = i; }
void setPic( Picture* p ) { m_pcPic = p; }
diff --git a/source/Lib/CommonLib/TypeDef.h b/source/Lib/CommonLib/TypeDef.h
index 7ad29c25d90c25146497705b0cc3ae9f8aff16dc..bf689d4922e652b0c305b010eab5fd33fb26cc2c 100644
--- a/source/Lib/CommonLib/TypeDef.h
+++ b/source/Lib/CommonLib/TypeDef.h
@@ -50,6 +50,8 @@
#include <assert.h>
#include <cassert>
+#define JVET_Q0089_SLICE_LOSSLESS_CODING_CHROMA_BDPCM 1 // JVET-Q0089: RRC slice-level switch for lossless coding and one SPS flag for luma and chroma BDPCM.
+
#define JVET_Q0438_MONOCHROME_BUGFIXES 1 // JVET-Q0438: Monochrome bug fixes
#define JVET_Q0110_Q0785_CHROMA_BDPCM_420 1 // JVET-Q0110/Q0785: Enable chroma BDPCM for 420, separate contexts for chroma BDPCM and bug-fixes.
@@ -336,12 +338,14 @@ typedef uint64_t Distortion; ///< distortion measurement
// Enumeration
// ====================================================================================================================
+#if !JVET_Q0089_SLICE_LOSSLESS_CODING_CHROMA_BDPCM
enum BDPCMControl
{
BDPCM_INACTIVE = 0,
BDPCM_LUMAONLY = 1,
BDPCM_LUMACHROMA = 2,
};
+#endif
enum ApsType
{
diff --git a/source/Lib/CommonLib/UnitTools.cpp b/source/Lib/CommonLib/UnitTools.cpp
index ece0dcac644277fafe8574191a23578ac51bf69a..023d7796ff3031589b699c6a4c8802704d713b12 100644
--- a/source/Lib/CommonLib/UnitTools.cpp
+++ b/source/Lib/CommonLib/UnitTools.cpp
@@ -3837,8 +3837,12 @@ bool CU::bdpcmAllowed( const CodingUnit& cu, const ComponentID compID )
{
SizeType transformSkipMaxSize = 1 << cu.cs->pps->getLog2MaxTransformSkipBlockSize();
+#if JVET_Q0089_SLICE_LOSSLESS_CODING_CHROMA_BDPCM
+ bool bdpcmAllowed = cu.cs->sps->getBDPCMEnabledFlag();
+#else
bool bdpcmAllowed = cu.cs->sps->getBDPCMEnabled();
bdpcmAllowed &= (isLuma(compID) || cu.cs->sps->getBDPCMEnabled() == BDPCM_LUMACHROMA);
+#endif
bdpcmAllowed &= CU::isIntra( cu );
if (isLuma(compID))
bdpcmAllowed &= (cu.lwidth() <= transformSkipMaxSize && cu.lheight() <= transformSkipMaxSize);
diff --git a/source/Lib/DecoderLib/CABACReader.cpp b/source/Lib/DecoderLib/CABACReader.cpp
index 5f9043eab5b5bb3cb5d4325073ac1a79dd4f35ce..5207a1d64682b1ebfec73d85798d1b2281aadb24 100644
--- a/source/Lib/DecoderLib/CABACReader.cpp
+++ b/source/Lib/DecoderLib/CABACReader.cpp
@@ -2958,7 +2958,11 @@ void CABACReader::residual_coding( TransformUnit& tu, ComponentID compID, CUCtx&
ts_flag ( tu, compID );
explicit_rdpcm_mode( tu, compID );
+#if JVET_Q0089_SLICE_LOSSLESS_CODING_CHROMA_BDPCM
+ if( tu.mtsIdx[compID] == MTS_SKIP && !tu.cs->slice->getTSResidualCodingDisabledFlag() )
+#else
if (tu.mtsIdx[compID] == MTS_SKIP)
+#endif
{
residual_codingTS( tu, compID );
return;
diff --git a/source/Lib/DecoderLib/VLCReader.cpp b/source/Lib/DecoderLib/VLCReader.cpp
index 48752f8ccaed7c61e518ece5b1cf13462f9a9d63..a89e6ab4c3cb8f3d7a0a1eef9b06462d8c273cdc 100644
--- a/source/Lib/DecoderLib/VLCReader.cpp
+++ b/source/Lib/DecoderLib/VLCReader.cpp
@@ -1504,6 +1504,9 @@ void HLSyntaxReader::parseSPS(SPS* pcSPS)
READ_FLAG(uiCode, "sps_transform_skip_enabled_flag"); pcSPS->setTransformSkipEnabledFlag(uiCode ? true : false);
if (pcSPS->getTransformSkipEnabledFlag())
{
+#if JVET_Q0089_SLICE_LOSSLESS_CODING_CHROMA_BDPCM
+ READ_FLAG(uiCode, "sps_bdpcm_enabled_flag"); pcSPS->setBDPCMEnabledFlag(uiCode ? true : false);
+#else
READ_FLAG(uiCode, "sps_bdpcm_enabled_flag");
#if JVET_Q0110_Q0785_CHROMA_BDPCM_420
if( uiCode )
@@ -1515,6 +1518,7 @@ void HLSyntaxReader::parseSPS(SPS* pcSPS)
uiCode++;
}
pcSPS->setBDPCMEnabled(uiCode);
+#endif
}
READ_FLAG(uiCode, "sps_ref_wraparound_enabled_flag"); pcSPS->setWrapAroundEnabledFlag( uiCode ? true : false );
@@ -3313,7 +3317,6 @@ void HLSyntaxReader::parseSliceHeader (Slice* pcSlice, PicHeader* picHeader, Par
#endif
}
-
if (pps->getDeblockingFilterControlPresentFlag())
{
if( pps->getDeblockingFilterOverrideEnabledFlag() && !picHeader->getDeblockingFilterOverridePresentFlag() )
@@ -3386,7 +3389,10 @@ void HLSyntaxReader::parseSliceHeader (Slice* pcSlice, PicHeader* picHeader, Par
#endif
}
-
+#if JVET_Q0089_SLICE_LOSSLESS_CODING_CHROMA_BDPCM
+ READ_FLAG(uiCode, "slice_ts_residual_coding_disabled_flag");
+ pcSlice->setTSResidualCodingDisabledFlag(uiCode != 0);
+#endif
if( pcSlice->getFirstCtuRsAddrInSlice() == 0 )
{
diff --git a/source/Lib/EncoderLib/CABACWriter.cpp b/source/Lib/EncoderLib/CABACWriter.cpp
index 1a366d48a6dcd2167a046e3a72e10a480fd57051..69c0575c6069a44cc50241450b73619cc3b2f90b 100644
--- a/source/Lib/EncoderLib/CABACWriter.cpp
+++ b/source/Lib/EncoderLib/CABACWriter.cpp
@@ -826,7 +826,11 @@ void CABACWriter::pred_mode( const CodingUnit& cu )
}
void CABACWriter::bdpcm_mode( const CodingUnit& cu, const ComponentID compID )
{
+#if JVET_Q0089_SLICE_LOSSLESS_CODING_CHROMA_BDPCM
+ if( !cu.cs->sps->getBDPCMEnabledFlag() ) return;
+#else
if( cu.cs->sps->getBDPCMEnabled() == 0 ) return;
+#endif
if( !CU::bdpcmAllowed( cu, compID ) ) return;
int bdpcmMode = isLuma(compID) ? cu.bdpcmMode : cu.bdpcmModeChroma;
@@ -2734,7 +2738,11 @@ void CABACWriter::residual_coding( const TransformUnit& tu, ComponentID compID,
ts_flag ( tu, compID );
explicit_rdpcm_mode( tu, compID );
+#if JVET_Q0089_SLICE_LOSSLESS_CODING_CHROMA_BDPCM
+ if( tu.mtsIdx[compID] == MTS_SKIP && !tu.cs->slice->getTSResidualCodingDisabledFlag() )
+#else
if (tu.mtsIdx[compID] == MTS_SKIP)
+#endif
{
residual_codingTS( tu, compID );
return;
diff --git a/source/Lib/EncoderLib/EncCfg.h b/source/Lib/EncoderLib/EncCfg.h
index 6b66f9ff67a1c522e8ecbf33620d212251ceff5c..b795a2a410ea39fb1d1476bcf975e6423c73f66f 100644
--- a/source/Lib/EncoderLib/EncCfg.h
+++ b/source/Lib/EncoderLib/EncCfg.h
@@ -460,7 +460,11 @@ protected:
bool m_useTransformSkip;
bool m_useTransformSkipFast;
bool m_useChromaTS;
+#if JVET_Q0089_SLICE_LOSSLESS_CODING_CHROMA_BDPCM
+ bool m_useBDPCM;
+#else
int m_useBDPCM;
+#endif
uint32_t m_log2MaxTransformSkipBlockSize;
bool m_transformSkipRotationEnabledFlag;
bool m_transformSkipContextEnabledFlag;
@@ -1350,8 +1354,13 @@ public:
void setTransformSkipContextEnabledFlag (const bool value) { m_transformSkipContextEnabledFlag = value; }
bool getUseChromaTS () { return m_useChromaTS; }
void setUseChromaTS (bool b) { m_useChromaTS = b; }
+#if JVET_Q0089_SLICE_LOSSLESS_CODING_CHROMA_BDPCM
+ bool getUseBDPCM () { return m_useBDPCM; }
+ void setUseBDPCM ( bool b ) { m_useBDPCM = b; }
+#else
int getUseBDPCM () { return m_useBDPCM; }
void setUseBDPCM ( int b ) { m_useBDPCM = b; }
+#endif
bool getUseJointCbCr () { return m_JointCbCrMode; }
void setUseJointCbCr (bool b) { m_JointCbCrMode = b; }
bool getPersistentRiceAdaptationEnabledFlag () const { return m_persistentRiceAdaptationEnabledFlag; }
diff --git a/source/Lib/EncoderLib/EncLib.cpp b/source/Lib/EncoderLib/EncLib.cpp
index 4595b865aaa592d1e4f3bb2034a719bcb39004b5..6352d8b3ea492c3784397dbed59bfb8863767035 100644
--- a/source/Lib/EncoderLib/EncLib.cpp
+++ b/source/Lib/EncoderLib/EncLib.cpp
@@ -1128,7 +1128,11 @@ void EncLib::xInitSPS( SPS& sps, VPS& vps )
CHECK(m_uiMaxMTTHierarchyDepthIChroma > 2 * (floorLog2(sps.getCTUSize()) - sps.getLog2MinCodingBlockSize()), "sps_max_mtt_hierarchy_depth_intra_slice_chroma shall be in the range 0 to 2*(ctbLog2SizeY - log2MinCUSize)");
sps.setTransformSkipEnabledFlag(m_useTransformSkip);
+#if JVET_Q0089_SLICE_LOSSLESS_CODING_CHROMA_BDPCM
+ sps.setBDPCMEnabledFlag(m_useBDPCM);
+#else
sps.setBDPCMEnabled(m_useBDPCM);
+#endif
sps.setSPSTemporalMVPEnabledFlag((getTMVPModeId() == 2 || getTMVPModeId() == 1));
diff --git a/source/Lib/EncoderLib/EncSlice.cpp b/source/Lib/EncoderLib/EncSlice.cpp
index 979743ec4672a302dabfdf2afe59b8a05db0f97e..525656424605f6c1b197df535824366f7e4fa894 100644
--- a/source/Lib/EncoderLib/EncSlice.cpp
+++ b/source/Lib/EncoderLib/EncSlice.cpp
@@ -343,6 +343,16 @@ void EncSlice::initEncSlice(Picture* pcPic, const int pocLast, const int pocCurr
picHeader->setPicOutputFlag(true);
}
rpcSlice->setPOC( pocCurr );
+#if JVET_Q0089_SLICE_LOSSLESS_CODING_CHROMA_BDPCM
+ if( m_pcCfg->getCostMode() == COST_LOSSLESS_CODING )
+ {
+ rpcSlice->setTSResidualCodingDisabledFlag(true);
+ }
+ else
+ {
+ rpcSlice->setTSResidualCodingDisabledFlag(false);
+ }
+#endif
#if SHARP_LUMA_DELTA_QP
pcPic->fieldPic = isField;
diff --git a/source/Lib/EncoderLib/IntraSearch.cpp b/source/Lib/EncoderLib/IntraSearch.cpp
index e850c0f084290c4d0ce583d3589e79329869a4aa..b704c014e98255cf11c0d106028b0190c04eec4d 100644
--- a/source/Lib/EncoderLib/IntraSearch.cpp
+++ b/source/Lib/EncoderLib/IntraSearch.cpp
@@ -448,7 +448,11 @@ bool IntraSearch::estIntraPredLumaQT(CodingUnit &cu, Partitioner &partitioner, c
}
}
+#if JVET_Q0089_SLICE_LOSSLESS_CODING_CHROMA_BDPCM
+ const bool testBDPCM = sps.getBDPCMEnabledFlag() && CU::bdpcmAllowed(cu, ComponentID(partitioner.chType)) && cu.mtsFlag == 0 && cu.lfnstIdx == 0;
+#else
const bool testBDPCM = (sps.getBDPCMEnabled()!=0) && CU::bdpcmAllowed(cu, ComponentID(partitioner.chType)) && cu.mtsFlag == 0 && cu.lfnstIdx == 0;
+#endif
static_vector<ModeInfo, FAST_UDI_MAX_RDMODE_NUM> uiHadModeList;
static_vector<double, FAST_UDI_MAX_RDMODE_NUM> CandCostList;
static_vector<double, FAST_UDI_MAX_RDMODE_NUM> CandHadList;
diff --git a/source/Lib/EncoderLib/VLCWriter.cpp b/source/Lib/EncoderLib/VLCWriter.cpp
index a4e36d7738868a58084e241b564dc055aa793162..6e2a884e07cc4e9a1a6ee7701ac442835707423d 100644
--- a/source/Lib/EncoderLib/VLCWriter.cpp
+++ b/source/Lib/EncoderLib/VLCWriter.cpp
@@ -961,6 +961,9 @@ void HLSWriter::codeSPS( const SPS* pcSPS )
WRITE_FLAG(pcSPS->getTransformSkipEnabledFlag() ? 1 : 0, "sps_transform_skip_enabled_flag");
if (pcSPS->getTransformSkipEnabledFlag())
{
+#if JVET_Q0089_SLICE_LOSSLESS_CODING_CHROMA_BDPCM
+ WRITE_FLAG(pcSPS->getBDPCMEnabledFlag() ? 1 : 0, "sps_bdpcm_enabled_flag");
+#else
WRITE_FLAG(pcSPS->getBDPCMEnabled() ? 1 : 0, "sps_bdpcm_enabled_flag");
#if JVET_Q0110_Q0785_CHROMA_BDPCM_420
if( pcSPS->getBDPCMEnabled() )
@@ -975,11 +978,16 @@ void HLSWriter::codeSPS( const SPS* pcSPS )
{
CHECK(pcSPS->getBDPCMEnabled() == BDPCM_LUMACHROMA, "BDPCM for chroma can be used for 444 only.")
}
+#endif
#endif
}
else
{
+#if JVET_Q0089_SLICE_LOSSLESS_CODING_CHROMA_BDPCM
+ CHECK(pcSPS->getBDPCMEnabledFlag(), "BDPCM cannot be used when transform skip is disabled");
+#else
CHECK(pcSPS->getBDPCMEnabled()!=0, "BDPCM cannot be used when transform skip is disabled");
+#endif
}
WRITE_FLAG( pcSPS->getWrapAroundEnabledFlag() ? 1 : 0, "sps_ref_wraparound_enabled_flag" );
@@ -2265,6 +2273,7 @@ void HLSWriter::codeSliceHeader ( Slice* pcSlice )
#endif
}
}
+
if (pcSlice->getPPS()->getDeblockingFilterControlPresentFlag())
{
@@ -2317,6 +2326,10 @@ void HLSWriter::codeSliceHeader ( Slice* pcSlice )
#endif
}
+#if JVET_Q0089_SLICE_LOSSLESS_CODING_CHROMA_BDPCM
+ WRITE_FLAG(pcSlice->getTSResidualCodingDisabledFlag() ? 1 : 0, "slice_ts_residual_coding_disabled_flag");
+#endif
+
if(pcSlice->getPPS()->getSliceHeaderExtensionPresentFlag())
{
WRITE_UVLC(0,"slice_segment_header_extension_length");