Skip to content
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

resample and warp now throw MethodError after loading ArchGDAL #478

Closed
wants to merge 11 commits into from
Closed

resample and warp now throw MethodError after loading ArchGDAL #478

wants to merge 11 commits into from

Conversation

JoshuaBillson
Copy link
Contributor

As discussed in #476, resample and warp were throwing BackendException after loading ArchGDAL instead of MethodError. This is fixed by redefining the stubs when ArchGDAL is loaded.

JoshuaBillson and others added 11 commits June 9, 2023 15:43
…opriate backend.

The previous error message for reading rasters without an imported backend was confusing. The new message informs callers of the change and suggests the appropriate backend to use.
…opriate backend.

The previous error message for reading rasters without an imported backend was confusing. The new message informs callers of the change and suggests the appropriate backend to use.
…opriate backend.

The previous error message for reading rasters without an imported backend was confusing. The new message informs callers of the change and suggests the appropriate backend to use.

The error now reads, "`Rasters.jl` requires backends to be loaded externally as of version 0.8. Run `import $packagename` to read $filename"
@codecov-commenter
Copy link

codecov-commenter commented Jul 23, 2023

Codecov Report

Merging #478 (58600a6) into main (bf81f58) will increase coverage by 0.06%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #478      +/-   ##
==========================================
+ Coverage   81.10%   81.16%   +0.06%     
==========================================
  Files          49       49              
  Lines        4101     4104       +3     
==========================================
+ Hits         3326     3331       +5     
+ Misses        775      773       -2     
Impacted Files Coverage Δ
ext/RastersArchGDALExt/RastersArchGDALExt.jl 100.00% <100.00%> (ø)

... and 2 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@rafaqz
Copy link
Owner

rafaqz commented Jul 25, 2023

Closing in favour of #480

@rafaqz rafaqz closed this Jul 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants