search

mutability / dump1090

Dump1090 is a simple Mode S decoder for RTLSDR devices
526 stars 136 forks source link

Web UI: parallel loading of history (speed up) #206

Closed smartynov closed 5 years ago

smartynov commented 7 years ago

I noticed that html page takes a lot of time to load, primarily caused by sequential requests for loading history. I propose this patch to enable loading history in parallel, which results in significant speed up of page loading time. We just start all requests using jquery and let the browser handle concurrency. Chrome allows for 4 concurrent requests, some older browsers may allow 2, but no browser will run all the request at once, so server-side will be safe.

smartynov commented 7 years ago

P.S. I have no idea why checks failed, it seems to be unrelated to my changes

OrbitTheSun commented 7 years ago

A very noticeable speed up. - Thank you for this improvement!

smartynov commented 6 years ago

Closing to re-open (hope it will pass test now)

smartynov commented 6 years ago

The test environment seems to be broken for this project, failing in C code build... As I see no changes in javascript file since last year, I still think it is a good idea to accept this PR.