Skip to content

Add irrigation projection workflow - #4106

Merged
mdietze merged 7 commits into
PecanProject:developfrom
ananyak534:irrigation-predictions
Sep 17, 2026
Merged

mdietze merged 7 commits into
PecanProject:developfrom
ananyak534:irrigation-predictions

Conversation

@ananyak534

Copy link
Copy Markdown
Contributor

Adds the statewide irrigation projection workflow

PR includes:

  • get_climate_data.R for converting the downloaded 198 Cal-Adapt design points into county-level daily climate inputs
  • irrigation.R for generating parcel-level irrigation projections
  • 06-irrigation_predictions.md as a documentation draft

The workflow combines projected crop identity and phenology with Cal-Adapt climate, crop water parameters, and SSURGO-derived available water capacity to generate annual irrigation events.

ccmmf_root = "/projectnb/dietzelab/ccmmf"
```

Most users should not need to modify `ccmmf_root`.

@mdietze mdietze Sep 14, 2026 •

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.

Suggested change
Most users should not need to modify `ccmmf_root`.
Most users will need to modify `ccmmf_root`.

├── planting_projections/
├── harvest_projections/
└── caladapt_county_daily_climate.csv
```

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.

make sure it's clear where each of these is coming from. For example, I don't think any of your previous documents indicated where caladapt_county_daily_climate.csv was generated from

2018–2023
```

The current climate selection is:

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.

maybe explain what the valid options for these are and what they mean

The raw files are stored under:

```text
/projectnb/dietzelab/ccmmf/ensemble/CalAdapt_runs/data_raw/CalAdaptWRF

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.

needs generalization. For CARB these should be in their S3 bucket. For other users maybe point to the code that generates this. Maybe @infotroph could provide a hint

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.


This prevents the full soil calculation from being repeated every time the irrigation workflow is run. :contentReference[oaicite:3]{index=3}

Rice does not use this soil AWC calculation because it is modeled using a separate flooded-field water balance.

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.

FYI rice fields should still need a AWC, it's just more that the thresholds for irrigation (minimum and field capacity) are different

Comment thread modules/data.remote/inst/get_climate_data.R
Comment thread modules/data.remote/inst/irrigation.R

## Setup

Setup remains the same that it has been throughout the workflow: running `pacman::p_load` and making

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.

per earlier comments, drop pacman and tweak to focus on what needs to be configured in the config.yml, not on the line of code that read it.

@@ -0,0 +1,601 @@
# Irrigation Projection

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.

This file has an almost identical name to the previous one, and seems to be an earlier version of the same info. That said, before deleting, please read the comments I left as most apply equally to the new version (e.g., documenting where climate scenario files come from and what the allowable options are for ssp and gcm)

@mdietze
mdietze enabled auto-merge September 17, 2026 15:34
@mdietze
mdietze added this pull request to the merge queue Sep 17, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Sep 17, 2026
@mdietze
mdietze added this pull request to the merge queue Sep 17, 2026
Merged via the queue into PecanProject:develop with commit 773baa8 Sep 17, 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