Audacity Bug Summary
••• Introduction •••
••• Keywords •••
    Audacity 3.0.3 development began 19th April 2021

Audacity Bugzilla



Bug 1617 - Clipfix very slow
Clipfix very slow
Status: RESOLVED QUICKFIXED
Product: Audacity
Classification: Unclassified
Component: Nyquist
unspecified
Per OS Linux
: P4 Repeatable
Assigned To: Steve Daulton
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2017-03-28 16:39 UTC by Steve Daulton
Modified: 2018-08-20 11:45 UTC (History)
5 users (show)

See Also:
Steps To Reproduce:
Release Note:
First Git SHA:
Group: ---
Workaround:
Closed: 2018-08-20 00:00:00
gale: Test‑OK‑Win?
gale: Test‑OK‑Mac?
gale: Test‑OK‑Lin?


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Steve Daulton 2017-03-28 16:39:16 UTC
Clipfix is significantly slower than it should be due to repeated passes over the same data.

Assigned to myself.
Comment 1 Steve Daulton 2017-03-30 19:55:40 UTC
https://github.com/audacity/audacity/commit/a9879bddf

Tested on Linux. Clipfix is still a slow effect, which is unavoidable, but the duplicate pass has been eliminated, giving about a 30% increase in speed on my machine.
Comment 2 Gale Andrews 2017-04-12 23:11:43 UTC
First, a general "thank you" to Steve for all the Clipfix improvements. 

Testing with individual tracks up to 2 hours long (44100 Hz) on the different platforms, and then with multiple tracks, I found Sierra Mac mini and Win 10 laptop about 75% to sometimes 100% faster than before, and the slow Ubuntu 14.04 netbook about one-third faster. 

The improvement was also seen with low (8000 Hz) and high (192000 Hz) tracks, and when track and project rate differed. 

I left a query open on the Test OK flags because the speed improvement tails off rapidly when threshold is below about 50% (only tested with 44100 Hz tracks but I see it on all platforms. At 20% threshold, 2.1.3 and 2.2.0 run at about the same slow speed. I don't know why anyone would want such a low threshold - it seems to give poor results to me.
Comment 3 Steve Daulton 2017-04-13 05:29:49 UTC
(In reply to Gale Andrews from comment #2)
> the speed improvement tails off rapidly when threshold is below about 50%

That's not surprising because, with a very low threshold settings a high proportion of the waveform will be marked as 'clipped' in the first pass, so the peak reconstruction pass will dominate the processing time.
Comment 4 Gale Andrews 2017-04-13 21:06:22 UTC
(In reply to Steve Daulton from comment #3)
>> (In reply to Gale Andrews from comment #2)
>> the speed improvement tails off rapidly when threshold is below about 50%
> That's not surprising because, with a very low threshold settings a high proportion
> of the waveform will be marked as 'clipped' in the first pass, so the peak 
> reconstruction pass will dominate the processing time. 
OK so on that basis it sounds as if we can just resolve this now.