menpo / landmarker.io

Image and mesh annotation web application
https://www.landmarker.io
BSD 3-Clause "New" or "Revised" License
114 stars 21 forks source link

purging of large assets #24

Closed jabooth closed 9 years ago

jabooth commented 10 years ago

22f436f adds support for thumbnails to the landmarker for quick loading. Every time an image is opened, we grab the full texture and store it. Going back to a full texture image triggers no extra call.

This raises the problem that in long annotation sessions we may run out of memory by storing all these full size images.

Ideally, ImageSource should keep an eye on the number of full size images stored, and run a cull every now and then to stop things getting out of hand.

A similar strategy should be employed for MeshSource too.

jabooth commented 10 years ago

61 takes a step to fix this. In general we let the browser cache handle the reloading of data efficiently, and we should just get rid of assets every time they are not displayed.

jabooth commented 9 years ago

This is still a problem as all texture information is cached and never released.

jabooth commented 9 years ago

Now improved on the GPU front. Still need to check if textures are fully released or not.

jabooth commented 9 years ago

this is now all fixed as of d17b3a4c053e49c1ea03530b9c8021d337cadc5b