Opened 11 years ago

Closed 6 years ago

#1178 closed enhancement (wontfix) does not handle zoom correctly

Reported by: openstreetmap@… Owned by: spaetz
Priority: minor Milestone:
Component: tilesathome Version:
Keywords: Cc:


When I do a

the result is

Render 'tile,maplint,captionless' (12,136,85) at priority 2

Whereas the change in priority is ok, the change in zoomlevel is not. The coordinates of zoom level 8 do not match the ones of 12. So either the request is ignored totally or it is correctly handled as level 8.

P.S. The above is the result of looking behind the scenes of, where these request are created.

Attachments (1)

a.diff (645 bytes) - added by Dirk Stoecker 11 years ago.
Proper diff :-)

Download all attachments as: .zip

Change History (14)

comment:1 Changed 11 years ago by openstreetmap@…

I also found the reason. The parameter is "min_z" and not z. Thus Javascript of IFW must be fixed.

comment:2 Changed 11 years ago by openstreetmap@…

And another note: The rendering should is done on level 6 and 12 only, no longer 8. So also the text telling me to request on level 8 must be fixed. And also on level 8 no longer the selection box must be drawn.

comment:3 Changed 11 years ago by Tom Hughes

Resolution: invalid
Status: newclosed

Information freeway is not operated or maintained by the OSM administrators - you need to contact the IFW owner (Etienne Cherdlu) to resolve any problems you have with it.

comment:4 Changed 11 years ago by Dirk Stoecker

Resolution: invalid
Status: closedreopened

Beside the above the topic is still correct. does not handle zoom correctly --> When a request in level 6 is made, it is handled as a level 12 request.

I tried to have a look at the source code, but did not find the place, where the 6 is changed into a 12.

A little patch (don't know if correct, but should show be clear what's meant) during my search. This fixes the max_z value, but not the general issue.

Index: requests/ =================================================================== --- requests/ (Revision 10480) +++ requests/ (Arbeitskopie) @@ -9,9 +9,9 @@

x = models.PositiveSmallIntegerField?() y = models.PositiveSmallIntegerField?()

  • # render tiles from zoom levl min_z through max_z

+ # render tiles from zoom leel min_z through max_z

min_z = models.PositiveSmallIntegerField?(default=12)

+ max_z = models.PositiveSmallIntegerField?(default=min_z+5)

# list of layers we should render layers = models.ManyToManyField?(Layer,null=True)

comment:5 Changed 11 years ago by Dirk Stoecker

Component: adminslippy_map

Changed 11 years ago by Dirk Stoecker

Attachment: a.diff added

Proper diff :-)

comment:6 Changed 11 years ago by Tom Hughes

Resolution: invalid
Status: reopenedclosed

As I explained before, this is an IFW issue, not an OSM issue, so please stop reopening this ticket and go talk to somebody that can actually help you, which is not anybody who will be reading this.

comment:7 Changed 11 years ago by Tom Hughes

Component: slippy_maposmarender
Resolution: invalid
Status: closedreopened

Gah. I see it is actually an osmarender issue, not an IFW issue...

comment:8 Changed 11 years ago by osm@…

I think this should go in the (new) tilesathome component...

comment:9 Changed 11 years ago by osm@…

Component: osmarendertilesathome

comment:10 Changed 10 years ago by Tom Hughes

Owner: changed from Tom Hughes to osm@…
Status: reopenednew

comment:11 Changed 10 years ago by spaetz

Owner: changed from osm@… to spaetz
Priority: majorminor
Status: newassigned
Type: defectenhancement

{{{When I do a the result is Render 'tile,maplint,captionless' (12,136,85) at priority 2 }}}

Yes and that is correct, there *IS* no zoom level 8 rendering. If the server accepted that what should it do with it? A tile set file is z6-11 or z12-17 and nothing inbetween, indeed if a client uploaded a z8-11 tileset file the server would (correctly) discard it as invalid. Therefore the patch to allow z=8 will not help anyway.

As for the z6 rendering. That is correct too, because t@h clients are AFAIK not capable of rendering low zoom requests by default, it currently needs much trickery to get that going. Also given the enormously bigger data/render efforts needed for lowzoom rendering, I don't think we should let people request those renderings from the user side. Make t@h client handle low zoom requests, and I'll change the server to automatically insert them to required areas and issue them to client.

comment:12 Changed 10 years ago by spaetz

To make the server future proof (when clients understand low zoom render requests), I applied your patch to the server.

unsupported operand type(s) for +: 'PositiveSmallIntegerField' and 'int'
Exception Location: 	/var/www/tilesAtHome/tah/requests/ in Request, line 14

so your patch doesn't work.

comment:13 Changed 6 years ago by iandees

Resolution: wontfix
Status: assignedclosed

Cleaning aging tickets.

Note: See TracTickets for help on using tickets.