Skip to content

Set up config file for irrigation workflow - #4083

Merged
mdietze merged 6 commits into
PecanProject:developfrom
sarahkanee:fix/chirps-preprocess-dates
Aug 25, 2026
Merged

Set up config file for irrigation workflow#4083
mdietze merged 6 commits into
PecanProject:developfrom
sarahkanee:fix/chirps-preprocess-dates

Conversation

@sarahkanee

@sarahkanee sarahkanee commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Setting up config_paths.yml for all parts of the irrigation workflow (preprocess + tar_make).

Also includes the CHIRPS extract day parsing fix for precip_N layer names.

Comment thread workflows/irrigation-statewide/preprocessing/chirps-preprocess.R Outdated
Comment thread workflows/irrigation-statewide/preprocessing/chirps-preprocess.R Outdated
sarahkanee and others added 3 commits August 25, 2026 09:00
Co-authored-by: Chris Black <chris@ckblack.org>
Co-authored-by: Chris Black <chris@ckblack.org>
@sarahkanee
sarahkanee force-pushed the fix/chirps-preprocess-dates branch from 7e65695 to 3e03d9f Compare August 25, 2026 17:12

cimis_f0 <- file.path(cimis_dir, "2019", "01", "01", "ETo.asc.gz")
date0 <- as.Date(Sys.getenv("CIMIS_WEIGHTS_DATE", "2019-01-01"))
date_alt <- as.Date(Sys.getenv("CIMIS_WEIGHTS_DATE_ALT", "2015-03-01"))

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'd recommend replacing the Sys.getenv with a simple config file

@sarahkanee sarahkanee changed the title Update day parsing Set up config file for irrigation workflow Aug 25, 2026
@sarahkanee
sarahkanee force-pushed the fix/chirps-preprocess-dates branch from 414c97b to 63c4607 Compare August 25, 2026 19:47
@sarahkanee
sarahkanee force-pushed the fix/chirps-preprocess-dates branch from 3f6f7ea to b4b22ef Compare August 25, 2026 19:49
@mdietze
mdietze enabled auto-merge August 25, 2026 19:51
@mdietze
mdietze added this pull request to the merge queue Aug 25, 2026
Merged via the queue into PecanProject:develop with commit f148287 Aug 25, 2026
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants