Skip to content
This repository has been archived by the owner on Dec 11, 2022. It is now read-only.

Big Chonky Boi #90

Merged
merged 45 commits into from
Apr 7, 2021
Merged

Big Chonky Boi #90

merged 45 commits into from
Apr 7, 2021

Conversation

tpoisot
Copy link
Member

@tpoisot tpoisot commented Mar 24, 2021

This is an API BREAKING release, in all the good ways. The plan is to tag a 0.5 release now that Julia 1.6 is out and we've had time to catch a few bugs.

Doc: http://docs.ecojulia.org/SimpleSDMLayers.jl/previews/PR90/

Reading and writing GeoTIFF files through the geotiff function is now fully supported without needing to define a custom type - this obsoletes the internal function raster

All data interaction are done through extensions of the SimpleSDMPredictor function, which accept a data provider and a dataset, as well as (in some cases) a climate model and a climate future. This obsoletes the worldclim, bioclim. and landcover functions, leading to fewer things in the namespace, which is good.

Future climates are represented by @enum types by model family.

The files are stored either at the location specified by ENV["SDMLAYERS_PATH"], or locally in the assets folder. The first solution is recommended as it allows projects to reuse folders.

The earthenv dataset has gained all of the texture layers.

WorldClim 2.1 has the CMIP6 scenarios under all four SSP6 for the BioClim dataset.

The geotiff function is able to burn multiple layers to a single file (and can read a specific band from a multi-band file).

There are still a few outstanding items, including:

  • docstrings for the new data interface
  • documentation update
  • future CHELSA
  • plotting recipes cleanup
  • README update

@codecov-io
Copy link

codecov-io commented Mar 24, 2021

Codecov Report

Merging #90 (a9c34b9) into master (993f286) will decrease coverage by 4.14%.
The diff coverage is 77.67%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #90      +/-   ##
==========================================
- Coverage   88.01%   83.86%   -4.15%     
==========================================
  Files          22       24       +2     
  Lines         609      719     +110     
==========================================
+ Hits          536      603      +67     
- Misses         73      116      +43     
Flag Coverage Δ
unittests 83.86% <77.67%> (-4.15%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/recipes/recipes.jl 90.47% <ø> (-2.63%) ⬇️
src/datasets/types.jl 26.66% <26.66%> (ø)
src/datasets/earthenv/download.jl 84.37% <84.37%> (ø)
src/datasets/geotiff.jl 96.72% <96.29%> (+0.29%) ⬆️
src/SimpleSDMLayers.jl 92.30% <100.00%> (+4.80%) ⬆️
src/datasets/chelsa/bioclim.jl 100.00% <100.00%> (ø)
src/datasets/chelsa/download.jl 100.00% <100.00%> (ø)
src/datasets/earthenv/habitatheterogeneity.jl 100.00% <100.00%> (ø)
src/datasets/earthenv/landcover.jl 100.00% <100.00%> (ø)
src/datasets/worldclim/bioclim.jl 100.00% <100.00%> (ø)
... and 7 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3198306...a9c34b9. Read the comment docs.

@tpoisot
Copy link
Member Author

tpoisot commented Apr 2, 2021

So worldclim is apparently down... This is fun. I like it.

@tpoisot tpoisot marked this pull request as ready for review April 7, 2021 14:56
@tpoisot tpoisot merged commit 42144e1 into master Apr 7, 2021
@gabrieldansereau gabrieldansereau deleted the tp/rasterdl branch May 13, 2021 12:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Burn multiple values to a single geotiff NODATA is not correctly handled
2 participants