Avatar for the weiji14 user
weiji14
cog3pio
BlogDocs

:sparkles: Implement cog3pio xarray BackendEntrypoint

#14Merged
Comparing
xarray_backend
(
a6b20de
) with
main
(
33285da
)
CodSpeed Performance Gauge
0%
Improvements
0
Regressions
0
Untouched
3
New
2
Dropped
0
Ignored
0

Benchmarks

New

test_xarray_backend_open_dataarray[cog3pio]
python/tests/test_xarray_backend.py::test_xarray_backend_open_dataarray[cog3pio]
CodSpeed Performance Gauge
N/A
N/A
253.9 ms
test_xarray_backend_open_dataarray[rasterio]
python/tests/test_xarray_backend.py::test_xarray_backend_open_dataarray[rasterio]
CodSpeed Performance Gauge
N/A
N/A
88 ms

Passed

test_read_geotiff_local
python/tests/test_io_geotiff.py::test_read_geotiff_local
CodSpeed Performance Gauge
0%
243.7 µs
243.5 µs
test_read_geotiff_multi_band
python/tests/test_io_geotiff.py::test_read_geotiff_multi_band
CodSpeed Performance Gauge
0%
23.9 ms
24 ms
test_read_geotiff_remote
python/tests/test_io_geotiff.py::test_read_geotiff_remote
CodSpeed Performance Gauge
-1%
8.3 ms
8.4 ms

Commits

Click on a commit to change the comparison range
Base
main
33285da
0%
:heavy_plus_sign: Add xarray N-D labeled arrays and datasets in Python!
9fee0ed
9 months ago
by weiji14
0%
:globe_with_meridians: Decode x and y coordinates from affine transform Work out the list of x and y coordinates for the raster grid from the Affine transformation matrix. Requires changes being developed at https://github.com/georust/geo/pull/1159.
899d77d
9 months ago
by weiji14
0%
:construction_worker: Disable CI builds on 32-bit Windows Getting an error with compiling pandas 2.2.1 on Windows x86 (32-bit), and pandas has moved away from providing 32-bit Windows wheels, see https://github.com/pandas-dev/pandas/issues/54979, so might as well not build cog3pio wheels for 32-bit Windows since we can't test them properly.
4946458
9 months ago
by weiji14
0%
:bug: Add half pixel offset to get centrepoint of top left pixel The affine transform from the GeoTIFF represents the top-left corner of the top-left pixel (gridline registration), but we need to convert that to the center of the top-left pixel (pixel registration) which is what xarray typically assumes. Added some more comments on how the xy_coords is calculated, and updated unit tests with new x/y min/max bounds. Commented out the mean value assertion for now as NaN handling is not implemented yet.
e0aaac3
9 months ago
by weiji14
0%
:twisted_rightwards_arrows: Merge branch 'main' into xarray_backend
4c08b64
8 months ago
by weiji14
0%
:pushpin: Pin to georust/geo at rev 481196b Getter methods on AffineTransform implemented in https://github.com/georust/geo/pull/1159 has been merged into the main branch of georust/geo, so no longer need to point to my personal fork. Also fixed a mismatched type when accessing the x_res and y_res attributes.
7a40655
7 months ago
by weiji14
0%
:busts_in_silhouette: Push out multi-dtype feature in roadmap to Q2 The multiple dtype feature is a little trickier than expected, so will push this out as a medium term goal for Q2 2024. Likely will depend on how the georust/geotiff crate's progress is like.
a6b20de
7 months ago
by weiji14
Home Terms PrivacyDocs