Minim memory leak related to AudioRecorder

Core Library Questions

hapticdata

Minim memory leak related to AudioRecorder

in Core Library Questions • 3 years ago

I'm making an application that begins recording once it receives an audio signal above a threshold and ends recording once that falls below the threshold for a few seconds straight. I have set a maximum recording length to 5 minutes consecutively so that it doesn't run out of memory, yet after several recordings (last crash was 10 recordings x 5 minutes each) I get an OutOfMemoryError as it writes to the saved file.

Here is the error (minus the Processing added part):

Exception in thread "Animation Thread" java.lang.OutOfMemoryError: Java heap space

at org.tritonus.share.sampled.FloatSampleBuffer.convertToByteArray(FloatSampleBuffer.java:394)

at ddf.minim.javasound.JSBufferedSampleRecorder.save(JSBufferedSampleRecorder.java:144)

at ddf.minim.AudioRecorder.save(AudioRecorder.java:107)

at AOVAudioRecord.completeRecord(AOVAudioRecord.java:213)

at AOVAudioRecord.draw(AOVAudioRecord.java:136)

at processing.core.PApplet.handleDraw(PApplet.java:1606)

at processing.core.PApplet.run(PApplet.java:1503)

at java.lang.Thread.run(Thread.java:637)

two relevant functions:

void initRecord()
{
currentFilename = getTimestamp() + ".wav";
recorder = minim.createRecorder(in, currentFilename, true);
worthSavingCounter.reset();
recorder.beginRecord();
}

void completeRecord()
{
recorder.endRecord();
recorder.save();
println("Done saving.");
addRecentFile(currentFilename);
currentRecordingLength = 0;
nRecordings++;
}

I can post all of the code if that helps. It seems clear its from saving the file, which is maxed out at 5 minutes in length, and only happens after it has recorded around 50 minutes worth... any suggestions for how I can fix this? I need this to run for 10 days straight

Replies(5)

hapticdata

Re: Minim memory leak related to AudioRecorder

3 years ago

I dramatically improved it just by setting the createRecorder()'s boolean buffered to false.

Currently the application has been running for about 15 hours, I've had constant audio input so it has recorded 230 wav files limited to 5 minutes each.

only problem is, the memory usage is still creeping upwards. 15 hours ago the application was using about 55mb, now its using 279.4mb. So with its current memory limit it would crash a little more than once every 3 days. Though this is not actually likely, because there isnt going to be constant audio to record like in my tests, it still is a vulnerability for my installation.

Any ideas how to fix this leak?

PhiLho

Re: Minim memory leak related to AudioRecorder

3 years ago

Hard to tell without seeing the source code of Minim...
As a wild guess, you can try and set recorder to null after save() call. It shouldn't be necessary, but who knows?

Note: you should try and get JVisualVM, a free tool (found in the JDK but also available separately), it can allow to track memory usage.

niguloto

Re: Minim memory leak related to AudioRecorder

1 year ago

Hi I have a similar problem with a minim memory leak but in slight different configuration. I didn't know if it was the case to open a new topic, but anyhow: I'm using processing to control arduino where 2 photoresistors track the ammount of light and if it is over a certain ammount they will turn on their respective LED on the arduino board and play a random track from a pool of 10 different .aiff files.

I'm quite a noob and I might just be using the wrong option to select from a randomic pool of audio files in minim, but that's the only solution I came up with yesterday...obviously minim now does not seem to be able to track down the files in order to clear the memory.

Here's the code, any help? or any solution to achieve the same result in a more elegant way?

import ddf.minim.*;
import ddf.minim.signals.*;
import ddf.minim.analysis.*;
import ddf.minim.effects.*;
import processing.video.*;
import processing.serial.*;
import cc.arduino.*;
Serial myPort; // The Serial Port
Arduino scheda;
boolean ledA = false;
int phoA = 0;
boolean ledB = false;
int phoB = 0;
int brightness = 255;
int fade = 1;
float heightValA;
float heightValB;
int x = 0;
int px= 0;
int delayVal = 0;
Minim audio = new Minim(this);
AudioPlayer track;
void setup(){
scheda = new Arduino(this, Arduino.list()[1], 57600);
size(800, 600);
rectMode(CENTER);
noStroke();
}
void draw() {
//set a random track to play
int randomTrack = (int)random(1, 11);
track = audio.loadFile("voce" + randomTrack + ".aiff");
//read analog pin from arduino (photoresistors)
phoA = scheda.analogRead(0);
phoB = scheda.analogRead(1);
//map photoresistors val to fit window size
heightValA = map(phoA, 0,1023, 0, height);
heightValB = map(phoB, 0,1023, 0, height);
//delay time: check every 1sec
if (millis() - delayVal > 1000) {
if (phoA < 100 && ledB == false && phoB > 100) {
ledA = true;
track.play();
delayVal = millis();
} else if (phoB < 100) {
ledA = false;
ledB = true;
track.play();
delayVal = millis();
} else if (phoA < 100 && ledB == true) {
ledA = true;
ledB = false;
track.play();
delayVal = millis();
} else if (brightness == 0) {
ledA = false;
ledB = false;
brightness = 255;
}
}
//graphic placeholder to track photoresistors light values
noStroke();
fill(225, 10);
rect(width/2, height/2, width, height);
x = x+1;
if (x > width) {
x = 0;
}
stroke(250, 100, 0);
line(x, heightValA, px, heightValA);
stroke(0, 100,250);
line(x, heightValB, px, heightValB);
println(phoA);
ledState();
px = x;
}
void ledState() {
// LED state A
if (ledA == true) {
scheda.digitalWrite(6, Arduino.HIGH);
noStroke();
fill(30, phoA);
rect(width/4,height/2, 10, 10);
// if (brightness > 0) {
// brightness -= fade;
// } else if (brightness <= 0) {
// brightness = 0;
// }
} else if (ledA == false) {
scheda.digitalWrite(6, Arduino.LOW);
}
// LED state B
if (ledB == true) {
scheda.digitalWrite(7, Arduino.HIGH);
noStroke();
fill(30, phoB);
rect(width-width/4,height/2, 10, 10);
// if (brightness > 0) {
// brightness -= fade;
// } else if (brightness <= 0) {
// brightness = 0;
// }
} else if (ledB == false) {
scheda.digitalWrite(7, Arduino.LOW);
}
}
//minim
public void stop()
{
track.close();
audio.stop();
super.stop();
}

and here follow the error message I get in processing, thanks in advance for any help you can give me!!

An OutOfMemoryError means that your code is either using up too much memory

because of a bug (e.g. creating an array that's too large, or unintentionally

loading thousands of images), or that your sketch may need more memory to run.

If your sketch uses a lot of memory (for instance if it loads a lot of data files)

you can increase the memory available to your sketch using the Preferences window.

Exception in thread "Animation Thread" java.lang.OutOfMemoryError: unable to create new native thread

at java.lang.Thread.start0(Native Method)

at java.lang.Thread.start(Thread.java:658)

at ddf.minim.javasound.JSBaseAudioRecordingStream.open(JSBaseAudioRecordingStream.java:399)

at ddf.minim.AudioSource.<init>(AudioSource.java:90)

at ddf.minim.AudioPlayer.<init>(AudioPlayer.java:57)

at ddf.minim.Minim.loadFile(Minim.java:360)

at ddf.minim.Minim.loadFile(Minim.java:341)

at tesiProj_audioLight.draw(tesiProj_audioLight.java:71)

at processing.core.PApplet.handleDraw(PApplet.java:1906)

at processing.core.PApplet.run(PApplet.java:1803)

at java.lang.Thread.run(Thread.java:680

niguloto

Re: Minim memory leak related to AudioRecorder

1 year ago

to make it easier this should be the "problematic part", where I assign the .aiff files every loop :/ and then I check if the conditions are "just right" so I call track.play()

void draw() {
//set a random track to play
randomTrack = (int)random(1, 11);
track = audio.loadFile("voce" + randomTrack + ".aiff");
//read analog pin from arduino (photoresistors)
phoA = scheda.analogRead(0);
phoB = scheda.analogRead(1);
//map photoresistors val to fit window size
heightValA = map(phoA, 0,1023, 0, height);
heightValB = map(phoB, 0,1023, 0, height);
//delay time: check every 1sec
if (millis() - delayVal > 1000) {
if (phoA < 100 && ledB == false && phoB > 100) {
ledA = true;
track.play();
delayVal = millis();
} else if (phoB < 100) {
ledA = false;
ledB = true;
track.play();
delayVal = millis();
} else if (phoA < 100 && ledB == true) {
ledA = true;
ledB = false;
track.play();
delayVal = millis();
} else if (brightness == 0) {
ledA = false;
ledB = false;
brightness = 255;
}
}

of course if i move

track = audio.loadFile("voce" + randomTrack + ".aiff");

inside each "if" like so

if (millis() - delayVal > 1000) {
if (phoA < 100 && ledB == false && phoB > 100) {
ledA = true;
track = audio.loadFile("voce" + randomTrack + ".aiff");
track.play();
delayVal = millis();
} else if (phoB < 100) {
ledA = false;
ledB = true;
track = audio.loadFile("voce" + randomTrack + ".aiff");
track.play();
delayVal = millis();
} else if (phoA < 100 && ledB == true) {
ledA = true;
ledB = false;
track = audio.loadFile("voce" + randomTrack + ".aiff");
track.play();
delayVal = millis();
}
}

it's not called 60 times per second but just 1 per second maximum; nonetheless I believe it'll lead to a memory leak just further in time...so it might crash again in 3hrs instead that in 30sec...any suggestion?

here's a screen taken from JVisualVM:

but I don't really know how to read it other than threads don't die!!

niguloto

Re: Minim memory leak related to AudioRecorder

1 year ago

No luck I guess :/

Does anyone even have any suggestion on how to do this differently? like with an array or something and minim classes?

Thanks anyway :)

Top Reply