Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#60 closed enhancement (fixed)

Feedback: increase MAXSIZE for OpenLayers3 demo

Reported by: Jeff McKenna Owned by: Jeff McKenna
Priority: major Milestone: 4.0 release
Component: MS4W - Packages Version: 3.1.4
Keywords: Cc: TC Haddad
Blocked By: Blocking:

Description

From Jukka:

---
Some observations about the demo that starts from http://localhost/openlayers/examples/mapserver-wms.html?mode=advanced

  • It would be good to increase MAXSIZE in mapfile from the default 2048. In full screen OL3 is sending 2810x600 sized request with my computer. That feels, by the way, odd because my computer has maximum resolution 1920x1080.

Change History (9)

comment:1 by Jeff McKenna, 5 years ago

From BobB:

---
The larger size is probably related to some overflow filling by the map window to aid in initial panning of the map.

We’ve set our default size to 4096. This gets by most UHD (4k) displays, but we’re starting to occasionally even hit that limit now too with the larger resolution displays, combined with some overflow settings.

comment:2 by TC Haddad, 5 years ago

Commenting so I can follow this ticket. Not sure how else to follow!

comment:3 by Jeff McKenna, 5 years ago

Cc: TC Haddad added

comment:4 by Jeff McKenna, 5 years ago

To follow a ticket, you have to click the little "modify" link (top-right of this page) and then add your username into the "CC" field - I did this for you here. Otherwise, Trac is weird that it doesn't notify you if you just comment. Tricky!

comment:5 by Jeff McKenna, 5 years ago

Resolution: fixed
Status: newclosed

The latest OpenLayers package now sets MAXSIZE 4096 as recommended here. Please test and provide feedback: http://www.ms4w.com/release/apps/openlayers-3.16.0-ms4w.zip

I personally don't see a difference, but, I guess someone else might. Let me know if anything needs changing.

comment:6 by TC Haddad, 5 years ago

I guess we need someone with a very large monitor to test and confirm that the previous 2048 barrier is successfully been surpassed.

In the back of my mind I was wondering if there was also a maxsize set somewhere during Mapserver compilation, but maybe that was only in versions before 7.x?

comment:7 by Jeff McKenna, 5 years ago

Apparently maxsize was added in version 4.2 But we need more testing indeed, as it also involves an OpenLayers-generated WMS GetMap request. Lots of variables.

I have several larger monitors and I don't see the issue. Maybe Jukka is referring to how the map appears a little "squished" in this OpenLayers demo? One could argue that.

comment:8 by TC Haddad, 5 years ago

I think for non-tiled UI requests, for example a print request, the server will just refuse to send the image if it is above Maxsize and will send an error instead - happened in a GeoMoose Issue recently as well (which is why I was following).

See: https://github.com/geomoose/geomoose/issues/148

comment:9 by Jeff McKenna, 5 years ago

Ah ok, interesting. In this case there is no error, the map is generated fine - I think it is a little squished, but I think it is ok for a demo.

Note: See TracTickets for help on using tickets.