-
Notifications
You must be signed in to change notification settings - Fork 2
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
Further efficiency increases #152
Further efficiency increases #152
Comments
Bounding box loading ideaWhat if the initial map view was zoomed in and we load the shapes in the viewed bounding box first (first one year, then across all years), then allow zooming out after it has loaded |
Could we use GeoArrow in Django?My Q:
Copilot answer:
For using GeoJSON, basically you would go via a view as we are |
Branches
adjust-loading
just does some of the below, see in progress belowTODO 6th June
adjust-loading
map_view_one_year
againcan we update this such that if thedisplayYear != enteredYear
whilst theworld_map_all
fetch is in progress, we exit that fetch to run themap_view_one_year
again before running theworld_map_all
fetch againadjust-loading
sort of does this, but it doesn't quite work as expected, keep fiddling and see if it could work, bit don't spend long and otherwiseb3c4732a2bd69776fe7ce288d8fef836c5fbe92d
into Processing speedups - June 6th 2024 #159TODO
assign_variables_to_shapes
andassign_categorical_variables_to_shapes
are called after the shapes are added to the map. It would require a separate django view and django object which could be a dictionary with the shape id as the key and the var info as the valuemap_view_all_with_vars
that gets run last and overwrites the shapeData instead of adding to it as the others do. The other views should not include variables.Set the initial polityBorderWeight in Python, not JS, to avoid looping though dict before page loadrunning the plotPolities() function is instant, nothing to improve hereIs it faster on a GPU machine?I don't think this is relevant since the live site won't have itassign_variables_to_shapes
Update: this has already been sped up via refactoring so further speed increases may not be neededAlternatively load these as a separate thing to the shapes and suppress at firstthat's pointless because the issue is running the function, it shouldn't slow down the Python=>JS transition - yes but you could have a separate view that runs this and updates the shapes?The text was updated successfully, but these errors were encountered: