-
Notifications
You must be signed in to change notification settings - Fork 108
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Investigate higher HEAD/GET Request with GDAL >3.7 #697
Comments
Rasterio 1.3.9 / GDAL 3.6.4
Rasterio 1.3.10 / GDAL 3.7.3
Rasterio 1.3.10 / GDAL 3.8.4
CURL Logs
|
Some weird stuff from the logs:
while we have
|
I don't confirm that using plain GDAL command line (3.8.5 or master):
|
For some reason doing
rio-tiler/tests/benchmarks/requests.py Lines 14 to 36 in 473de24
|
well gdalwarp gives the expected result
😬 |
you may want to break into GTiffDataset::LookForProjectionFromXML() . Normally papszSiblingFiles = GetSiblingFiles() should be such that papszSiblingFiles != nullptr and papszSiblingFiles[0] == nullptr if GDAL_DISABLE_READDIR_ON_OPEN=EMPTY_DIR |
just to summarize: with |
@rouault is |
it has been introduced in 3.7 |
I'm going back to this and to me there seems to be a bug somewhere in GDAL Rasterio (1.4.1, GDAL=3.9.2) + WarpedVRTimport logging
import rasterio
from rasterio.vrt import WarpedVRT
logging.basicConfig(level=logging.DEBUG)
src_path = "https://sentinel-cogs.s3.us-west-2.amazonaws.com/sentinel-s2-l2a-cogs/15/T/VK/2023/10/S2B_15TVK_20231008_0_L2A/TCI.tif"
with rasterio.Env(GDAL_DISABLE_READDIR_ON_OPEN="EMPTY_DIR", GDAL_INGESTED_BYTES_AT_OPEN=32768, GDAL_HTTP_MERGE_CONSECUTIVE_RANGES="YES", CPL_DEBUG="ON", CPL_CURL_VERBOSE="YES"):
with rasterio.open(src_path) as src:
with WarpedVRT(src, dst_crs="epsg:3857") as vrt:
vrt.read(indexes=1, out_shape=(1, vrt.height//5, vrt.width//5)) logs
@rouault I don't see any mentione of what I don't understand is why gdalwarp doesn't result in the same behaviour GDAL 3.8.5 (gdalwarp)
We can also see that rasterio doesn't I've tested with both gdal 3.8 and 3.9 |
should be resolved per OSGeo/gdal#10957 |
Merci @rouault 🙏 |
with https://github.com/cogeotiff/rio-tiler/blob/main/tests/benchmarks/requests.py we test some tile request fetching and use
tilebench
to monitor the number or GET/HEAD requests made.with the latest commits, this test is failing
rio-tiler/tests/benchmarks/requests.py
Lines 35 to 67 in 4d95c28
👇 this used to return HEAD: 1 and GET 3 🤷
The text was updated successfully, but these errors were encountered: