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

Lazy Load Data Points #7

Closed
aswinmohanme opened this issue Aug 16, 2018 · 13 comments
Closed

Lazy Load Data Points #7

aswinmohanme opened this issue Aug 16, 2018 · 13 comments
Labels
enhancement New feature or request

Comments

@aswinmohanme
Copy link
Owner

Currently we get the entire data dump from the sever. This is expected to grow and considerably slows down the site.

We should load the data of the immediate surroundings first

@aswinmohanme aswinmohanme added the help wanted Extra attention is needed label Aug 16, 2018
@appunnicer13
Copy link
Contributor

Any implementation to filter the whole data will result in use of more CPU and RAM from USERs device which will considerably slow down. Its always a better idea to get the data dynamically by passing latitude longitude and letting server take the load for it. But If we can get district wise data may be this will be possible, but as more people mark it here more the load is going to be

@appunnicer13
Copy link
Contributor

appunnicer13 commented Aug 16, 2018

https://www.npmjs.com/package/geolib, but this is a solution I will work on it

@appunnicer13
Copy link
Contributor

What is the initial window ?

@aswinmohanme
Copy link
Owner Author

I don't understand

@appunnicer13
Copy link
Contributor

Currently I don't load points unless they click on near me

@appunnicer13
Copy link
Contributor

What should be the initial window , if I shouldn't be loading points

@aswinmohanme
Copy link
Owner Author

We should zoom near to the location of the user and load those points. The zoom level should not be as close as the one which shows up when we click on the locate near me

@RevathiKannanGithub
Copy link

RevathiKannanGithub commented Aug 16, 2018

More than district, do we have zones/divisions of districts in kerala? That might narrow down further and reduce the load time.

@appunnicer13
Copy link
Contributor

Yeah that is happening automatically, I am showing points near 10km radius

@appunnicer13
Copy link
Contributor

But is initial window is empty now. I will give a pull request you let me know

@appunnicer13
Copy link
Contributor

Create a branch called lazy loading
I will do pull request there

@appunnicer13
Copy link
Contributor

intial commit on lazy loading #12

@aswinmohanme aswinmohanme added enhancement New feature or request and removed help wanted Extra attention is needed labels Aug 16, 2018
@appunnicer13
Copy link
Contributor

distance selection #17

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

No branches or pull requests

3 participants