ecc521 / rivers.run

Water levels, notifications, search tools, and more, built to help paddlers.
https://rivers.run
7 stars 1 forks source link

Serious Lag expanding Niagara river (and others) due to graph #108

Closed ecc521 closed 2 years ago

ecc521 commented 4 years ago

The Niagara river (https://rivers.run/#Niagara%20Fort%20Erie) reports data extremely frequently, so graph contains several hundred points. This results in a multi second lag opening the river view once full data has been loaded (the initial load is only a few data points, so works at normal speed. Once full data is loaded, full data is drawn on future openings, resulting in heavy lag)

ecc521 commented 3 years ago

This is now significantly more important with the NWS gauges - offscreen canvas should probably be used

ecc521 commented 3 years ago

d1d5a1ca massively mitigates this issue, making it far less catastrophic. Backgrounding should be done, but it is now an enhancement, not a critical issue.

ecc521 commented 2 years ago

This issue is more serious with additional USGS data as of 0ccaf5e7, and impedes the potential addition of more data (like 7 days)

ecc521 commented 2 years ago

Accidentally closed - reopening

ecc521 commented 2 years ago

Fixed in a306a029 Absolutely stunning speed boost