Feature request #15905
QgsGeometryAnalyzer, buffer: Allow specification of segments
Status: | Closed | ||
---|---|---|---|
Priority: | Normal | ||
Assignee: | - | ||
Category: | Geometry | ||
Pull Request or Patch supplied: | Yes | Resolution: | wontfix |
Easy fix?: | No | Copied to github as #: | 23822 |
Description
For the QgsGeometryAnalyzer buffer function, it would be useful to be able to specify the number of segments (for approximating arcs).
Number of segments is the second parameter to the QgsGeometry buffer function (that is used by QgsGeometryAnalyzer bufferFeature that is in turn used by QgsGeometryAnalyzer buffer). Current behaviour is to always use 5 for the segment parameter of QgsGeometry buffer.
It seems as if not many changes would need to be made in the code.
QgsGeometryAnalyzer::bufferFeature would need a new parameter ("segments"), and in the line
bufferGeometry = featureGeometry.buffer( currentBufferDistance, 5 );
"5" would have to be changed to "segments".
QgsGeometryAnalyzer::buffer would also need a new parameter ("segments"), and in the two lines
bufferFeature( currentFeature, processedFeatures, &vWriter, dissolve, dissolveGeometry, bufferDistance, bufferDistanceField );
the "segments" parameter would need to be added.
If the new "segments" parameter is optional and added as the last one, existing applications should not be affected.
If there is interest, I can try to create a pull request on github, but I guess there may be more to do then just what I have suggested above...
History
#1 Updated by Håvard Tveite over 7 years ago
Pull request: https://github.com/qgis/QGIS/pull/4302
#2 Updated by Giovanni Manghi over 7 years ago
- Easy fix? set to No
#3 Updated by Håvard Tveite over 7 years ago
If my pull request does not fix this, it would be nice if someone could explain what the problem is.
#4 Updated by Giovanni Manghi over 7 years ago
- Pull Request or Patch supplied changed from No to Yes
- Description updated (diff)
Try raise the problem directly in the PR or in the dev mailing list, thanks!
#5 Updated by Håvard Tveite over 7 years ago
Thanks. I have tried the dev list.
#6 Updated by Alexander Bruy over 7 years ago
This is not necessary/useful with upcomming Processing refactoring.
#7 Updated by Alexander Bruy almost 7 years ago
- Status changed from Open to Closed
QgsGeometryAnalyzer was removed from analysis lib
#8 Updated by Giovanni Manghi almost 7 years ago
- Resolution set to wontfix