Bug 359132 - Git master 2016-02-08 - Audio power meter does not work for timeline
Summary: Git master 2016-02-08 - Audio power meter does not work for timeline
Status: RESOLVED FIXED
Alias: None
Product: kdenlive
Classification: Applications
Component: User Interface & Miscellaneous (show other bugs)
Version: unspecified
Platform: Arch Linux Linux
: NOR normal
Target Milestone: ---
Assignee: Jean-Baptiste Mardelle
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-02-08 12:58 UTC by Evert Vorster
Modified: 2016-03-29 09:38 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Evert Vorster 2016-02-08 12:58:21 UTC
Just noticed, when I watch a clip in the clip monitor, the audio power meter works fine. When watching that same clip on the timeline with the Project monitor, the audio power meter is greyed out, and does not appear to work. 



Reproducible: Always


Actual Results:  
Audio power meter does not work in the Project monitor

Expected Results:  
Audi power meter to work everywhere

I split the audio from clips automatically, not sure if this has something to do with it. 
Nah, just tested, the Project Monitor's audio power meter also does not work even when the audio is not split out.
Comment 1 Unknown 2016-02-08 22:22:38 UTC
I can confirm this issue. When I play a clip in the clip monitor, the audio levels meter works fine. But when I insert the clip into the timeline and playback the project, the project monitor audio level meter doesn't work at all. Instead, it plays on the clip monitor instead.
Comment 2 Jean-Baptiste Mardelle 2016-02-08 23:49:37 UTC
Git commit 9177c05d5c014045a1c015adc1d80777c1c87b25 by Jean-Baptiste Mardelle.
Committed on 08/02/2016 at 23:44.
Pushed by mardelle into branch 'master'.

Fix audio level monitor activated on wrong monitor

M  +3    -4    src/monitor/monitor.cpp
M  +1    -1    src/monitor/monitor.h
M  +7    -1    src/monitor/monitormanager.cpp

http://commits.kde.org/kdenlive/9177c05d5c014045a1c015adc1d80777c1c87b25
Comment 3 Evert Vorster 2016-02-09 12:54:39 UTC
I just re-built kdenlive, and still the audio power meter does not work on the project monitor. 

https://youtu.be/DFgPahItlUA

Has a short clip showing the issue
Comment 4 Jean-Baptiste Mardelle 2016-02-09 21:34:04 UTC
Oh, I think I know what the problem is. Evert, there is a small but in the audio level that causes it to appear when user wants to disable it. So my gues is that audio level is disabled for your project monitor.

Can you go in the project monitor menu button (left of the audio level) and check if "Show Audio Levels" is enabled ?
Comment 5 Jean-Baptiste Mardelle 2016-02-09 22:48:49 UTC
Git commit af7080d1617e964bdee5014d6569ccf34781eec3 by Jean-Baptiste Mardelle.
Committed on 09/02/2016 at 22:48.
Pushed by mardelle into branch 'master'.

Fix audio level monitor visibility

M  +0    -1    src/dialogs/kdenlivesettingsdialog.cpp
M  +16   -9    src/monitor/monitor.cpp
M  +18   -2    src/monitor/scopes/monitoraudiolevel.cpp
M  +3    -1    src/monitor/scopes/monitoraudiolevel.h

http://commits.kde.org/kdenlive/af7080d1617e964bdee5014d6569ccf34781eec3
Comment 6 Unknown 2016-02-10 05:50:42 UTC
The latest commit fixed the problem for me -- using the latest git master build.
Comment 7 Evert Vorster 2016-02-10 11:05:13 UTC
JBM, you were right, of course. 
The "Show audio levels" was disabled for the project monitor. 

Before the last commit, though, I could see the audio levels greyed out. After the fix there was just nothing in that window until I enabled it. 

Thanks for the quick fix, and your patience.
Comment 8 Paul Konecny 2016-03-29 09:38:22 UTC
I just wanted to file a bug for this behaviour in 16.04-af89183
Thanks for the clarification. 
Just one question: Would it be possible to enable the audio levels by default for the project monitor?
I think less people would get confused that way. 
Cheers!