Almeida100 / Dissect-and-dissolve-overlaps

QGis 3.16+ plugin to detect, zoom to, dissect and dissolve overlapping polygons in one polygon layer.
GNU General Public License v3.0
1 stars 0 forks source link

[Suggestion] Add extra info in the metadata. #1

Closed jonnyforestGIS closed 2 years ago

jonnyforestGIS commented 2 years ago

Hi Antonio, can I suggest you add a little note about using a saga algorithm in the metadata file? This can prevent some future open issues because users forgot or don't have SAGA GIS installed or the SAGA algorithm isn't working because it depends on the SAGA provider.

I look at your code, and you call the SAGA algorithm Polygon Self Intersection.

In the future SAGA GIS will work as an external plugin so adding this extra info will save some users doubts.

Cheers

Almeida100 commented 2 years ago

Hello Joao Many thanks for your suggestion. I'll prepare a new version with a warning about SAGA in the metadata file. Regards, Antonio

João Gaspar @.***> escreveu no dia terça, 5/07/2022 à(s) 09:03:

Hi Antonio, can I suggest you add a little note about using a saga algorithm in the metadata file? This can prevent some future open issues because users forgot or don't have SAGA GIS installed or the SAGA algorithm isn't working because it depends on the SAGA provider.

I look at your code, and you call the SAGA algorithm Polygon Self Intersection.

In the future SAGA GIS will work as an external plugin so adding this extra info will save some users doubts.

Cheers

— Reply to this email directly, view it on GitHub https://github.com/Almeida100/Dissect-and-dissolve-overlaps/issues/1, or unsubscribe https://github.com/notifications/unsubscribe-auth/AJAXO5EQ3LUTDKSQG7FC3ITVSPT53ANCNFSM52VJCGBQ . You are receiving this because you are subscribed to this thread.Message ID: @.***>

jonnyforestGIS commented 2 years ago

I will close because you already add the changes with commit https://github.com/Almeida100/Dissect-and-dissolve-overlaps/commit/9c35cd7e866dd16bcea1bbafe1a3804e800a13d5. Cheers.