Skip to content
This repository has been archived by the owner on Jul 29, 2024. It is now read-only.

Add crf ratecontrol support #70

Open
wants to merge 106 commits into
base: master
Choose a base branch
from

Conversation

kirithika7
Copy link

Add new rate control mode-crf that takes the cli -rc 2 and -crf value in [0-51]

Add new rate control mode-crf that takes the cli -rc  2 and -crf  value in [0-51]
@tianjunwork tianjunwork added the enhancement New feature or request label Jan 11, 2019
# Conflicts:
#	Config/Sample.cfg
#	Docs/svt-hevc_encoder_user_guide.md
#	Source/API/EbApi.h
#	Source/App/EbAppConfig.c
#	Source/App/EbAppConfig.h
#	Source/App/EbAppContext.c
#	Source/Lib/Codec/EbEncDecProcess.c
#	Source/Lib/Codec/EbEncHandle.c
#	Source/Lib/Codec/EbEncodeContext.c
#	Source/Lib/Codec/EbEncodeContext.h
#	Source/Lib/Codec/EbEntropyCoding.h
#	Source/Lib/Codec/EbPacketizationProcess.c
#	Source/Lib/Codec/EbPacketizationReorderQueue.c
#	Source/Lib/Codec/EbPacketizationReorderQueue.h
#	Source/Lib/Codec/EbPictureControlSet.c
#	Source/Lib/Codec/EbPictureDecisionProcess.c
#	Source/Lib/Codec/EbRateControlProcess.c
…edictBitsDup() in EbEncDecProcess.c

Signed-off-by: kirithika <[email protected]>
EncodeLcu

This commit also has some cleanups done

Signed-off-by: kirithika <[email protected]>
…avoid underflow while removing a single frame

Signed-off-by: kirithika <[email protected]>
Signed-off-by: kirithika <[email protected]>

# Conflicts:
#	Source/Lib/Codec/EbEncDecProcess.c
#	Source/Lib/Codec/EbPictureControlSet.c
@tianjunwork
Copy link
Contributor

Hi @kirithika7, is there a way to use for example CRF=20 encoding, but also limit the maximum bitrate? If yes, could you add more usage example in the user guide? Thx.

@thjr
Copy link

thjr commented Feb 7, 2020

Is this going forward?

@Austin-Hu
Copy link
Contributor

Is this going forward?

Hi @thjr,

In order to merge the CRF implementation, we need to make sure that there isn't PSNR/SSIM BDRate regression. But due to the PSNR/SSIM BDRate drop issue caused by another PR #348 , we planed to fix it at first, then go ahead to merge this CRF PR. Thanks!

@shalin186
Copy link

Any updates on this PR? It has been opened for a long time.

@mxsrm
Copy link

mxsrm commented Aug 21, 2021

Any updates on this PR? It has been opened for a long time.

I'm pretty sure development on svt-hevc is dead.

@bigfatbrowncat
Copy link

Hi! What about this PR? What is needed to be fixed to upmerge it?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.