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.

Sign up for GitHub

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

Jump to bottom

WMTS delayed tile loading #20

Closed
Ylannl opened this issue Oct 20, 2020 · 3 comments
Closed

WMTS delayed tile loading #20

Ylannl opened this issue Oct 20, 2020 · 3 comments
Assignees
@Ylannl
Labels
bug Something isn't working

Comments

@Ylannl
Copy link
Member

Ylannl commented Oct 20, 2020

In some cases, eg when you pan around and zoom a lot, the WMTSTilesRenderer seems to be doing more downloads than necessary (ie because the we already moved on to a different view/zoom level). Then it can happen that there is a delay of minutes before the tiles in the current view show up, since it first has to complete all the unnecessary downloads.
@Ylannl Ylannl added the bug Something isn't working label Oct 20, 2020
@liberostelios
Copy link
Contributor

51fc455 partially tackles this issue. But a more permanent solution might be better.

@Ylannl
Copy link
Member Author

Ylannl commented Nov 1, 2020

e751d0f fixes this by tile aborting pending downloads for tiles that are no long in the camera frustum.

@Ylannl Ylannl closed this as completed Nov 1, 2020
@Ylannl Ylannl reopened this Nov 3, 2020
@Ylannl
Copy link
Member Author

Ylannl commented Nov 3, 2020

This now lives in the branch: https://github.com/tudelft3d/3dbag-viewer/tree/feature/wmts-abort-download .

It works, but the textures don't show in Safari and Firefox.

@Ylannl Ylannl self-assigned this Nov 16, 2020
Ylannl added a commit that referenced this issue Nov 30, 2020
@Ylannl
fix #20 for other browsers than Chrome
f373049
Ylannl added a commit that referenced this issue Nov 30, 2020
@Ylannl
Merge pull request #38 from tudelft3d/feature/wmts-abort-download
3a9a6c0 
fix #20 for other browsers than Chrome
@Ylannl Ylannl closed this as completed Nov 30, 2020
@Ylannl Ylannl mentioned this issue Dec 1, 2020
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Ylannl Ylannl

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Ylannl @liberostelios