Commit 43809ea
authored
⚗️ Benchmark GDAL Libertiff driver vs async-tiff (#3)
* ⚗️ Benchmark GDAL Libertiff driver vs async-tiff
Obtain Sentinel-2 TCI image and convert it to a tiled GeoTIFF, apply LZW compression with Horizontal differencing predictor. Then run benchmarks comparing GDAL's Libertiff driver against async-tiff, using 16 threads.
* 💚 Download and process sample GeoTIFF
Need to remember to put the sample GeoTIFF on CI. Processing it with a newer version of GDAL from official docker image.
* 👷 Run on ghcr.io/osgeo/gdal:ubuntu-small-3.12.0
Try running directly on official docker image for GDAL 3.12.0. Should have Libertiff driver included.
* 🔥 One-liner to download and process TCI.tif file
The ghcr.io/osgeo/gdal:ubuntu-small-3.12.0 docker image doesn't have wget, so might as well use a one-liner to get and process the file.
* 👷 Compile on ghcr.io/osgeo/gdal:ubuntu-small-3.11.5
Fixing various linker issues, also pin georust/gdal to a version that supports GDAL 3.11, xref georust/gdal#664.
* 🛂 Run docker container with --privileged
Try to fix `setarch: failed to set personality to x86_64: Operation not permitted` on CodSpeedHQ/action. Adapted from suggestion at https://stackoverflow.com/questions/57830375/github-actions-workflow-error-permission-denied/70207187#70207187
* ⬆️ Bump CodSpeedHQ/action from 4.3.3 to 4.3.4
Bumps [CodSpeedHQ/action](https://github.com/codspeedhq/action) from 4.3.3 to 4.3.4.
- [Release notes](https://github.com/codspeedhq/action/releases)
- [Changelog](https://github.com/CodSpeedHQ/action/blob/main/CHANGELOG.md)
- [Commits](CodSpeedHQ/action@v4.3.3...v4.3.4)
Also install git following instructions at https://codspeed.io/docs/integrations/ci/github-actions#using-container-images
* 📌 Install Criterion.rs compatibility layer for CodSpeed
Xref https://codspeed.io/docs/benchmarks/rust/criterion#installation
* 💚 Only do actual codspeed run on Rust stable
Compile on Rust MSRV (1.85), but run benchmarks on stable (currently 1.91.0).
* 🧵 Parametrize functions to run on n_threads=X
Allow setting number of threads to use for both the read_geotiff_gdal and read_geotiff_async_tiff functions. Set to 4 cores to align with GitHub Actions linux runners.
For some reason, LiberTIFF doesn't seem to respect NUM_THREADS properly? Either it uses 1 core or all cores...
* ⬆️ Bump MSRV from 1.85.0 to 1.89.0
To get temporary lifetime extension, xref rust-lang/rust#140593
* 👷 Consolidate apt install steps1 parent a12d7d9 commit 43809ea
File tree
5 files changed
+323
-42
lines changed- .github/workflows
- benches
5 files changed
+323
-42
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
19 | 19 | | |
20 | 20 | | |
21 | 21 | | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
22 | 25 | | |
23 | 26 | | |
24 | 27 | | |
25 | | - | |
| 28 | + | |
26 | 29 | | |
27 | 30 | | |
| 31 | + | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
28 | 37 | | |
29 | 38 | | |
30 | 39 | | |
31 | 40 | | |
32 | 41 | | |
33 | | - | |
| 42 | + | |
34 | 43 | | |
35 | | - | |
36 | | - | |
| 44 | + | |
| 45 | + | |
37 | 46 | | |
38 | 47 | | |
39 | 48 | | |
| |||
47 | 56 | | |
48 | 57 | | |
49 | 58 | | |
50 | | - | |
| 59 | + | |
| 60 | + | |
51 | 61 | | |
52 | 62 | | |
53 | 63 | | |
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
6 | 6 | | |
7 | 7 | | |
8 | 8 | | |
9 | | - | |
| 9 | + | |
10 | 10 | | |
11 | 11 | | |
12 | 12 | | |
| |||
18 | 18 | | |
19 | 19 | | |
20 | 20 | | |
21 | | - | |
| 21 | + | |
22 | 22 | | |
23 | | - | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
24 | 27 | | |
25 | 28 | | |
26 | 29 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
53 | 53 | | |
54 | 54 | | |
55 | 55 | | |
| 56 | + | |
| 57 | + | |
| 58 | + | |
| 59 | + | |
| 60 | + | |
| 61 | + | |
| 62 | + | |
| 63 | + | |
| 64 | + | |
| 65 | + | |
| 66 | + | |
| 67 | + | |
| 68 | + | |
| 69 | + | |
| 70 | + | |
| 71 | + | |
| 72 | + | |
| 73 | + | |
| 74 | + | |
| 75 | + | |
| 76 | + | |
| 77 | + | |
56 | 78 | | |
57 | 79 | | |
58 | 80 | | |
| |||
0 commit comments