attilaancsin / PolygonSimplifier

PolygonSimplifier is a QGIS plugin written in python.
GNU General Public License v2.0
5 stars 1 forks source link

No result #6

Closed Gustry closed 8 years ago

Gustry commented 8 years ago

I used the plugin but I can't see any result. "Zoom to layer" doesn't move my map canvas.

Do you have an idea what to check ? simplify

attilaancsin commented 8 years ago

Try to increase the parameter of the generaliztation algorithm significantly. I would try to multiply it with 10 until I see results.

Gustry commented 8 years ago

Ok thanks I will try. So maybe think about that for the help #7 :)

attilaancsin commented 8 years ago

Could you notify me, if you can use it? The plugin is pretty new and I don't receive much positive feedback. :) Thanks.

kgjenkins commented 8 years ago

I had the same issue, but was able to fix it by setting the CRS of the result layer to match the CRS of the input layer. PolygonSimplifier appears to be incorrectly saving all outputs as EPSG:4326.

The problem seems to be in SimplifierAlgorithmRunner.py line 38, where the CRS is getting defined as EPSG:4326 regardless of the input file CRS:

layer = QgsVectorLayer('Polygon?crs=epsg:4326&index=yes', name, 'memory')
kgjenkins commented 8 years ago

See suggested fix here: https://github.com/attilaancsin/PolygonSimplifier/pull/10/files

attilaancsin commented 8 years ago

Thanks for the fix. The plugin was meant to work like that, but it seems I forgot this. I will upload the new version to the site asap.

Gustry commented 8 years ago

I couldn't get the plugin works. Yes I'm using EPSG:2975 on my screenshot, it may be related. Thanks

attilaancsin commented 8 years ago

I have uploaded a new version. Waiting for approval.

Gustry commented 8 years ago

You can approve by yourself. Go back in your plugin setting and check the box ;-)

attilaancsin commented 8 years ago

Cool, thanks. ;) I hope it will work for you guys.

attilaancsin commented 8 years ago

If it's working, I would close this issue.

Gustry commented 8 years ago

It should be fixed in https://github.com/attilaancsin/PolygonSimplifier/pull/10 I will try soon.