Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Performance #24

Open
trietmnj opened this issue Aug 9, 2021 · 8 comments
Open

Performance #24

trietmnj opened this issue Aug 9, 2021 · 8 comments
Assignees

Comments

@trietmnj
Copy link
Contributor

trietmnj commented Aug 9, 2021

If the map is zoomed in, and then zoomed out, the part of the map outside of the zoomed in section will not load. This is likely tied to the VTL layer, I don't think I see the problem without it. There's no error thrown in console. Could be due to my laptop not having the sufficient computing capacity to both host the development server and compute the graph on the front end.

image

@HenryGeorgist
Copy link
Member

i have noticed this too - it is probably the vtl. One strategy is to narrow the zoom level to not display at the national level... kinda a bummer.

@HenryGeorgist
Copy link
Member

or we could have hexbins show at the high level and switch to the point vtl on the way down past some zoom level

@trietmnj
Copy link
Contributor Author

Is rgoss branch the same as current production for the survey tool? I see the same problem with the VTL on the local development server. Same with the hexbins, I think I see a bigger performance drop with the hexbins.

image

image

@trietmnj
Copy link
Contributor Author

Also, where is the survey tool hosted? I want to see what it looks like in production.

@HenryGeorgist
Copy link
Member

the rgoss branch should be the branch in production.

https://apps.rsgis.dev/nsi-survey/

@HenryGeorgist
Copy link
Member

there are three hexbin levels, it looks like you are displaying one of the higher resolution layers, if you display a different resolution at different zoom levels it might be best. i think @rgoss might have some ideas for how to play around with that.

@HenryGeorgist
Copy link
Member

for v0.0.1 we will drop support for VTL in CM3

@trietmnj
Copy link
Contributor Author

trietmnj commented Sep 8, 2022

The issue is caused by a data race condition from the VTL using the same load queue as the base map. Without modifying dependency packages, a solution is to force trigger load on mouse movement (see link below). Issue should be resolve when re-adding VLT.

https://github.com/HydrologicEngineeringCenter/nsi-survey-tool/pull/57/files#diff-e4da92e7fbe58d1d2df7b533e2f5bebd163df2d000b95528fc4f7acf14e99f93R135

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants