ArchGDAL.jl

ArchGDAL.jl copied to clipboard

performance and type stability fixes for geometries

8

Improves type stability where possible by keeping enums as types in `Val{wkbLineString}` and by forcing child object types from `getgeom`. ~~Also adds a low allocation `getpoint` method.~~ Edit: review when...

rafaqz

Enhance ArchGDAL OGR objects with parametric composite types

36

Here is a draft for review, demonstrating: 1. the use of `ogr_f_stealgeometry` for a gain of an additionnal 7% gain on layer to table conversion when there is only one...

mathieu17g

Hi! Here comes the vector dataset writer. It still is far from perfect and needs tests, but you can already take a look.

maxfreu

Automated changes by [create-pull-request](https://github.com/peter-evans/create-pull-request) GitHub action

github-actions[bot]

write geopackage

8

Either I don't understand the documentation or there might be a bug in writing geopackage vector layers to a file. It creates the error: ERROR: LoadError: GDALError (CE_Failure, code 6):...

pwidas

Why does `createlinearring` create a wkbLineString

4

See: https://github.com/yeesian/ArchGDAL.jl/blob/master/test/test_geometry.jl#L292 I'm trying to fix type stability of geometry creation for performance (some large gains from small changes) but `createlinearring` is a strange wart to special-case. Why does it...

rafaqz

(The last time I went through such an exercise was in https://github.com/yeesian/GDALUtils.jl/issues/2#issue-141488880.) The considerations I can think of are: 1. it will not always be 1:1 in behavior with the...

yeesian

rename layerdefn to getlayerdefn or provide alias

1

This is a really minor issue, but the inconsistency tripped me up; I was looking for the layer definition of a dataset and looked through everything starting with "get", like...

maxfreu

Support user defined progress functions on :aarch64

1

ArchGDAL use on :aarch64 platforms, such as the recent M1 macs, is blocked by `@cfunction($f)` code, as reported in #281. This code is only used to wrap a user defined...

evetion

Add GDT_Int64 and GDT_UInt64 data types

1

A reminder for when we get to building the just released GDAL 3.5. This would need a small update to ArchGDAL to use these: > Add GDT_Int64 and GDT_UInt64 data...

visr