OSGeo / tilecache

Tilecache
35 stars 18 forks source link

Wrong total number of tiles in seed protocol #14

Open seidlmic opened 9 years ago

seidlmic commented 9 years ago

The line #101 in Client.py

ztiles = int(math.ceil(float(topright[1] - bottomleft[1]) / metaSize[0]) * math.ceil(float(topright[0] - bottomleft[0]) / metaSize[1]))

Does not eqaules

ztiles = len(range(startY, endY, stepY))*len(range(startX, endX, stepX))

Which is actualy correct number of ztiles as shows code on line #118 This produce buggy protocol that ends with

06 (000019, 000008) = (14.5464 50.0942 14.5472 50.0950) [0.1321s : 3.546/s] 180/152
winkey commented 9 years ago

posible dupe of https://github.com/OSGeo/tilecache/issues/13

seidlmic commented 9 years ago

I am not sure but it looks like changes from int() to round() from 2011 can have a lot of side affects like #13. What is exactly reason for that Fix. What was cache missing?

winkey commented 9 years ago

yes the cache was missing, "An error occurred: Current y value... is too far from tile corner y ..."

diegobill commented 8 years ago

:+1: