Comments (7)
Original issue was fixed in GDAL couple of hours ago: OSGeo/gdal@f7c27a8. It is possible to use the similar approach for calculating max factor value: if we already have a 1x1 overview then don't create a new one.
from telluric.
It is the overview code,
disable the overviews and it works:
rr.save('demo.tif', overviews=False)
the problem is the factor calculation
https://github.com/satellogic/telluric/blob/master/telluric/georaster.py#L565
if we create overview only upto factor 32, then the code will pass.
How can we know if the factors make sense?
from telluric.
We can introduce maximum width or height of the smallest overview level like it has been done in recent GDAL (minsize
option of gdaladdo).
from telluric.
Thanks @drnextgis! I guess a fix will be released as part of GDAL 2.3.1, or 2.4.0 at most? In any case we should support older versions as well, so perhaps we should do as you say:
We can introduce maximum width or height of the smallest overview level like it has been done in recent GDAL (
minsize
option of gdaladdo).
from telluric.
Also, thanks for reporting this upstream: rasterio/rasterio#1333
from telluric.
@Juanlu001 what do you think the best place to fix this issue: here or directly in Rasterio?
from telluric.
@drnextgis The rasterio developers have labeled the issue as post-1.0
, so I think it would be positive if we have a workaround in place ourselves. But I will leave this issue as good first issue
, for potential contributors.
from telluric.
Related Issues (20)
- supply telluric-core package with less dependencies HOT 3
- test_geographic_crop_with_resize failing locally HOT 2
- memory leak in raster_opener? HOT 2
- Switch to new pyproj.crs.CRS? HOT 4
- Shapely 1.7 breaks `test_polygonize_line_square_cap_style` HOT 1
- Deprecation warnings
- Migrate off Travis CI
- Cannot import GeoVector if vis dependencies not present
- Cannot import georaster.py classes if vis depedencies ar not present HOT 1
- Expose warp_mem_limit and num_threads parameters of rasterio reproject method
- CI broken on master HOT 5
- Modifying image attribute's dtype in a MutableGeoRaster doesn't update object dtype attribute HOT 3
- Rework Matrix <-> Slack bridge
- Minimum handling of rasters without CRS and avoid meaningless RecursionError
- resize and similar methods should conserve raster mutability
- problem with nodata semantics
- Add support for RPCs HOT 6
- Don't import rasterio.path
- Problem with raster resizing HOT 2
- IReadBlock failed
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from telluric.