[Geoserver-devel] Switching GetFeatureInfo buffer tolerance to 5 pixles

Hi,
on the users list we have a thread going on about GeoServer supposedly too small 3 pixel min search area (when SLD does not tell us otherwise) for GetFeatureInfo.

Jonathan proposes to go 5, Jukka agrees.
I’m not taking sides, but it is certainly true that the DPI of the screen increased since 3 was put as the default (e.g., my notebook has 1900x1200 resolution on a 17" screen, a few years ago you would have had to be happy with 1152x864) so the real world length of those 3 pixels shortened over time.

The change would be a 2 lines of code one, so it should be pretty easy.

Opinions?

Cheers
Andrea

==
Our support, Your Success! Visit http://opensdi.geo-solutions.it for more information.

Ing. Andrea Aime

@geowolf
Technical Lead

GeoSolutions S.A.S.
Via Poggio alle Viti 1187
55054 Massarosa (LU)
Italy
phone: +39 0584 962313
fax: +39 0584 1660272
mob: +39 339 8844549

http://www.geo-solutions.it
http://twitter.com/geosolutions_it


Sounds fairly reasonable to me. +1 on the change.

···

On Thu, Aug 8, 2013 at 7:46 AM, Andrea Aime <andrea.aime@anonymised.com> wrote:

Hi,
on the users list we have a thread going on about GeoServer supposedly too small 3 pixel min search area (when SLD does not tell us otherwise) for GetFeatureInfo.

Jonathan proposes to go 5, Jukka agrees.
I’m not taking sides, but it is certainly true that the DPI of the screen increased since 3 was put as the default (e.g., my notebook has 1900x1200 resolution on a 17" screen, a few years ago you would have had to be happy with 1152x864) so the real world length of those 3 pixels shortened over time.

The change would be a 2 lines of code one, so it should be pretty easy.

Opinions?

Cheers
Andrea

==
Our support, Your Success! Visit http://opensdi.geo-solutions.it for more information.

Ing. Andrea Aime

@geowolf
Technical Lead

GeoSolutions S.A.S.
Via Poggio alle Viti 1187
55054 Massarosa (LU)
Italy
phone: +39 0584 962313
fax: +39 0584 1660272
mob: +39 339 8844549

http://www.geo-solutions.it
http://twitter.com/geosolutions_it



Get 100% visibility into Java/.NET code with AppDynamics Lite!
It’s a free troubleshooting tool designed for production.
Get down to code-level detail for bottlenecks, with <2% overhead.
Download for free and get started troubleshooting in minutes.
http://pubads.g.doubleclick.net/gampad/clk?id=48897031&iu=/4140/ostg.clktrk


Geoserver-devel mailing list
Geoserver-devel@anonymised.comsts.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel


Justin Deoliveira
OpenGeo - http://opengeo.org
Enterprise support for open source geospatial.

Makes sense, +1

···

On Thu, Aug 8, 2013 at 3:59 PM, Justin Deoliveira <jdeolive@anonymised.com> wrote:

Sounds fairly reasonable to me. +1 on the change.


Get 100% visibility into Java/.NET code with AppDynamics Lite!
It’s a free troubleshooting tool designed for production.
Get down to code-level detail for bottlenecks, with <2% overhead.
Download for free and get started troubleshooting in minutes.
http://pubads.g.doubleclick.net/gampad/clk?id=48897031&iu=/4140/ostg.clktrk


Geoserver-devel mailing list
Geoserver-devel@anonymised.comsts.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel

DI Christian Mueller MSc (GIS), MSc (IT-Security)
OSS Open Source Solutions GmbH

On Thu, Aug 8, 2013 at 7:46 AM, Andrea Aime <andrea.aime@anonymised.com> wrote:

Hi,
on the users list we have a thread going on about GeoServer supposedly too small 3 pixel min search area (when SLD does not tell us otherwise) for GetFeatureInfo.

Jonathan proposes to go 5, Jukka agrees.
I’m not taking sides, but it is certainly true that the DPI of the screen increased since 3 was put as the default (e.g., my notebook has 1900x1200 resolution on a 17" screen, a few years ago you would have had to be happy with 1152x864) so the real world length of those 3 pixels shortened over time.

The change would be a 2 lines of code one, so it should be pretty easy.

Opinions?

Cheers
Andrea

==
Our support, Your Success! Visit http://opensdi.geo-solutions.it for more information.

Ing. Andrea Aime

@geowolf
Technical Lead

GeoSolutions S.A.S.
Via Poggio alle Viti 1187
55054 Massarosa (LU)
Italy
phone: +39 0584 962313
fax: +39 0584 1660272
mob: +39 339 8844549

http://www.geo-solutions.it
http://twitter.com/geosolutions_it



Get 100% visibility into Java/.NET code with AppDynamics Lite!
It’s a free troubleshooting tool designed for production.
Get down to code-level detail for bottlenecks, with <2% overhead.
Download for free and get started troubleshooting in minutes.
http://pubads.g.doubleclick.net/gampad/clk?id=48897031&iu=/4140/ostg.clktrk


Geoserver-devel mailing list
Geoserver-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel


Justin Deoliveira
OpenGeo - http://opengeo.org
Enterprise support for open source geospatial.

+1 here too. pixel size has got smaller.

Notice: This email and any attachments are confidential.
If received in error please destroy and immediately notify us.
Do not copy or disclose the contents.

Note we may also have the optional DPI parameter on the GetMap request to refer to, any interest in using this (if available)?

···

On Thu, Aug 8, 2013 at 11:46 PM, Andrea Aime <andrea.aime@anonymised.com> wrote:

Hi,
on the users list we have a thread going on about GeoServer supposedly too small 3 pixel min search area (when SLD does not tell us otherwise) for GetFeatureInfo.

Jonathan proposes to go 5, Jukka agrees.
I’m not taking sides, but it is certainly true that the DPI of the screen increased since 3 was put as the default (e.g., my notebook has 1900x1200 resolution on a 17" screen, a few years ago you would have had to be happy with 1152x864) so the real world length of those 3 pixels shortened over time.

The change would be a 2 lines of code one, so it should be pretty easy.

Opinions?

Cheers
Andrea

==
Our support, Your Success! Visit http://opensdi.geo-solutions.it for more information.

Ing. Andrea Aime

@geowolf
Technical Lead

GeoSolutions S.A.S.
Via Poggio alle Viti 1187
55054 Massarosa (LU)
Italy
phone: +39 0584 962313
fax: +39 0584 1660272
mob: +39 339 8844549

http://www.geo-solutions.it
http://twitter.com/geosolutions_it



Get 100% visibility into Java/.NET code with AppDynamics Lite!
It’s a free troubleshooting tool designed for production.
Get down to code-level detail for bottlenecks, with <2% overhead.
Download for free and get started troubleshooting in minutes.
http://pubads.g.doubleclick.net/gampad/clk?id=48897031&iu=/4140/ostg.clktrk


Geoserver-devel mailing list
Geoserver-devel@anonymised.comsts.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel

On Fri, Aug 9, 2013 at 2:43 PM, Jody Garnett <jody.garnett@anonymised.com> wrote:

Note we may also have the optional DPI parameter on the GetMap request to
refer to, any interest in using this (if available)?

There is also &buffer=xyz that allows to customize the search area, but we
are not talking about vendor parameters here,
we are talking about the defaults (which is often all you get from normal
desktop clients anyways)

Cheers
Andrea

--

Our support, Your Success! Visit http://opensdi.geo-solutions.it for more
information.

Ing. Andrea Aime
@geowolf
Technical Lead

GeoSolutions S.A.S.
Via Poggio alle Viti 1187
55054 Massarosa (LU)
Italy
phone: +39 0584 962313
fax: +39 0584 1660272
mob: +39 339 8844549

http://www.geo-solutions.it
http://twitter.com/geosolutions_it

-------------------------------------------------------