Re: [Qgis-developer] Exported SLD size or displacementX/Y are not reimported correctly due to mm/pixel ambiguity

2017-01-25 Thread Luigi Pirelli
The fact is that in the qgssymbollay[v2]utils.cpp there are already all encode/decode UOM stuffs since 2012 (by G.Sucameli), but these static routines seems not used in any part of qgis code. I'm still investigating... Luigi Pirelli

Re: [Qgis-developer] Exported SLD size or displacementX/Y are not reimported correctly due to mm/pixel ambiguity

2017-01-25 Thread Jorge Gustavo Rocha
Hi, Maybe we can add support for uom (units of measure), like in [1], in the export and import to/from SLD. I think we could archive better compatibility if we use real world units. The uom attribute is necessary. Numbers only have meaning when they also have units. I would like to try this

Re: [Qgis-developer] Exported SLD size or displacementX/Y are not reimported correctly due to mm/pixel ambiguity

2017-01-25 Thread Andrea Aime
On Wed, Jan 25, 2017 at 4:21 PM, Luigi Pirelli wrote: > BTW this suggest that any export fix would check also import side to > maintain QGIS consistency. > Yep... sorry I was not aware that the geoserver plugin was doing round-tripping of styles, never used it seriously and

Re: [Qgis-developer] Exported SLD size or displacementX/Y are not reimported correctly due to mm/pixel ambiguity

2017-01-25 Thread Luigi Pirelli
Ha ok, now it's clear I'll downgrade the issue to simple bug. BTW this suggest that any export fix would check also import side to maintain QGIS consistency. Luigi Pirelli ** * Boundless QGIS

Re: [Qgis-developer] Exported SLD size or displacementX/Y are not reimported correctly due to mm/pixel ambiguity

2017-01-25 Thread Andrea Aime
Hi Luigi, I don't think it's a regression but an incomplete fix, the qgis code still imports SLD assigning mm as units, while in SLD everything is in pixels unless otherwise stated. This needs to be fixed in all SLD import procedures (large-ish change). Before QGIS was wrongly exporting and

[Qgis-developer] Exported SLD size or displacementX/Y are not reimported correctly due to mm/pixel ambiguity

2017-01-25 Thread Luigi Pirelli
Hi I opened the following tiket http://hub.qgis.org/issues/16112 a regression respect SLD fixies/compatibility with geoserver SLD. Andrea, can you give a look? Luigi Pirelli ** * Boundless QGIS