guardianproject / haven

Haven is for people who need a way to protect their personal spaces and possessions without compromising their own privacy, through an Android app and on-device sensors
https://guardianproject.github.io/haven/
GNU General Public License v3.0
6.61k stars 726 forks source link

OOM error while converting a luminance matrix to a RGB grayscale bitmap #274

Closed archie94 closed 6 years ago

archie94 commented 6 years ago

Crashlog

java.lang.OutOfMemoryError: Failed to allocate a 8294412 byte allocation with 6502296 free bytes and 6MB until OOM
 at org.havenapp.main.sensors.media.ImageCodec.lumaToGreyscale(ImageCodec.java:44)
 at org.havenapp.main.sensors.media.ImageCodec.lumaToBitmapGreyscale(ImageCodec.java:55)
 at org.havenapp.main.sensors.media.MotionAsyncTask.run(MotionAsyncTask.java:102)
n8fr8 commented 6 years ago

Should be fixed in 0.2.0 alpha 2... reduced preview size and improved memory usage overall

lukeswitz commented 6 years ago

Seems to be fixed, tested simulator v5.1-P & devices Nexus 4 5.1 & Pixel P. Could not reproduce, memory use down 40% in preview! Excellent

jbizzle2 commented 6 years ago

jbwolf On Jul 6, 2018 9:17 AM, "Nathan Freitas" notifications@github.com wrote:

Should be fixed in 0.2.0 alpha 2... reduced preview size and improved memory usage overall

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/guardianproject/haven/issues/274#issuecomment-403031233, or mute the thread https://github.com/notifications/unsubscribe-auth/AmDdVFWODgg2Ev2d0YkIxSiUbbiPlgjoks5uD2NUgaJpZM4S6GEq .

jbizzle2 commented 6 years ago

jbwolf On Jul 6, 2018 9:17 AM, "Nathan Freitas" notifications@github.com wrote:

Should be fixed in 0.2.0 alpha 2... reduced preview size and improved memory usage overall

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or mute the thread.