[mapserver-dev] FW: [mapserver/mapserver] Default Map Size too small (#5349)

[Jeff McKenna]

Just closing the loop on this: we also discussed this in today's PSC 
meeting and all agreed to proceed.

-jeff



On 2016-11-18 1:15 PM, Daniel Morissette wrote:
> +1 (as I wrote in the ticket already)
>
> Daniel
>
> On 2016-11-18 1:03 AM, Rahkonen Jukka (MML) wrote:
>> Same here, we seem to have MAXSIZE at 5000 for high resolution printing.
>> 4096 would be a better default. If people in 2003 were gathering data
>> from WMS with large requests now they are collecting tiles.
>>
>> -Jukka Rahkonen-
>>
>>
>>
>> *Lähettäjä:*mapserver-dev [mailto:mapserver-dev-bounces at lists.osgeo.org]
>> *Puolesta *Lime, Steve D (MNIT)
>> *Lähetetty:* 18. marraskuuta 2016 1:39
>> *Vastaanottaja:* mapserver-dev at lists.osgeo.org
>> *Aihe:* [mapserver-dev] FW: [mapserver/mapserver] Default Map Size too
>> small (#5349)
>>
>>
>>
>> Any reason not to do this? The reason the setting exists is rather
>> obvious. I checked many of my mapfiles and indeed I bump the value to
>> 4096. Not for printing but for generating metatiles. I think that's a
>> stronger argument... --Steve
>>
>>
>>
>> ------------------------------------------------------------------------
>>
>> *From:*Seth G [notifications at github.com]
>> *Sent:* Wednesday, November 16, 2016 2:47 AM
>> *To:* mapserver/mapserver
>> *Subject:* [mapserver/mapserver] Default Map Size too small (#5349)
>>
>> Currently the maximum image size default is set to 2048 pixels. With
>> increasingly large monitors and higher resolutions I keep hitting this
>> limit when maximising browser windows, and getting an error message.
>>
>> It is confusing to new users, and requires having to always add a
>> |MAXSIZE 4096|line to each new map file.
>>
>> I'd propose increasing the default to 4096. Users concerned about
>> increased stress on their servers are more likely to know how to change
>> this setting, than new users getting an error.
>>
>> I believe it should be a case of simply increasing the following?
>>
>> |#define MS_MAXIMAGESIZE_DEFAULT 2048|
>>
>> https://github.com/mapserver/mapserver/blob/66309eebb7ba0dc70469efeb40f865a8e88fafbd/mapserver.h#L262
>>
>>
>> It was last updated in 2003 - so hopefully most hardware has been
>> updated since then..
>>
>> —