Skip to content

Moved update_gddlai subroutine into new module file for lai calculations - #1729

Merged
sunt05 merged 9 commits into
masterfrom
matthewp/new-lai-module
Sep 16, 2026
Merged

sunt05 merged 9 commits into
masterfrom
matthewp/new-lai-module

Conversation

@MatthewPaskin

Copy link
Copy Markdown
Contributor

No description provided.

@MatthewPaskin MatthewPaskin self-assigned this Sep 2, 2026
@MatthewPaskin MatthewPaskin added the 4-in-progress Being worked on label Sep 2, 2026
@github-actions

github-actions Bot commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

CI Build Plan

Changed Files

Fortran source (3 files)

  • src/suews/Makefile
  • src/suews/src/suews_phys_dailystate.f95
  • src/suews/src/suews_phys_lai.f95

Build system (1 file)

  • src/supy/meson.build

Build Configuration

Configuration
Platforms Linux x86_64, macOS ARM64, Windows x64
Python 3.12
Test tier standard (non-slow + core physics regressions)
PR status Ready (standard matrix)

Rationale

  • Fortran source changed -> multiplatform build required
  • Build system changed -> multiplatform build required

Updated by CI on each push. See path-filters.yml for category definitions.

@sunt05

sunt05 commented Sep 15, 2026

Copy link
Copy Markdown
Contributor

Hi @MatthewPaskin — CI is green here and it merges cleanly, but the 4-in-progress label is still on. Is this ready for review, or still in flight?

@MatthewPaskin

Copy link
Copy Markdown
Contributor Author

@sunt05 This PR was simply intended as base to be expanded with further ideas to migrate LAI related code to a new file. Currently only the recent refactored code is stored here, largely because of the change in style compared to the rest of the daily state file, therefore separating the styles for maintainability.

That said, I don't have time to move any other code to this file at this time and other branches that expand the LAI/GDD behaviour have been built upon this branch and I think it is a worthy merge in its current state.

I will mark it for review. Thanks!

@MatthewPaskin MatthewPaskin added 4-ready Ready for work and removed 4-in-progress Being worked on labels Sep 16, 2026
@sunt05

sunt05 commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

Thanks Matthew, this is a clean lift. I have reviewed it against master and the only changes are the renames, crit_days promoted to module scope, and the three LAI calculations turned into functions, so no physics moves. CI agrees: the full test_laimethod.py suite and the full-year sample-output comparison are green.

Merging now. A few bits of tidying came out of the review and are in #1811, none of which needed to hold this up.

@sunt05
sunt05 added this pull request to the merge queue Sep 16, 2026
Merged via the queue into master with commit ed0ecef Sep 16, 2026
23 checks passed
@MatthewPaskin
MatthewPaskin deleted the matthewp/new-lai-module branch September 17, 2026 07:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4-ready Ready for work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants