-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Inconsistent use of gdal #1863
Labels
bug
Something isn't working
Comments
#1867 wasn't quite right. I will try to make a PR this week that does what I think was intended. Sorry that my explanation above wasn't clear. |
Merged
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Here
geemap/geemap/common.py
Line 10794 in 9c990ba
Whereas here
geemap/geemap/common.py
Line 15128 in 9c990ba
UseExceptions
is great as that will become the default behavior of GDAL in the future. I suggest always doing that right after the imports each time there is afrom osgeo import
gdal.PushErrorHandler
calls are missing the matchinggdal.PopErrorHandler()
. You are welcome to use this context manager if it makes things easier. https://github.com/schwehr/gdal-autotest2/blob/577974592837fdfc18fd90b338cb657f1f2332bd/python/gcore/gcore_util.py#L50The text was updated successfully, but these errors were encountered: