streets-gl icon indicating copy to clipboard operation
streets-gl copied to clipboard

Performance improvement

Open ManDay opened this issue 2 years ago • 2 comments

streets.gl is rather hard on the CPU/GPU. I think an easy remedy to that, and to the loading time, could be reducing texture size. Given how beautiful streets.gl already is, I hope this can move a bit up on the priority list, as it feels like the most impactful limitation currenty. Most textures could probably be dealt at a fraction (1/10 to 1/100) of their size and quality, seen how people usually don't want to use streets.gl to zoom "1 meter" close to buildings.

For example, there is a 1.36 Mb normal map for a brick wall in the reposity (I don't know whether it's actually being used, but to illustrate the point): https://github.com/StrandedKitty/streets-gl/blob/dev/src/resources/textures/buildings/facades/block_wall_normal.png . I don't think anyone is ever going to zoom in to the point where that normal map becomes relevant, much less where we would need such a "high" resolution.

Thanks!

ManDay avatar Oct 26 '23 17:10 ManDay

This file is not send to the browser with this size:

https://github.com/StrandedKitty/streets-gl/issues/37#issuecomment-1546648390

HolgerJeromin avatar Oct 27 '23 04:10 HolgerJeromin

That's a good improvement but regardless, from the files that are downloaded according to the FF Inspector and from the on-screen quality, the textures are still an overkill, in my opinion. Zooming in to such a detail is just not expected, as are normal maps of centimeter resolution:

image

I'm not convinced texture quality is the actual bottleneck; I was just pointing this out as one possible cause for the large performance drag.

ManDay avatar Oct 28 '23 14:10 ManDay