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

Perf issue: Imagery lookup is very slow #5226

Closed
bhousel opened this issue Aug 15, 2018 · 3 comments
Closed

Perf issue: Imagery lookup is very slow #5226

bhousel opened this issue Aug 15, 2018 · 3 comments

Comments

@bhousel
Copy link
Member

bhousel commented Aug 15, 2018

iD needs to pretty frequently check what imagery is available in the area where the user is editing. This has recently gotten very slow.

We can speed this up a lot by:

It should not take 1 second to know what polygons cover the current view.. This is just really really bad.

screenshot 2018-08-15 13 16 38

@bhousel bhousel added the performance Optimizing for speed and efficiency label Aug 15, 2018
@prototype99
Copy link

you mean the satellite imagery right? for me it isn't even rendering the imagery beyond a vague blur if i zoom in from the worldwide view. if i refresh it fixes, but otherwise....

@bhousel
Copy link
Member Author

bhousel commented Aug 16, 2018

hey @prototype99 - yes this is about the satellite imagery.
I've fixed the performance issue in 8b3637e but it sounds like your issue is something else.

If you are still having issues with imagery not working, can you do these steps:

  • verify the issue exists in http://preview.ideditor.com/master/
  • open a new issue with details about the map location you are testing (url would be great), your browser, and a screenshot?

Thanks!

@bhousel
Copy link
Member Author

bhousel commented Aug 18, 2018

Just wanted to flag that I broke the "custom" and "none" options when I did this .
They currently don't show up because these options don't have a polygon, so the index doesn't find them.

I'm working on some other changes to custom backgrounds and data and will include a fix for this in an upcoming PR.

@bhousel bhousel reopened this Aug 18, 2018
@bhousel bhousel added the wip Work in progress label Aug 18, 2018
@bhousel bhousel removed performance Optimizing for speed and efficiency wip Work in progress labels Nov 8, 2018
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

2 participants