inaturalist / react-native-inat-camera

React Native package that provides a camera with optional species recognition.
MIT License
10 stars 6 forks source link

Possible memory leak in Android AR Camera #29

Closed albullington closed 4 years ago

albullington commented 4 years ago

It looks like there could be a memory leak in the AR camera on Android. Here's a visual from the Google Play Console, which tested an Xperia XZ1 Compact running Android 8.0. The two places where the memory spikes and then stays constant are the times when the AR camera is opened.

Screen Shot 2019-11-20 at 11 34 32 AM

Here's a second visual on a Mate 9 running Android 7.0:

Screen Shot 2019-11-20 at 11 42 10 AM

And finally, here's a Pixel 2 running Android 9.0. This looks a little better since memory usage drops when the user moves from the camera to the photo gallery tab, but overall memory used is still twice as high as it was before the user opened the camera.

Screen Shot 2019-11-20 at 11 46 54 AM
albullington commented 4 years ago

Closing since this library is being rewritten.