gianni-rosato / svt-av1-psy

The Scalable Video Technology for AV1 (SVT-AV1 Encoder and Decoder) with perceptual enhancements for psychovisually optimal AV1 encoding
https://svt-av1-psy.com
BSD 3-Clause Clear License
251 stars 20 forks source link

[ENHANCEMENT] Add eta and est. size in encoding progress status #17

Closed bex39 closed 9 months ago

bex39 commented 9 months ago

Is your feature request related to a problem? Please describe. --progress 2 (aom style) is only show frames, bitrate, and fps

Describe the solution you'd like It would be nice if add info such as time elapsed/eta and est. size

Describe alternatives you've considered SvtAv1EncApp (Patman's build) that included in staxrip has mentioned feature, https://github.com/Patman86/SVT-AV1-Mod-by-Patman/commit/670343f257a1c79bfcdb876ca23021886cd85de0 hope it can be adopted to SVT-AV1-Psy

Additional context Add any other context or screenshots about the feature request here. Screenshot 2024-02-11 113539

Screenshot 2024-02-14 211841

Thanks.

shssoichiro commented 9 months ago

If this is implemented, please be careful that it doesn't break av1an's parsing of the svt command line. We should try to keep this backwards compatible with mainline svt.

gianni-rosato commented 9 months ago

I'm in favor of implementing this as a separate --progress 3 option instead of overriding progress 2. I am a fan of SVT-AV1-PSY attempting to preserve compatibility with mainline to a certain extent, because there could be people using the output of vanilla progress 2 in a specific way where changing it would cause issues for their workflows.

I'll look into merging this to testing & implementing a progress 3 when I have more time

gianni-rosato commented 9 months ago

e192425d49e4da053b8c8c605cfd8ba377b50f85

Enjoy! This will be merged to the master branch with our first release!

bex39 commented 9 months ago

e192425

Enjoy! This will be merged to the master branch with our first release!

Nice, thank you 👍