Bug 462650

Summary: Ut Video Encoder Not Being Used
Product: [Applications] kdenlive Reporter: vi.kurtz
Component: Rendering & ExportAssignee: Jean-Baptiste Mardelle <jb>
Status: RESOLVED FIXED    
Severity: normal CC: erjiang
Priority: NOR    
Version First Reported In: 22.08.3   
Target Milestone: ---   
Platform: Arch Linux   
OS: Linux   
Latest Commit: https://invent.kde.org/multimedia/kdenlive/commit/05fa0568d1511ab65dae958a20c342204937baca Version Fixed/Implemented In:
Sentry Crash Report:

Description vi.kurtz 2022-12-04 22:45:36 UTC 
SUMMARY
***
NOTE: If you are reporting a crash, please try to attach a backtrace with debug symbols.
See https://community.kde.org/Guidelines_and_HOWTOs/Debugging/How_to_create_useful_crash_reports
***
When using the Ut Video lossless encoder, the resulting file is not Ut encoded. It is instead rendered as an H.264 file with medium quality settings.

STEPS TO REPRODUCE
1. Import video to timeline
2. Render as Ut Video (utvideo+pcm_s24le)

OBSERVED RESULT
H.264 medium quality encoded output.

EXPECTED RESULT
Ut lossless encoded output.

SOFTWARE/OS VERSIONS
Linux/KDE Plasma: Linux 6.0.11
(available in About System)
KDE Plasma Version: 5.26.4
KDE Frameworks Version: 5.100.0
Qt Version: 5.15.7

ADDITIONAL INFORMATION
Looking at the generated .mlt file, the consumer tag has:
<consumer properties="lossless/Ut" ...>
Looking at the settings in the render window when selecting Ut, I see
properties=lossless/Ut Video channels=2 ...
I suspect that the space in "lossless/Ut Video" is resulting in "Video" being cut off, which may be resulting in the codec not being located properly and the renderer falling back to a default.
Comment 1 erjiang 2022-12-07 04:27:38 UTC 
Can confirm on Mac (release/22.12, built 2022-11-25) that the utvideo preset creates an h.264 video. Did not look into the parameters yet.
Comment 2 Jean-Baptiste Mardelle 2022-12-08 05:48:05 UTC 
Git commit 1c50d7d88de316da6a82841498006046b61fd73d by Jean-Baptiste Mardelle.
Committed on 08/12/2022 at 05:47.
Pushed by mardelle into branch 'master'.

Fix rendering when using an MLT properties file with a space in it

M  +4    -4    src/dialogs/renderwidget.cpp
M  +3    -3    src/renderpresets/renderpresetrepository.cpp

https://invent.kde.org/multimedia/kdenlive/commit/1c50d7d88de316da6a82841498006046b61fd73d
Comment 3 Jean-Baptiste Mardelle 2022-12-13 10:51:33 UTC 
Git commit 05fa0568d1511ab65dae958a20c342204937baca by Jean-Baptiste Mardelle.
Committed on 13/12/2022 at 10:43.
Pushed by mardelle into branch 'release/22.12'.

Fix rendering when using an MLT properties file with a space in it

M  +4    -4    src/dialogs/renderwidget.cpp
M  +3    -3    src/renderpresets/renderpresetrepository.cpp

https://invent.kde.org/multimedia/kdenlive/commit/05fa0568d1511ab65dae958a20c342204937baca
Comment 4 Julius Künzel 2022-12-18 22:07:23 UTC 
Git commit 3a5c22910267c490960c36179f2933b3c119c2eb by Julius Künzel.
Committed on 18/12/2022 at 22:06.
Pushed by jlskuz into branch 'master'.

[Render Settings] Reduce conversion between String and Map

Using a single string for render parameters caused several issues in the
past. This should reduce the risk for bugs.

Related to a1304676a61959783884f90473fd2306a6ffd835 and
05fa0568d1511ab65dae958a20c342204937baca
Related: bug 458718

M  +2    -2    src/dialogs/renderpresetdialog.cpp
M  +64   -72   src/dialogs/renderwidget.cpp
M  +3    -1    src/dialogs/renderwidget.h
M  +32   -15   src/renderpresets/renderpresetmodel.cpp
M  +9    -3    src/renderpresets/renderpresetmodel.hpp
M  +3    -3    src/renderpresets/renderpresetrepository.cpp
M  +27   -8    tests/rendermodeltest.cpp

https://invent.kde.org/multimedia/kdenlive/commit/3a5c22910267c490960c36179f2933b3c119c2eb