Dan Baston

Results 265 comments of Dan Baston

> so having to disable them specifically for these calls then ``` try: datasource.StartTransaction() except RuntimeError: pass ```` And if that is too verbose, put it in a function `start_transaction_if_supported(ds)`...

Committed as 0341df45280423f0fa51b16b7249b7847b68834b

> Should the GDAL < 2.3 examples still be kept? I think they should be removed.

At the moment, this function only exists on the R side. It would not be difficult to expose in Python, but I haven't done so yet. Here are some questions...

Hi @theroggy . The current setup is copied from GDAL. I can appreciate that it's not state-of-the-art, but given that this is a pretty small project, my initial thought is...

@macdrevx @nyalldawson My understanding is that this API is preferable to #761 for your use cases, but I'm inclined to let this PR sit until you've had a chance to...

@nyalldawson if you're happy with it, then I think all that remains would be to address the good suggestions from @macdrevx . @pramsey , is there a target release date...

Should be reasonable to finish this up before the next release.

Thanks for the feedback. I've implemented a new version with the custom type, but I don't want to rush it out before Friday. I'm OK leaving this until 3.14.

Since we wouldn't want to change the existing signatures of `GEOSLineMerge`, I think adding `GEOSClusterLineMerge` makes sense, but to do so the `LineMerger` it would need to create a `Clusters`...