search

meeg / hybrid

0 stars 0 forks source link

Memory leak #1

Closed perhansson closed 9 years ago

perhansson commented 9 years ago

I see the memory being used when running pellet_baseline go to about 2GB while running the loop and then increasing slightly while making plots after. This just scales with more input files which makes me think it's memory leak in the reader (as well). Should check that old baseline analysis that doesn't use my TI data bank hack.

meeg commented 9 years ago

I do see this in pelle_baseline, but I didn't notice it in meeg_all_tp, which is the only other script that runs over multiple EVIO files.

I also see some weird behavior in my code (my code segfaults because it sees events with bad data) that is probably coming from your changes to the EVIO reader. I don't have time to figure this out, but I'm putting this comment here so we don't forget.

meeg commented 9 years ago

I moved your TI bank stuff to a branch. Now meeg_all_tp works, and it uses the same amount of memory independent of the number of files I run over. So I think this must be a problem with your changes.

perhansson commented 9 years ago

Boring

Sent from my phone

On Nov 3, 2015, at 13:58, Sho Uemura notifications@github.com wrote:

I moved your TI bank stuff to a branch. Now meeg_all_tp works, and it uses the same amount of memory independent of the number of files I run over. So I think this must be a problem with your changes.

— Reply to this email directly or view it on GitHub.