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

Speed Improvement #32

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Speed Improvement #32

wants to merge 8 commits into from

Conversation

nitrag
Copy link

@nitrag nitrag commented May 27, 2020

This is probably not ready for an out-right merge but I want to bring it to you attention that it's possible to 10x the speed of tile downloads.

I'm able to achieve 450 tiles/sec (680Mbps) with this new method while downloading zooms 12, 13, 14 as separate commands. These stats are with an 8 CPU server, SSD (0% IO-Wait). So the limitation is my bandwidth.

Perhaps you want to incorporate this method into the larger download mechanisms.

@drewda
Copy link
Member

drewda commented Jun 4, 2020

@nitrag thanks for this contribution. It can take a while to download a file planet's worth of terrain tiles, so it's a great idea to parallelize that.

We've got a number of other projects going so aren't able to work at the moment, but feel free to continue your PR and ping us when it's ready for a code review.

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

Successfully merging this pull request may close these issues.

2 participants