| Summary: | Extract and save zone not working | ||
|---|---|---|---|
| Product: | [Applications] kdenlive | Reporter: | stefano.droghetti |
| Component: | Rendering & Export | Assignee: | Jean-Baptiste Mardelle <jb> |
| Status: | RESOLVED WORKSFORME | ||
| Severity: | normal | CC: | fritzibaby, inivyro, scubbx, snd.noise |
| Priority: | NOR | Flags: | fritzibaby:
Brainstorm+
|
| Version First Reported In: | Appimage - Refactoring | ||
| Target Milestone: | --- | ||
| Platform: | Other | ||
| OS: | Linux | ||
| Latest Commit: | Version Fixed/Implemented In: | ||
| Sentry Crash Report: | |||
|
Description
stefano.droghetti
2019-09-16 13:42:24 UTC
I also get this error. The language of my OS is "German". The reason might be that the decimal separator value for SS used by Kdenlive is a comma and not a dot as expected by ffmpeg. (In reply to scubbx from comment #1) > I also get this error. The language of my OS is "German". > The reason might be that the decimal separator value for SS used by Kdenlive > is a comma and not a dot as expected by ffmpeg. I looked around a bit and found the following in the source: ---- In the file src/jobs/cutclipjob.cpp the variable "startString" (used for extraction of the clip) is created by "m_in.seconds()" : QString startString = locale.toString(m_in.seconds()); ---- The same is true for the "durationString" in src/jobs/cutclipjob.cpp : QString durationString = locale.toString((m_out - m_in).seconds()); ---- Both of these variables are used to create the FFMPEG call for clipping in the file src/jobs/cutclipjob.cpp: QStringList params = {QStringLiteral("-y"),QStringLiteral("-stats"),QStringLiteral("-i"),m_sourceUrl,QStringLiteral("-v"),QStringLiteral("error"),QStringLiteral("-ss"),startString, QStringLiteral("-t"), durationString}; Confirmed. Has nothing to do with coma or point setting (checked on Windows). Git commit d854b4f22af26e5211efc465be0e237287ea4307 by Jean-Baptiste Mardelle. Committed on 30/12/2019 at 14:40. Pushed by mardelle into branch 'master'. Fix extract zone. M +1 -2 src/jobs/cutclipjob.cpp https://invent.kde.org/kde/kdenlive/commit/d854b4f22af26e5211efc465be0e237287ea4307 "Save zone" and "Extract zone" in project monitor still not working. Hi, In KDE Plasma System Settings, changing "Regional Settings"->"Formats" from Spanish to US_English fixes the bug. The problem seems to be, as pointed by "scubbx", variables startString and durationString. Removing locale.toString in both could fix the error. Git commit efda53588d152ca6aa3baf85e04780f4f2cc0fd9 by Jean-Baptiste Mardelle. Committed on 02/01/2020 at 12:27. Pushed by mardelle into branch 'master'. Fix extract zone (improve ffmpeg arguments and drop locale specific seconds conversion) M +1 -3 src/jobs/cutclipjob.cpp https://invent.kde.org/kde/kdenlive/commit/efda53588d152ca6aa3baf85e04780f4f2cc0fd9 "Extract zone" in project monitor is working. "Save zone" is not implemented so far. Git commit ebed22b1c827f0e4988c51dfa99f99836613176d by Jean-Baptiste Mardelle. Committed on 06/01/2020 at 07:49. Pushed by mardelle into branch 'release/19.12'. Fix extract zone (improve ffmpeg arguments and drop locale specific seconds conversion) M +1 -3 src/jobs/cutclipjob.cpp https://invent.kde.org/kde/kdenlive/commit/ebed22b1c827f0e4988c51dfa99f99836613176d Can we close this one? Dear Bug Submitter, This bug has been in NEEDSINFO status with no change for at least 15 days. Please provide the requested information as soon as possible and set the bug status as REPORTED. Due to regular bug tracker maintenance, if the bug is still in NEEDSINFO status with no change in 30 days the bug will be closed as RESOLVED > WORKSFORME due to lack of needed information. For more information about our bug triaging procedures please read the wiki located here: https://community.kde.org/Guidelines_and_HOWTOs/Bug_triaging If you have already provided the requested information, please mark the bug as REPORTED so that the KDE team knows that the bug is ready to be confirmed. Thank you for helping us make KDE software even better for everyone! |