Skip to content

Align period handling across workflows#11

Open
erin-glen wants to merge 1 commit intopreprocessing/disturbancesfrom
codex/review-scripts-for-processing-flow-and-inconsistencies
Open

Align period handling across workflows#11
erin-glen wants to merge 1 commit intopreprocessing/disturbancesfrom
codex/review-scripts-for-processing-flow-and-inconsistencies

Conversation

@erin-glen
Copy link
Owner

Motivation

  • Different workflows were using inconsistent time-period sets (TCC-gated vs full adjacent NLCD endpoints), causing mismatches in produced outputs.
  • Ensure insect/disease merge, Hansen-based harvest, fire expansion, and final combination consistently operate over the intended period set.
  • Preserve backward compatibility for existing TCC-gated consumers while adding an explicit full-period configuration.

Description

  • Added TIME_PERIODS_ALL, TIME_PERIODS_TCC, and preserved TIME_PERIODS as a back-compat alias in disturbance_config.py and introduced INSECT_TIME_PERIODS for insect/disease defaults.
  • Rewired FIRE_TIME_PERIODS to expand from TIME_PERIODS_ALL so fire processing always uses the full adjacent endpoint set.
  • Updated harvest_other.py and final_disturbance.py to iterate cfg.TIME_PERIODS_ALL, and updated insect_disease_merge.py to use getattr(cfg, "INSECT_TIME_PERIODS", cfg.TIME_PERIODS_ALL) for period iteration.
  • Aligned run_insect_disease.py to import disturbance_config and default HARDCODED_PERIODS to the configured insect-period keys, and fixed the run_disturbance.py filename/comment inconsistency.

Testing

  • No automated tests were executed as part of this change.

Codex Task

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.

1 participant