Conversation
Running tests on Ubuntu-24.04 and Rust 1.85.0 and current stable (1.91.0).
So that `cargo build` works
See if this fixes `wrapper.h:5:10: fatal error: 'cpl_atomic_ops.h' file not found`.
weiji14
changed the title
Can't be too empty. Also renamed ci.yml to bench.yml and explicitly set mode: instrumentation on CodSpeedHQ/action.
CodSpeed Performance Report
|
weiji14
added a commit
to developmentseed/async-tiff
that referenced
this pull request
Nov 22, 2025
2 tasks
weiji14
added a commit
to developmentseed/async-tiff
that referenced
this pull request
Jan 26, 2026
* Benchmark code for reading sample Sentinel-2 TCI file Adapted from weiji14/foss4g2025#3 * ci: Setup benchmark CI running cargo-codspeed Xref weiji14/foss4g2025#2 * Set id-token: write permissions for OIDC auth Xref https://codspeed.io/changelog/2025-11-19-simpler-authentication-with-oidc. Also change from the deprecated 'instrumentation' mode to 'simulation' * Change dbg! of x/y IFD counts to assert_eq! * Refactor to used ReadaheadMetadataCache instead of PrefetchBuffer * Split read_tiff into open_tiff and decode_tiff functions While making decode_tiff generic over readers that implement AsyncFileReader trait. * Add note on why returned bytes is 363528192 It is because we have 1849 tiles of 196608 bytes (3*256*256) each, there is extra padding added to the tiled TIFF. * Carve out async read_tiles function That encompasses the async block where all tiles are fetched and read. * Refactor to separate async network fetch from multi-threaded decoding Make open_tiff the async part that does async fetching of compressed tiles, and decode_tiff being the multi-threaded part that does tile decompression to actual values. * Try decoding to Array instead of u8 After #165, but doesn't quite work for edge tiles due to a bug. * Update to check tile_array and tile_bytes length properly --------- Co-authored-by: Kyle Barron <kyle@developmentseed.org>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Running benchmarks on Ubuntu-24.04 and Rust 1.85.0 and current stable (1.91.0). Benchmarks will be logged to codspeed.
References: