View Bug Details

IDProjectCategoryView StatusLast Update
0002616DCP-o-maticFeaturespublic2024-04-17 08:03
ReporterCarsten Assigned Tocarl  
PrioritynormalSeverityfeatureReproducibilityalways
Status confirmedResolutionopen 
PlatformMacOSOS XOS Version10.12
Product Version2.16.65 
Target Version2.16.x 
Summary0002616: audio analysis has become very slow in 2.16.x/2.17.x
Description

It takes around five times longer than in 2.14.59 (using the same content)

Typical analysis time in 2.14.59 was around 3-4min for a full length feature, while in 2.16.65 it takes 15-20 minutes.

TagsNo tags attached.
Branch
Estimated weeks required
Estimated work requiredUndecided

Activities

carl

2023-09-30 12:28

administrator   ~0005973

Last edited: 2023-10-01 21:54

audio_analysis_test seems to take just over 2s in 2.14.59 non-debug build on laptop (though I think this was without ebur128 due to a mistake when building)

Carsten

2023-09-30 12:53

manager   ~0005974

Since when is Leq(m) in it?

carl

2023-09-30 20:27

administrator   ~0005975

I'll check... on main the same test takes 14s...

carl

2023-09-30 20:29

administrator   ~0005976

Removing _leqm->add(interleaved) from main brings it down to 5s.

carl

2023-10-01 21:53

administrator   ~0005977

Using a PeriodTimer around the main pass() loop:

Removed _leqm->add() from main,
main 3.13 2.14.x 1.93 62%

Removed _ebur128->process call() from both,
removed _leqm->add() from main,
main 0.436s 2.14.x 0.378s 86%

carl

2023-10-01 22:15

administrator   ~0005978

Last edited: 2023-10-01 22:15

Looks like building leqm_nrt with -O2 is about 4 times faster (judging by test.sh in that repo). Probably also worth looking at doing the convolution there with FFT.

carl

2023-10-01 22:17

administrator   ~0005979

Also leqm_nrt seems to loop over channels, so maybe we don't need to interleave the data before passing it in.

carl

2023-10-01 22:55

administrator   ~0005980

See opt branch on leqm_nrt

carl

2023-10-08 22:05

administrator   ~0005996

Merged the -O2 change as e7a04275703b8cd54e1325d42f484ed7d0acab92

Carsten

2024-03-17 01:38

manager   ~0006354

Last edited: 2024-03-17 01:41

Seems to work a lot faster now, at least in 2.17.13 - full length (2hrs) 5.1 audio incl. LUFS/LEQ in about 5 minutes.

Bug History

Date Modified Username Field Change
2023-09-26 00:39 Carsten New Bug
2023-09-26 00:39 Carsten Status new => assigned
2023-09-26 00:39 Carsten Assigned To => carl
2023-09-26 00:41 Carsten Description Updated
2023-09-26 00:41 Carsten Estimated work required => Undecided
2023-09-26 00:47 Carsten Description Updated
2023-09-30 12:28 carl Note Added: 0005973
2023-09-30 12:53 Carsten Note Added: 0005974
2023-09-30 20:27 carl Note Added: 0005975
2023-09-30 20:29 carl Note Added: 0005976
2023-10-01 21:53 carl Note Added: 0005977
2023-10-01 21:54 carl Note Edited: 0005973
2023-10-01 22:15 carl Note Added: 0005978
2023-10-01 22:15 carl Note Edited: 0005978
2023-10-01 22:17 carl Note Added: 0005979
2023-10-01 22:55 carl Note Added: 0005980
2023-10-08 22:05 carl Note Added: 0005996
2023-10-08 22:06 carl Target Version 2.16.66 => 2.16.x
2024-01-03 00:14 carl Status assigned => confirmed
2024-01-03 11:53 carl Severity major => feature
2024-03-17 01:38 Carsten Note Added: 0006354
2024-03-17 01:41 Carsten Note Edited: 0006354