2026 Software Engineering meeting notes - NGEET/fates GitHub Wiki
- Wins (1 min)
- Machine status and meeting notices (5 min)
- Upcoming Meetings (5 min)
- PR board status (10 min)
- Latest issues (5 min)
- FATES-CTSM6 Code freeze status (5 min)
- Calibration progress update
- Greg figured out the ERS issue for the Multi-column MVP
- Ryan making good progress on nutrients model updates
- FATES modeling meeting: May 7, Junyan Ding
-
Transatlantic coordination meeting setup
- Remember to fill out the Rally form sent via email
- Looks like Tuesdays at 10AM ET / 7AM PT
Integrated
Ready to Integrate
- Update FATES tag to API 44 (E3SM-Project/E3SM#8278)
- ctsm5.4.037: mirror: ctsm5.4.002_noresm_v6: fix for (fates) year-boundary restart issue (ESCOMP/ctsm#3940)
Updates
-
Split initial density into to parameters with a switch on HLM side (#1550): Reviewed. Matvey to revert update to seed init parameter setting.
- add new fates namelist switch for dbh init when nocomp is on (ctsm#3910): Reviewed. Matvey to add test for new option that updates seed init on the fly.
- improve hydro stability (#851): Ryan working on bringing this up-to-date and addressing issue that stalled this.
Latest issues and discussions of note
- no new issues
- Reference: maintenance branch document
- Rosie: Issues with historical runs
- Jessie's run should be done tomorrow.
- Matvey reconciling ctsm5.4.034: https://github.com/NorESMhub/CTSM/pull/212
- We should get an updated fates parameter file after this
- Jennifer: report back from E3SM leadership meeting, v4/v5 planning
- Seasonal-to-decadal
- Water and energy, extremes, biology
- Rosie: ECHOSAT product bookmark issue
- Wins (1 min)
- Machine status and meeting notices (5 min)
- Upcoming Meetings (5 min)
- PR board status (10 min)
- Latest issues (5 min)
- FATES-CTSM6 Code freeze status (5 min)
- Calibration progress update
- Jing's A2MC agent found it's first bug in the code.
- Ryan to create and issue and PR for this
- FATES modeling meeting: May 7, Junyan Ding
-
Transatlantic coordination meeting setup
- Ryan to use rally or timeful.app
Integrated
- ctsm5.4.034: bug fix to the FATES land use driver input code (ESCOMP/ctsm#3902), new LUPF variables (cherry-picked mirror of NorESMhub) (#1548), Update Grazing Logic (cherry-picked mirror of NorESMHub 55) (#1547), Complete the FATES-CLM nitrogen coupling (#1472)
Testing
- Update FATES tag to API 44 (E3SM-Project/E3SM#8278): Greg waiting for final e3sm land developer test to complete. Should be integrated this week.
- Complete the FATES-CLM nitrogen coupling (ESCOMP/ctsm#3409): Sam L addressing some review comments prior to testing
Under Review
- Split initial density into to parameters with a switch on HLM side (#1550): Greg to review this week. Last planned PR before maintenance branch creation.
Updates
New
- FATES API 45 (Nutrient NL switch updates) (#8325): Ryan drafted to bring align with #1472 updates
-
Boundary condition refactor - multi-column FATES MVP (glemieux/#36): building on top of #1477
- This will be turned into its on PR on NGEET repo once #1477 is merged.
Latest issues and discussions of note
- Cold-starting full-FATES, options (#1559)
- Adding a subdirectory to tools for supporting workflow pipelines (#1560)
- no updates
- Wins (1 min)
- Machine status and meeting notices (5 min)
- Upcoming Meetings (5 min)
- PR board status (10 min)
- Latest issues (5 min)
- FATES-CTSM6 Code freeze status (5 min)
- Calibration progress update
- Sam R: Presentation to NOAA about agentic coding
-
CISL: Tuesday May 05, derecho
- Sam R: this is moving to May 12-13
- NERSC: Wednesday May 20, perlmutter
-
FATES modeling meeting
- April 23: Cancelled
- May 7: Junyan Ding, COMPASS FATES-hydro work
-
Transatlantic coordination meeting setup
- Purpose: discuss future plans for FATES API changes. Coordination focus (e.g. avoid cherry-picking)
- Poll feedback
- Sam: Would be ideal to have date x time matrix selection. Note NCAR can't use when2meet due to malware in ads
Integrated
- Update LUH missing value checks (#1555), ctsm5.4.033: Update FATES tag with fix for LUH no_nan data set issue (ESCOMP/ctsm#3926)
Testing
- ctsm5.4.034: bug fix to the FATES land use driver input code (ESCOMP/ctsm#3902)
- new LUPF variables (cherry-picked mirror of NorESMhub) (#1548): Charlie made suggestions noting duplicates
Under Review
-
Complete the FATES-CLM nitrogen coupling (#1472)
- Complete the FATES-CLM nitrogen coupling (ESCOMP/ctsm#3409)
- Greg: Move this to testing or more review? Note that Sam Levis has been running build/run tests
- Sam R to add review
- Will: should this have testing beyond the regression testing?
- Discuss this within context of queue at CTSM SE meeting on Thursday as well as how to conduct future answer changing tests for ctsm-fates in the future.
Updates
-
Update FATES tag to API 44 (E3SM-Project/E3SM#8278): added landuse transition option (#1489), land use no nan data sets (#1555), fates restart issue fix (ctsm#3940)
- Merge fixed order of LUxPFT matrix read (#E3SM-Project/E3SM#8259) into #8278 if above is B4B during testing
- Split initial density into to parameters with a switch on HLM side (#1550): CTSM SE focused discussion last week has majority in support of brining this in
New
Latest issues and discussions of note
- Cold-starting full-FATES, options (#1559): Discussion shows interest in enabling this capability. Discuss with absent members next week as well.
- deferred
- Wins (1 min)
- Machine status and meeting notices (5 min)
- Upcoming Meetings (5 min)
- PR board status (10 min)
- Latest issues (5 min)
- Poll results
- FATES-CLM6 Code freeze status (5 min)
- Calibration progress update
- FATES modeling meeting: April 23, TBD
Integrated
New
-
Update LUH missing value checks (#1555): B4B results against pre-intel compiler update
- Charlie: recommend keeping the old logic as well to provide backwards compat with folks who might be using the NaN
_FillValueland use data sets- Ryan: Note that
isnandoesn't always work; Recommending checking that the value matches itself.
- Ryan: Note that
- Ryan: Historically we don't do logic tests against floats.
- ctsm5.4.033: Update FATES tag with fix for LUH no_nan data set issue (ESCOMP/ctsm#3926)
- Update FATES tag to API 44 (E3SM-Project/E3SM#8278)
- Charlie: recommend keeping the old logic as well to provide backwards compat with folks who might be using the NaN
- fixed order of LUxPFT matrix read (#E3SM-Project/E3SM#8259)
Under Review
-
Update Grazing Logic (cherry-picked mirror of NorESMHub 55) (#1547): Question to Rosie on status of last comment
- Is the NorESM team not implementing the recent commits?
- If so, should the main fates branch mirror that fact (i.e. should we revert those updates)?
- Charlie is ok with going whichever way that Rosie wants to go.
- Action: change paranthesis to match previous logic
- If so, should the main fates branch mirror that fact (i.e. should we revert those updates)?
- Is the NorESM team not implementing the recent commits?
-
new LUPF variables (cherry-picked mirror of NorESMhub) (#1548): Charlie made suggestions noting duplicates
- Ryan to review
Updates
-
Split initial density into to parameters with a switch on HLM side (#1550)
- add new fates namelist switch for dbh init when nocomp is on (ctsm#3910)
- Ryan to work up alternate version of this
-
Complete the FATES-CLM nitrogen coupling (#1472): bring this in for fates-maintenance?
- Rosie: the sooner this comes into NorESM, the better.
Latest issues and discussions of note
- error in land use driver data in land use x PFT data (#1551)
- LUH
NaNissues (see above new PRs): #1552 with children #1553, #1554 - FYI: not reproducible at year boundary (NorESMhub/NorESM#790): coupled restart issue
- Full support for update sci tag to 2.0.0
- no updates
- Rosie: Bringing MIMICS plus to FATES
- Wins (1 min)
- Machine status and meeting notices (5 min)
- Upcoming Meetings (5 min)
- PR board status (10 min)
- Latest issues (5 min)
- FATES-CLM6 Code freeze status (5 min)
- Calibration progress update
- FATES modeling meeting: April 9, Zachary Robbins
- Improving Scientific Software conference: April 6-9
- CTSM SE meeting impacted
Integrated
-
Updates to land use clearing logic (#1489)
- ctsm5.4.030: Add a FATES namelist option for land use transition logic (ctsm#3728): minor build update and re-test
New
- Add guidance on the use of AI contributions (#1549)
- Split initial density into to parameters with a switch on HLM side (#1550)
Under Review
- Update Grazing Logic (cherry-picked mirror of NorESMHub 55) (#1547)
- new LUPF variables (cherry-picked mirror of NorESMhub) (#1548)
Latest issues and discussions of note
- No new issues or discussions
- Machine status and meeting notices (5 min)
- Upcoming Meetings (5 min)
- Latest issues (5 min)
- Calibration progress update
- FATES modeling meeting: March 12: Performance Improvements (Ryan)
- Machine status and meeting notices (5 min)
- Upcoming Meetings (5 min)
- PR board status (10 min)
- Latest issues (5 min)
- Debrief from LMWG Meeting
- Calibration progress update
- FATES-CLM6 Code freeze status (5 min)
- Speed Refactor Updates
- FATES modeling meeting: March 12: TBD
New
Integrated
- Frozen Hydro Mort Fix (#1533)
- E3SM JSON API (7954)
- Adding some more land-use-change and globally-relevant history variables (#1485)
Testing
- Damage Loop Escape (1532)
- LU Super PR
- Nitrogen Coupling (#1472): Reproducing ERS restart failure.
Updates/Reviewing
Discussing
Latest issues and discussions of note
- Machine status and meeting notices (5 min)
- Upcoming Meetings (5 min)
- PR board status (10 min)
- Latest issues (5 min)
- Speed Refactor Overview
- Calibration progress update
- FATES-CLM6 Code freeze status (5 min)
- FATES modeling meeting: 26 February: TBD
- CLM Land Model Working Group meeting: 24-26 February
Integrated
Testing
- NEXT: frozen soil mortality
- Adding some more land-use-change and globally-relevant history variables (#1485) - Need Sam R approval
-
JSON formatted FATES parameters: ready for review, tested on pm-cpu
- Get this into the integration queue ASAP so we can see how regression tests work out across other machines
- Needs rebasing against master after #7899 integrated
Updates/Reviewing
- Add some biophysical variables (#1407)
- Refactor boundary condition interface structure (#1477)
- Damage Loop Escape (1532)
Discussing
New -add area and biomass history variables indexed by landuse x pft #1535
Latest issues and discussions of note
- Machine status and meeting notices (5 min)
- Upcoming Meetings (5 min)
- PR board status (10 min)
- Latest issues (5 min)
- FATES-CLM6 Code freeze status (5 min)
- Calibration progress update
- FATES modeling meeting: 12 February: TBD (likely office hours)
- CLM Land Model Working Group meeting: 24-26 February
Integrated
Testing
- Adding some more land-use-change and globally-relevant history variables (#1485) - Need Sam R approval
-
JSON formatted FATES parameters: ready for review, tested on pm-cpu
- Get this into the integration queue ASAP so we can see how regression tests work out across other machines
- Needs rebasing against master after #7899 integrated
- Refactor boundary condition interface structure (#1477): ready for review, tests are (near) B4B
Updates/Reviewing
Discussing
New
Latest issues and discussions of note
- Wins (1 min)
- Machine status and meeting notices (5 min)
- Upcoming Meetings (5 min)
- PR board status (10 min)
- Latest issues (5 min)
- FATES-CLM6 Code freeze status (5 min)
- Outputs for CMOR and simulations
- Calibration progress update
- FATES modeling meeting: 12 February: TBD (likely office hours)
- CLM Land Model Working Group meeting: 24-26 February
Integrated
- Updates to FATES zenith angles on the first step of non-continue restarts (E3SM#7899): waiting to be integrated
Testing
-
Testing Framework Update (#1519): Greg to finish this
- Adrianna to update the code to convert CDL to NetCDF.
- Greg to check in end of day. It's ok to just move forward with uploading the netcdf file.
- Adrianna to update the code to convert CDL to NetCDF.
Updates
-
JSON formatted FATES parameters: ready for review, tested on pm-cpu
- Get this into the integration queue ASAP so we can see how regression tests work out across other machines
- Needs rebasing against master after #7899 integrated
- Refactor boundary condition interface structure (#1477): ready for review, tests are (near) B4B
New
- direct usage of parameter datastructure - allometry proof of concept
-
Add a FATES namelist option for land use transition logic #3728
- Coupled with Updates to land use clearing logic (#1489)
- Keep the parameter file update, Jessie to update to JSON format, provide B4B values for new param in comments of the PR
- Coupled with Updates to land use clearing logic (#1489)
Latest issues and discussions of note
-
CLM-FATES results in odd global forest patterns when hydro is on (#1527)
- Marcos meeting with Steven on Thursday to discuss this more
- Charlie: Soil texture or temperature initial condition wrt. Simple check, initialize soils to saturated and check.
- Marcos: Difference in pedotransfer between ELM and CLM?
- Will: Maybe surface dataset differences?
- relevant discussion below
-
NorESM CTSM issue #188
- Rosie: Is CTSM going through this?
- Will: Yes, bringing in Sam Levis who has been working on this
- Charlie to reply in thread to remaining issues we didn't get to today
- Rosie: Is CTSM going through this?
- Lack of very big trees in the boreal region email thread (Adrianna, Rosie, Jessica)
- AGB allometry parameters seem to have the greatest impact on creating large trees (reducing too many little trees)
- GPP increased faster (wider DBH -> more leaves)
- AGB allometry parameters seem to have the greatest impact on creating large trees (reducing too many little trees)
- Wins (1 min)
- Machine status and meeting notices (5 min)
- Upcoming Meetings (5 min)
- PR board status (10 min)
- Latest issues (5 min)
- FATES-CLM6 Code freeze status (1 min)
- Calibration progress update
- FATES modeling meeting:
- CLM Land Model Working Group meeting: 24-26 February
- Will: Request for presentations open until the 06 February. Note this year is a CuPID tutorial.
- Will asked Rosie to give a talk about NorESM. Adrianna will give a talk. Would be good to get FATES overview talk like previous years.
- Adrianna: We should hold a hackathon to get FATES output variables into CuPID
- Will: Request for presentations open until the 06 February. Note this year is a CuPID tutorial.
Integrated
- changes for NorESM nocomp simulation efforts #1510 Testing
- Updates to FATES zenith angles on the first step of non-continue restarts (E3SM#7899)
Latest issues and discussions of note
- Will: Matvey has a number of PRs that address bugfixes
Canceled due to federal holiday
- Wins (1 min)
- Machine status and meeting notices (5 min)
- Upcoming Meetings (5 min)
- PR board status (10 min)
- Latest issues (5 min)
- FATES-CLM6 Code freeze status (1 min)
- Calibration progress update
- FATES modeling meeting: 15 January: Office hours
- CLM Land Model Working Group meeting: 24-26 February
- Will: Request for presentations open until the 06 February. Note this year is a CuPID tutorial.
- Will asked Rosie to give a talk about NorESM. Adrianna will give a talk. Would be good to get FATES overview talk like previous years.
- Adrianna: We should hold a hackathon to get FATES output variables into CuPID
- Will: Request for presentations open until the 06 February. Note this year is a CuPID tutorial.
Integrated
Testing
- changes for NorESM nocomp simulation efforts #1510: Waiting on computing account top up
-
Updates to FATES zenith angles on the first step of non-continue restarts (E3SM#7899): Reviewing unexpected DIFFs
- Greg and Ryan think they've ID'd a fix.
Latest issues and discussions of note
- Will: Matvey has a number of PRs that address bugfixes
- no updates
- Wins (1 min)
- Machine status and meeting notices (5 min)
- Upcoming Meetings (5 min)
- PR board status (10 min)
- Latest issues (5 min)
- FATES-CLM6 Code freeze status (1 min)
- Greg: nutrients test in BC refactor now B4B
- FATES modeling meeting: 15 January: TBD
- NGEE-Arctic All-Hands: 12-13 January
Integrated
Review
-
changes for NorESM nocomp simulation efforts #1510
- Ryan: Ryan is creating a patch file for holding the nocomp negative init density runs. Rosie can use this to update with the nocomp calibration values.
Needs Reviews
- Unit Fix to Smoothed NPP used for free living fixation: Greg to review toda
Updates
- Nitrogen Coupling (#1472): Ryan and Sam continued to meet
- Updates to FATES zenith angles on the first step of non-continue restarts (E3SM#7899): Greg to test
New
-
Testing Framework Update (#1519)
- Walkthrough
- New script:
generate_empty_testsets up boilerplate stuff for new test development- See templates directory
- Same top line scripts for running tests
- Improved readme with test build walkthrough example
- Put tests in the
configfile when creating new ones - Tests actually live in
testing/tests/<test_type> -
frameworkis where the python testing infrastructure lives- Expanded classes for different test types
-
builderis responsible for compiling everything
- New script:
- Walkthrough
Latest issues and discussions of note
- Create release github branch (#1520): task issue for the FATES-CLM6 freeze (see below)
- Greg: Release tag deadline for ctsm still in flux
- #1510 is likely the last stuff that will need to come in to make a release
- NorESM freeze is imminent
- #1510 is likely the last stuff that will need to come in to make a release
- Discussion of boreal forest LAI issue
- MODIS LAI is much lower than fates
- Charlie: do we need to improve the canopy snow interception model (to adjust LAI due to snow occlusion)
- Ryan: two-stream model does account for snow occlusion, but are we passing this information to the HLM?
- Adrianna: working on c4 grass calibration