Skip to content

add ca fertilization and ncc compost statewide event workflows#4003

Open
divine7022 wants to merge 20 commits into
PecanProject:developfrom
divine7022:event-workflows
Open

add ca fertilization and ncc compost statewide event workflows#4003
divine7022 wants to merge 20 commits into
PecanProject:developfrom
divine7022:event-workflows

Conversation

@divine7022
Copy link
Copy Markdown
Member

Description

adds two statewide event workflows for CA ag parcels and the matching cleaners that feed into preprocess-event-parquet

workflows/fertilization-statewide draws ensemble annual N rates per parcel from PEcAn.data.land::ca_n_application_rate and writes parcel range sharded parquets. workflows/ncc-statewide does the same shape for compost amendments, calling the bundled sample_ca_compost_* functions from PEcAn.data.land. both write to /projectnb/dietzelab/ccmmf/usr/akash/event_files/ for now

workflows/preprocess-event-parquet picks up 01c-clean-fertilization.R and 01d-clean-ncc.R

depends on PR #4002 for few functions plus the extended events schema

cc @sarahkanee, @ashiklom, @mdietze would love your eyes on this;
fyi as well @infotroph @dlebauer since u have both been following thread

Motivation and Context

Review Time Estimate

  • Immediately
  • Within one week
  • When possible

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My change requires a change to the documentation.
  • My name is in the list of CITATION.cff
  • I agree that PEcAn Project may distribute my contribution under any or all of
    • the same license as the existing code,
    • and/or the BSD 3-clause license.
  • I have updated the CHANGELOG.md.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

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.

1 participant