Skip to content

Add cr udata support#81

Open
mrke wants to merge 3 commits intoEcoJulia:masterfrom
mrke:add-CRUdata-support
Open

Add cr udata support#81
mrke wants to merge 3 commits intoEcoJulia:masterfrom
mrke:add-CRUdata-support

Conversation

@mrke
Copy link
Copy Markdown
Contributor

@mrke mrke commented Oct 14, 2025

Add support for CRU data, specifically CL v. 2. Note that this data comes in tabular form and needs to be reconstructed into a raster.

Comment thread Project.toml
Comment thread src/RasterDataSources.jl
include("modis/utilities.jl")
include("modis/examples.jl")

include("cru/shared.jl")
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does it need shared if there is only one?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was thinking we might add the other datasets later

Comment thread src/cru/shared.jl
@@ -0,0 +1,9 @@
struct CRU{X} <: RasterDataSource end
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it likely we will add the Weather timeseries (CRU TS) as well as Climate ?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it's all monthly so no?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I mean as a separate dataset (e.g. that makes the shared.jl necesary)

Comment thread src/cru/crucl2.0.jl Outdated
Comment thread src/interface.jl
WorldClim

"""
TODO
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We will need to explain their crazy versioning scheme where 1.1 is newer than 2.0 !

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes crazy

Comment thread src/cru/crucl2.0.jl
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there a reason to do CL 2.0 when 1.1 is (bizzarly) newer by 20 years?

Probably we need a versioning parameter to make it clear

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's 10min res rather than 0.5deg

Co-authored-by: Rafael Schouten <rafaelschouten@gmail.com>
@rafaqz
Copy link
Copy Markdown
Member

rafaqz commented Feb 10, 2026

Its weird that this is not a spatial raster file that e.g. GDAL can load. Not totally convinced it should live here just for the confusion it will cause.

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.

2 participants