-
Notifications
You must be signed in to change notification settings - Fork 34
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
Wrong input for resample gives confusing error ArchGDALBackendException #476
Comments
PR? If you can make it so it still gives that warning when ArchGDAL isnt available... |
I believe this is caused by the following in resample(args...; kw...) = throw(BackendException("ArchGDAL"))
warp(args...; kw...) = throw(BackendException("ArchGDAL")) We're defining these stubs with the goal of raising a I propose that we could fix it like so: function resample(args...; kw...)
if @isdefined ArchGDAL
throw(MethodError(resample, args))
else
throw(BackendException("ArchGDAL"))
end
end
function warp(args...; kw...)
if @isdefined ArchGDAL
throw(MethodError(warp, args))
else
throw(BackendException("ArchGDAL"))
end
end |
@JoshuaBillson that looks like a good solution to me |
I've opened PR #478 to fix this issue. My proposed solution didn't work as intended, so I ended up making it so that the Unfortunately, the |
Do you not get a duplicate method warning doing that? |
Yes, but I'm not sure what else we can do. Using |
Yeah but we cant have a warning. We can check if a method signature defined in the extension exists instead? Im not sure ArchGDAL will be available in that scope anyway |
I am trying to resample an extent to a different CRS so that I don't have to resample all data points and I get this confusing error where it complainst that ArchGDAL should be loaded.
ArchGDAL is loaded as visible from [3] and [2]
I would prefer to have a proper MethodError which would give a hint on what other methods are available.
The text was updated successfully, but these errors were encountered: