-
Notifications
You must be signed in to change notification settings - Fork 92
2023 Software Engineering meeting notes
- Upcoming FATES meetings (5 min)
- Weekly synthesis of the PR Status Board (5-10 min)
- Latest issues (5-10 min)
- Calibration progress update (5-10 min)
- Walk-on topics
-
FATES modeling meeting: 4 Jan 2024, Zachary Robbins
- Next FATES Software meeting? 8 January 2024?
- AGU reflections?
- Updates
- V1 Landuse Change: Derecho regression testing in progress. Nag testing on izumi revealed some issues last week.
-
FATES testmod update (6018):
SOIL2N_TNDNCY_VERT_TRANS
seems to be fates-specific issue- Put this on back burner and move onto E3SM#6027 (refactor API update)
- Two-stream: working on retaining Norman b4b-ness
- New
-
New allometric modes, ST3 leaf phenology,
Leafless hydraulic failure disable, Optional crown area output,
Exponential-based C starvation mortality
- Marius' hardness work in the context of leafless hydraulic failure PR discussion
- Consideration of BTRAN for leafless hydraulic failure PR?
- SP mode + new allometic mode testing
-
New allometric modes, ST3 leaf phenology,
Leafless hydraulic failure disable, Optional crown area output,
Exponential-based C starvation mortality
Latest issues and discussions
-
Q10 maint resp for c4 grasses
- We are moving away from Ryan (to Atkin), so is this a "wont-fix" issue?
- Both Ryan and Atkin don't have values for c4 baserate, so both suffer from a similar lack of data.
- Consensus is that we should convert this to a discussion and continue there.
- We are moving away from Ryan (to Atkin), so is this a "wont-fix" issue?
- Email discussion: pan-amazon fates hydro errors in nocomp and SP configurations
- Chongang looking into a possible solution to the dry failure cases that Jenny is seeing in a forthcoming PR. He will point Jenny to the old PR.
- Jessie: allometry updates degrade ILAMB scores. Pattern is similar, just worse.
- That said, this is constrained by data, so plan is to move forward calibrating with this.
- Incorporating fire improved biomass, but did not impact LAI, likely due to there not being any understory.
- Upcoming FATES meetings (5 min)
- Weekly synthesis of the PR Status Board (5-10 min)
- Latest issues (5-10 min)
- Calibration progress update (5-10 min)
- Walk-on topics
- FATES modeling meeting: 7 December, AGU Lightning Round
-
AGU: 11-15 December
- Question: cancel this meeting next week?
-
CISL: none planned
- Note: Erik is working on the first ctsm tag for running on derecho
- NERSC: Wed 17 Jan, perlmutter
- OLCF: Tue 05 Dev, frontier
- Updates
- V1 Landuse Change: Regression testing in progress. Investigating CNBalanceCheck error.
- Two-stream:
Latest issues and discussions
- New issue
-
NPP flux calculation timing: Jessie communicated this from folks in DBEN project.
- Jessie: Upon reflection, while this specific issue is valid, it may not be specific to the issue Jessie is seeing.
-
NPP flux calculation timing: Jessie communicated this from folks in DBEN project.
- Discussion
- Marcos answered #1120
- Adrianna: Re-running PPE. May have to move over to Derecho.
- Rosie: No updates (in Sweden last week)
- Jessie: Waiting on runs to complete
- Adrianna: Burn area results in Alaska is looking odd. Charlie and Adrianna to follow-up.
- Ryan: Note that hardcoded localization parameters affects distribution of litter to disturbed and new patch.
- Near term goal probably should be to unbury this and put it into the
- Ryan: Note that hardcoded localization parameters affects distribution of litter to disturbed and new patch.
- Rosie: seed dispersal discussion. Rosie learned about seed dispersal with LPJ-GUESS. Debra.
- Savannah workshop this week. C4 grass physiology. Who is working on this currently?
- Not sure current status. Check in with Nick Smith, Danica.
- Savannah workshop this week. C4 grass physiology. Who is working on this currently?
- Upcoming FATES meetings (5 min)
- Weekly synthesis of the PR Status Board (5-10 min)
- Latest issues (5-10 min)
- Calibration progress update (5-10 min)
- Walk-on topics
- Meetings, conferences, seminars, etc.
-
FATES modeling meeting: 7 December, AGU Lightning Round
- Charlie will send out email soliciting AGU lightning talks
-
FATES modeling meeting: 7 December, AGU Lightning Round
- Machine outages
- Integrated
- Updates
-
FATES testmod update (6018) and API31 update (6027): waiting on e3sm review
- Odd singular RMS diff with
AllVars
for e3sm-only output.
- Odd singular RMS diff with
- LUH2 V1: Greg to catch up with Erik soon about where ctsm-side will fall in queue
- Question (Ryan): what should the calibration branch bring in next?
- Charlie: When Jessie has calibrated nocomp branch, Charlie is going to attempt to bring in landuse + nocomp
- Aside: Charlie relayed that Jessie is seeing the model run slowly.
- Charlie and Adrianna suspect that the Medlyn convergence "toughness" maybe be major contributing factor.
-
FATES testmod update (6018) and API31 update (6027): waiting on e3sm review
Latest issues and discussions
- No new issues or discussion
- Adrianna: Tuning Kumarathunge?
- Charlie: Prefer to use paper's parameter which are tuned to observations.
- Charlie/Rosie: How conditional on their own pft vcmax numbers are the other non-acclimation parameters provided in the paper (Table 2)
- Check in with Alistair on this; Rosie to take action.
- Charlie/Rosie: How conditional on their own pft vcmax numbers are the other non-acclimation parameters provided in the paper (Table 2)
- Marcos: from ED2, it was important to have C4
vcmax25top
lower than C3 grasses to avoid GPP spiking.
- Charlie: Prefer to use paper's parameter which are tuned to observations.
- Adrianna: working on boreal fire
- Bulk density seems low for boreal. Moss isn't present, which is a major contribution.
- Additionally, perhaps we want to have different classes of leaf litter (needle versus broad)?
- Bulk density seems low for boreal. Moss isn't present, which is a major contribution.
- Marcos: testing drought-deciduous phenology (particularly timing)
- Run SP + ST3? Idea is to have some initial forest inventory from SP, but keep it static via ST3
- Ryan: alternate running ST3, but manually turn on phenology, but bypass some checks
- Upcoming FATES meetings (5 min)
- Weekly synthesis of the PR Status Board (5-10 min)
- Post NGEE-Tropics All Hands Reflection
- Latest issues (5-10 min)
- Calibration progress update (5-10 min)
- Walk-on topics
- Meetings, conferences, seminars, etc.
- FATES modeling meeting: 7 December, AGU Lightning Round
- Machine outages
- Integrated
- Updates
- LUH V1 (1040): Greg addressed ctsm-side review comments
- FATES testmod update (6018)
- API31 update (6027)
- Two-stream: fine tuning smaller area thresholds are triggering area checks
- Rosie: canopy shyness available in two-stream?
- Ryan: we don't have logic for this in two-stream, but could indeed use two-stream structure to accomplish this in the future.
- Rosie: canopy shyness available in two-stream?
- Tree allometry status update?
- Will kick off comparison test to previous run.
- Question: should we incorporate #990? Consensus is yes. Set to calibration value.
- Long term discussion: should we remove this? Complexity versus usefulness.
- New
- Type decleration AMD issue fix (1121): regression test in progress
Latest issues and discussions
- Adrianna: Incorporated crop -> grass mapping. GPP is higher now (both in global total and map structure).
- Do we want to calibrate grasses and crops together, or separately (i.e. separate crop pft)? Consensus is separately.
- Adrianna post-doc opening: https://ucar.wd5.myworkdayjobs.com/UCAR_Careers/job/Mesa-Lab-Building/TSS-Postdoctoral-Fellow_REQ-2023-402-1
- Cable crop paper: https://www.science.org/doi/10.1126/sciadv.adh9444
- Rosie: https://essd.copernicus.org/articles/13/4881/2021/
- Are we aware of any other work in this area? Looking at doing demography MIP.
- Jessie: Tom Pugh, https://www.pnas.org/doi/10.1073/pnas.1810512116
- Charlie: Ben poulter database, https://doi.pangaea.de/10.1594/PANGAEA.897392
- Another product that we discussed adding to lamb https://github.com/rubisco-sfa/ILAMB-Data/issues/51
- Are we aware of any other work in this area? Looking at doing demography MIP.
- Upcoming FATES meetings (5 min)
- Weekly synthesis of the PR Status Board (5-10 min)
- Latest issues (5-10 min)
- Tree allometry update discussion (5-10 min)
- Calibration progress update (5-10 min)
- Walk-on topics
- Meetings, conferences, seminars, etc.
- NGEET Annual All-hands meeting: 14-17 November
- FATES modeling meeting: 7 December, AGU Lightning Round
- Machine outages
- Integrated
-
het resp fix (1114), long run restart fix (1098)
- Both required API updates (API 29 and 30, respectively)
- E3SM repo closed last week; E3SM-side API PRs still waiting in integration queue
-
het resp fix (1114), long run restart fix (1098)
- Updates
- LUH v1: Greg addressing and resolving review comments
- e3sm testing PR: Greg and Ryan discussed expanded test lists on e3sm side for fates
- Greg and Ryan will run the expanded 'fates' list for API updates
- Actions: ask e3sm land folks how many tests we can add to e3sm_land_developer suite
- Note that this will increase the number of fates tests that run nightly and as such will increase our 'exposure' to maintaining these tests as e3sm does not really allow "expected failures".
- New
-
LUH2 data tool "year" fix (1112): Ryan has approved.
- This should have automated unit testing to make sure to avoid issues like this in the future
- Landuse V2 (1116): this is still a WIP technically. One edge case to still handle.
-
User history density option (1119)
- The namelist controls not just the output, but the allocation as well
- Will have graceful failure for when user request including an output that conflicts with the level requested
-
LUH2 data tool "year" fix (1112): Ryan has approved.
Latest issues and discussions
- Medlyn FPE
- patch dimension bound error, which relates to this line of code
- Path forward: Fit to BAAD database but use crown area assuming exponent is the same.
- deferred to next week
- no topics
- Upcoming FATES meetings (5 min)
- Weekly synthesis of the PR Status Board (5-10 min)
- Calibration progress update (5-10 min)
- Two stream (5-10 min)
- Walk-on topics
- Meetings, conferences, seminars, etc.
- FATES modeling meeting: 09 November, Sophia Macarawich, Paleoclimate FATES work
- Machine outages
- Updates
- Notice: Greg added upcoming API plan to the User's Guide HLM table
- Developers can use this to see what the integration plan is a little more cleanly than the PR board.
-
Het resp fix (1114): Reviewed and approved. CTSM-side API update testing complete. E3SM-side testing underway.
- Issue is that the
bc_out%ema_npp
was not getting updated in time for the hi frequency output as the model restart and the fates update ofema_npp
was not in sync (due toema_npp
update only happening daily). - Note this is being rolled into E3SM-side seed dispersal PR.
- Issue is that the
- Notice: Greg added upcoming API plan to the User's Guide HLM table
- New
- Hydraulic failure frozen soils (1115): Needs review, but looks like a slam dunk.
Latest issues and discussions
- None to discuss
- Jessie: tried pulling in Rosie's crop updates (PR 817), two-stream, and Medlyn parameters. Got a bunch of failures that she's working on fixing before comfortable with the runs.
- Rosie believes she was using Ball-Berry and not Medlyn. Adrianna running default Medlyn parameters?
- Adrianna: not using default. Using Charlie's recommended from another reference. Also saw same crashes that Jessie is seeing.
- Discussion of low GPP in Adrianna's recent results.
- Adrianna: not using default. Using Charlie's recommended from another reference. Also saw same crashes that Jessie is seeing.
- Note: changes in landuse v2 reformulates the no-comp initialization logic
- Rosie believes she was using Ball-Berry and not Medlyn. Adrianna running default Medlyn parameters?
- Jenny triggered an error: check on solver results.
- One particular pft was causing the issue
- Ryan looked into LAPACK to understand what could be causing the issue (i.e. what is acceptable precision for matrix inversion?)
- Issue turned out to be if two differencing terms (beam and diffusion) in eq 26 of Ryan's manuscript is near zero, values become huge. Ryan to introduce check and small perturbation to avoid this state.
- Adrianna: fire model refactoring discussion. Proposing creating a patch fuel data class.
- Would take in litter and other states and calculate/store fuel fraction, bulk density, etc.
- Has unit tests and functional tests :D!
- Consensus is yes.
- Upcoming FATES meetings (5 min)
- Weekly synthesis of the PR Status Board (5-10 min)
- Latest issues (5-10 min)
- Calibration progress update (5-10 min)
- Walk-on topics
-
Integrated
- Cross grid seed dispersal is in!
-
Bottlenecks:
- Demography initialized gridded testing would require:
- #1093 (allometry updates)
- #1104 (cleanup initialization file format)
- #1040 (v1 landuse change?)
- Demography initialized gridded testing would require:
-
Updates
- Solar Radiation Unit Tests #1034
- Maybe adding bleaf allometry updates to #1093 ?
- Active crown fire #1108
- Long-run restarts #1098
-
New
Latest issues and discussions
- Rosie: spectral element grids continued?
- Adrianna: LAI difference continued?
- Jennifer: FATES overview at Biogeosciences ESA?
- Upcoming FATES meetings (5 min)
- Weekly synthesis of the PR Status Board (5-10 min)
- Latest issues (5-10 min)
- Calibration progress update (5-10 min)
- Walk-on topics
- Meetings, conferences, seminars, etc.
- FATES modeling meeting: 26 October, Bharat Sharma, ELM-FATES at ORNL and Duke FACE sites
- Machine outages
- Integrated
- cross-grid seed (1005): API-side still to be integrated. Izumi testing TBD. E3SM testing today.
- Updates
-
Two stream: Marcos and Adrianna providing reviews.
- Aside: Rosie noted that Mariana is looking to do some performance testing of fates. Rosie suggested she waited until this one was integrated.
- Ryan wants to create a PR (HLM-side) to add namelist variable to tailor the set of history variable "classes" (i.e. site versus muliplexed). This would avoid calling the subroutines that populate the particular history variables.
- Aside: Rosie noted that Mariana is looking to do some performance testing of fates. Rosie suggested she waited until this one was integrated.
-
tree pfts update (1093): Jessie reviewed the impact of
fates_allom_blca_expnt_diff
. Including this, which adjusts LAI wrt size, improves ILAMB results. Let move this in.
-
Two stream: Marcos and Adrianna providing reviews.
- New
-
clean inventory init: removes extra columns and adds ed2 to fates python tool
- Put
c_height
back in.- Aside: Marcos noted that dbh -> height isn't perfectly reversible
- Consensus is that we should just remove the ED2 compatible format instead of keeping both FATES-only and ED compatible.
- We should try running this globally to get a sense of the common assumption that this is going to be very slow.
- Put
-
clean inventory init: removes extra columns and adds ed2 to fates python tool
Latest issues and discussions
- Remove allometry thresholds: started per discussion from last meeting
Calibration progress update (Adrianna, Jessica, Rosie)
- Adrianna: submitted min/max one-at-a-time run. Some of them crashed. Investigating.
- Rosie: spectral element grids. Does anyone know if CLM run on different grids.
- Adrianna: LAI difference
- Charlie: bigleaf and fates have differences in where they are being run for antarctic, greenland, and some other minor area. FATES is nan in the ice sheets.
- Charlie:
fates_max_nocomppft_landuse_vector
discussion: we need to manage that we may have more pfts than we have patches available for a given landuse type. This new parameter would allow user to control this.- We define a maximum number of patches by landuse type. The new parameter would tell fates how many of the top pfts to prioritize to meet this limit.
- Upcoming FATES meetings (5 min)
- Weekly synthesis of the PR Status Board (5-10 min)
- Latest issues (5-10 min)
- Calibration progress update (5-10 min)
- Walk-on topics
- Meetings, conferences, seminars, etc.
- FATES modeling meeting: 26 October, Bharat Sharma, ELM-FATES at ORNL and Duke FACE sites
- Machine outages
- Integrated
- New
-
ERS fix (1098): needs review, fast track fix. Ask Matvey/Mariana to confirm?
-
ERS fix API update (2199): updates
ed_update_site
call, to pass a restart flag- Ryan wants to use this API update to add the first test that uses an initiliazation file to provide structure
-
ERS fix API update (2199): updates
-
param I/O (1096): Is John ready for review?
- Adrianna: Yes. This is coming in a first in a series of changes.
-
ERS fix (1098): needs review, fast track fix. Ask Matvey/Mariana to confirm?
- Updated
-
Two-stream (1036): needs review
- Reach out to Anthony
-
tree pfts update (1093): under review
- Ryan: should we notify the community?
- Aside: Rosie suggested that we add an agenda item once a monthly to pull together items for our "fates digest" monthly email. This would be a good candidate for this.
- This was a previous discussion that we had regarding how to update fates users of software updates
-
LUH2 v1 (1040): reverted back to original
terminate_patches
method - cross-grid seed (1005): API-side test changes need update per CTSM#2151
-
Two-stream (1036): needs review
Latest issues and discussions
- Carbon Starvation and termination (1095)
-
Stature growth truncation error (1097)
- Jessie to start a discussion on what caps we should remove
- [Guardrails (https://github.com/NGEET/fates/issues/1095)]
- Link this to the aforementioned discussion about removing caps
Calibration progress update (Adrianna, Jessica, Rosie)
- Adrianna: struggling with getting sparse grid to work with NUOPC. Sam Levis helping out.
- Ryan to apply two-stream to another branch to help Adrianna to move forward around the NUOPC issue.
- Rosie: Low LAI in tropics discussion. What are our next steps from here (assuming GPP and carbon use efficiency is in the ballpark)
- Jessie/Rosie: Why doesn't increasing root longevity improve this?
- Charlie: should we try decoupling root/leaf optimization (in trimming)?
- Upcoming FATES meetings (5 min)
- Weekly synthesis of the PR Status Board (5-10 min)
- Latest issues (5-10 min)
- Calibration progress update (5-10 min)
- Walk-on topics
- Meetings, conferences, seminars, etc.
- FATES modeling meeting: Thursday October 12 - Ryan Knox and Adrianna Foster, Two-stream update and ILAMB results
- Machine outages
- New
-
1093 (tree default allometry update)
- Charlie: maybe we need to be running with more patches to to smooth out
- Consensus is that this should become the default param values (based on data, no worse than existing set)
- Erik: does FATES plan on holding multiple parameter files aside from the default on main?
- Consensus seems to be our goal of building param files on the fly would cover regional or run-mode specific variations of the param files
-
1093 (tree default allometry update)
- Updated
- 1091 (hite rename): testing in progress
- CTSM#2168 (gru product initialization): testing
- 1005 (seed dispersal): CTSM-side reviews completed. Greg to make requested changes and test aux_clm.
- 1040 (land use v1): working on maxpatch issue
Latest issues and discussions
- Updates
- SurfaceAlbedo() in both CLM and ELM appear to be running unnecessary code that is not consistent with FATES (issue forthcoming): https://github.com/ESCOMP/CTSM/blob/ctsm5.1.dev142/src/biogeophys/SurfaceAlbedoMod.F90#L818-L833
- Ryan: Should we bump up in prioritization the ability to build the param file on the fly?
- Issue CTSM#2126 documents and the potential implementation steps
- Ryan to set up a break out meeting to discuss this
- Ryan: Should we bump up in prioritization the ability to build the param file on the fly?
- 1051 (nocomp restart fails ers)
- Re-evaluate urgency of fixing: https://github.com/NGEET/fates/issues/261
- SurfaceAlbedo() in both CLM and ELM appear to be running unnecessary code that is not consistent with FATES (issue forthcoming): https://github.com/ESCOMP/CTSM/blob/ctsm5.1.dev142/src/biogeophys/SurfaceAlbedoMod.F90#L818-L833
Calibration progress update (Adrianna, Jessica, Rosie)
- See above PR 1093 discussion
- Discussion of next steps wrt turnover times
- Rosie: individual-based version of fates? Ratju working on detailed fire model and curious about if this is a possibility with a future fates version. How difficult would this be?
- Charlie: what would they do about spatial ambiguity?
- Marcos: having an individual based model option would be helpful in implementing new models in fates
- Upcoming FATES meetings (5 min)
- Weekly synthesis of the PR Status Board (5-10 min)
- Latest issues (5-10 min)
- Calibration progress update (5-10 min)
- Walk-on topics
- Meetings, conferences, seminars, etc.
- FATES modeling meeting: Thursday October 12 - Ryan Knox and Adrianna Foster, Two-stream update and ILAMB results
- Jessie and Greg starting up regular meetings for the upcoming fates tutorial
- Machine outages
- New
- 1091 (hite rename): to be tested
- Two PR updates to fates-docs integrated. The
hite
rename PR waiting on PR 1091 before integration.
- Integrated
- Updated
- 1005 (seed dispersal): Tests added to ctsm code along with expected failure issue. Erik and Adrianna to review ctsm-side.
- 1040 (land use v1): Ryan reviewed
Latest issues and discussions
- New
- Updated
- 1051 (nocomp restart fails ers): Mariana and Ryan investigating in parallel. Next meeting to discuss is Fri Oct 13.
Calibration progress update (Adrianna, Jessica, Rosie)
- Jessie: has a boreal forest that looks good now. Currently has a run spinning up that has good ILAMB results so far. Can share results next week.
- How do we decide what species are representative of a particular pft?
- Looking at extra tropical trees, it suggests we may want more pfts (e.g. late-succesional deciduous, early-succesional deciduous).
- As first pass, update since this is an improvement (provinence-wise and results-wise)
- Potential future work: Weighting by abundance or iding species that are representative for the given pft?
- How do we decide what species are representative of a particular pft?
- Adrianna: kicking off one at a time min/max case with two-stream branch.
- Upcoming FATES meetings (5 min)
- Weekly synthesis of the PR Status Board (5-10 min)
- Latest issues (5-10 min)
- Calibration progress update (5-10 min)
- Walk-on topics
- Meetings, conferences, seminars, etc.
- FATES modeling meeting: Thursday September 28 - Chaopeng Shen and Doaa Aboelyazeed
- Machine outages
- Integrated
- fates-docs integrations and configuration yaml file
- Updated
- Greg to close out ctsm-side seed dispersal updates for Adrianna to review
- Ryan to conduct review of LandUse v1
- Jessie's PRs to be tested this week
- Marcos getting back to review of 2-stream
Latest issues and discussions
- New
- 1087 (rework last ditch effort to fuse very small patches)
-
1088 (fire trace gas emissions)
- Rosie comes down on the side that fates should own the emission parameters since they are per-pft
- This and BVOCs are on a priority list for NorESM, which is what prompted Rosie to review and create the issue
- 1089 (seed dispersal not b4b during pe layout)
- Updated
- 1051 (nocomp restart fails ers): Greg, Rosie, Ryan, and Mariana to meet Friday to talk about ERS issue
Calibration progress update (Adrianna, Jessica, Rosie)
- Jessie: Getting boreal forest with new parameter file, too much boreal now
- Also tested BAD allometric updates
- Seeing carbon balance error 42 years into run
- Rosie: What needs to be done to bring ctsm-fates nutrients up to parity with elm-fates?
- Ryan: Root uptake and acquisition
- hoping for someone on the ctsm-side to this as a way to develop familiarity and experience with ctsm-fates nutrients
- Rosie to look at making a ctsm issue if one does not already exist
- Ryan: Root uptake and acquisition
- Check-in on format of this meeting
- Upcoming FATES meetings (5 min)
- Weekly synthesis of the PR Status Board (5-10 min)
- Latest issues (5-10 min)
- Calibration progress update (5-10 min)
- Walk-on topics
- General balance of time between reporting and discussing
- Adrianna: all for optimizing meeting time and reducing total hours.
- Jennifer: Thinking about wants from an E3SM liaison standpoint.
- Maybe including an ELM update for FATES team?
- Marcos: as we bring new stuff on, rotate things across weeks as opposed to trying to put into a single meeting each week
- How to format CPU going forward?
- Jennifer: would like to keep there as an optional check-in.
- PR Board Status plan to optimize
- How to create opportunities for everyone to guide topics (more open forum time, check-in at beginning, query email?)
- Should we have different formats? Long-term planning meeting once a month?
-
Meetings, conferences, seminars, etc.
- ILMF shareable modules webinar: Thursday September 21, 08:00 AM PDT (15:00 UTC)
-
FATES modeling meeting: Thursday September 28 - Chaopeng Shen and Doaa Aboelyazeed
- Adrianna will reach out to solicit a talk in late Oct/early Nov on
- Ryan and Adrianna to team up on a two-stream talk for October
- Ask Rosie to reach out to Eva or Lasse about presenting Land Site Platform talk
-
Machine outages
- Total number - 14 open - 6 stale WIPs - 4 are active but not ready (2Stream,Site-refactor,rad unit tests,Ryan’s gnu ctsm debug fix) - 4 are moving through integration pipeline ( Jessie’s no-comp error fix, luh2 data tool, LU v1, and seed dispersal)
- Big items - LUH2, seed dispersal, 2-stream, unit tests
- Ryan and Adrianna to meet to discuss 2-stream unit test
- Refactors - site refactors
- Bug Fixes - Jessie no comp init fix
- Order - Jessie’s, Cross grid and LUH2
- Big items - LUH2, seed dispersal, 2-stream, unit tests
- Author feedback needs or a hold-up
- Reviewer feedback needs hold-up - none
- Bottlenecks - none
- Long in queue - site refactor?
- Adrianna is planning on closing this out and create a new one in the future. LUH2 and seed dispersal creates lots of conflicts and expectation is that it will likely be easier to start anew.
- Who needs help?
- Updated
- 1005 (seed dispersal): on hold due to ERP issue
- 1040 (v1 landuse): patch fusion during termination handling update
- 1072 (large scale issue fix): to be tested
Latest issues and discussions
- Issues
Calibration progress update (Adrianna, Jessica, Rosie)
- Jessie: seeing issues with boreal trees. Using Rosie's parameter file
- Arianna: latest ILAMB updates with two stream. Encouraging results. Albedo bias is better. Discussion of next steps. Medlyn parameter update. Use atkins.
- Jennifer to create PR with daylight parameter code.
- [ ]
- Upcoming FATES meetings (5 min)
- Weekly synthesis of the PR Status Board (5-10 min)
- Land Use static file import (10 min)
- Latest issues (5-10 min)
- Calibration progress update (5-10 min)
- Walk-on topics
-
Meetings, conferences, seminars, etc.
-
FATES modeling meeting:
- Thursday September 14 - Calibration update
- Greg to check in with Rosie Tuesday evening
- Thursday September 28 - Chaopeng Shen and Doaa Aboelyazeed
- Thursday September 14 - Calibration update
- ILMF shareable modules webinar: Thursday September 21, 08:00 AM PDT (15:00 UTC)
-
FATES modeling meeting:
-
Machine outages
Landuse x PFT static file discussion (Erik, Greg, Charlie)
- Discussed potential switch to streams infrastructure usage in the long term. For now, use simple scheme to allow fates Land Use v2 development to move forward.
- Integrated
-
1069 (PR template)
- Github task list application installed as well. This requires all checkboxes to be filled to merge PR.
-
1069 (PR template)
- Updated
- 1005 (seed dispersal): ERP fix still underway.
- 1072 (large scale issue fix): under review
Latest issues and discussions
- Issues
- Discussions
Calibration progress update (Adrianna, Jessica, Rosie)
- Deferred
- None
No meeting due to federal holiday
- Upcoming FATES meetings (5 min)
- Weekly synthesis of the PR Status Board (5-10 min)
- Latest issues (5-10 min)
- E3SM restart issue
- FATES paleo
- 'Outside' contributions to CTSM-FATES API
- Calibration progress update (5-10 min)
- Walk-on topics
- Meetings
- FATES modeling meeting: Thursday August 31 - Zachary Robbins and Chonggang Xu
- Conferences
- Machine outages
- Integrated
- Updated
- 1005 (seed dispersal): ERS fix in progress
- New
- 1069 (PR template): reworks checklist to ask user for fates documentation PR or issue. Addresses 1068.
- 1072 (large tree init fix): Addresses 1071
Latest issues and discussions
- Discussions
- E3SM commit 0777e39115 on April 11th onwards introduces EAMXX. From that point onwards Jessie was finding restarts were not working.
- Removing
./xmlchange DOUT_S=TRUE
(short term archiving enabled) from script solved the restart issue
- Removing
- Sophia Macarewich running CLM-FATES Amazon single site runs in three different scenarios including 1850, last glacial maximum and paleo
- Greg suggested this might be good for a future FATES modeling meeting topic. Adrianna suggests maybe November. She is providing support to Sophia.
- Charlie noted that Jenny has paleo background and might be a good person to connect with.
- Discussion about having John Alex contribute improvements through refactors for FATES API
- Team is comfortable with John contributing
Calibration progress update (Adrianna, Jessica, Rosie)
- No major updates
- No topics
- Upcoming FATES meetings (5 min)
- Weekly synthesis of the PR Status Board (5-10 min)
- Latest issues (5-10 min)
- Calibration progress update (5-10 min)
- Walk-on topics
- Meetings
- FATES modeling meeting: Thursday August 31 - Zachary Robbins and Chonggang Xu
- Conferences
- Machine outages
-
Integrated
-
999 (CLM BGC compatibility)
- Ryan would like to assess impact of reordering of root fraction calculations on the elm-fates results.
- This would ideally be before the elm-side luh2 and seed dispersal pull requests (but after the refactor).
- Ryan would like to assess impact of reordering of root fraction calculations on the elm-fates results.
-
999 (CLM BGC compatibility)
-
Updated
-
E3SM#5849 (refactor) tested and waiting on e3sm approval/integration
- Greg to check in with Gautum and Peter S.
- 1055 (too much canopy): added final regression testing forthcoming
-
1040 (LUH2): updated patch insertion and fusion procedures
- Fusion routine updated to fuse very small patches into largest patch of different landuse type if not possible to merge into same type
- Need to assess if landuse transition rates could create small patches below threshold
- Fusion routine updated to fuse very small patches into largest patch of different landuse type if not possible to merge into same type
-
E3SM#5849 (refactor) tested and waiting on e3sm approval/integration
-
Next tags
- Given BGC-reorder check and refactor merge on e3sm side, move forward with seed dispersal and landuse on ctsm side.
- Find reviewers for #1040. Retest seed dispersal and move this ahead of landuse in the ctsm queue.
- Given BGC-reorder check and refactor merge on e3sm side, move forward with seed dispersal and landuse on ctsm side.
Latest issues and discussions
- 1062 (Inventory init file cleanup)
- 1063 (sp mode leaf_c update)
- 1064 (init sp mode cohorts with hgt_min)
Calibration progress update (Adrianna, Jessica, Rosie)
- Adrianna: Ryan to call into calibration meeting this week, to talk about getting 2-stream ready for PPE.
- Adrianna: Sparse grid for dominant pfts. What threshold should we use for "dominant?"
- Greg to create an issue for assessment of sp mode area tolerances
- Rosie noted that the genesis for this was due to needing canopy to fully fill error (or we see downstread rad errors).
- Perhaps "canopy shyness" gap mechanism would help with rad error? Ryan agrees that 2-stream will help enable this if desired.
- Rosie noted that the genesis for this was due to needing canopy to fully fill error (or we see downstread rad errors).
- Upcoming FATES meetings (5 min)
- Weekly synthesis of the PR Status Board (5-10 min)
- Latest issues (5-10 min)
- Calibration progress update (5-10 min)
- Walk-on topics
-
Meetings
- FATES modeling meeting: Thursday August 17 - TBD
-
Conferences
-
Machine outages
-
Integrated
-
1024 (refactor) & CTSM#2000 (refactor)
- E3SM#5849 (refactor) tested and waiting on e3sm approval/integration
-
1024 (refactor) & CTSM#2000 (refactor)
-
Updated
- 999 (CLM BGC compatibility): ERS tests fixed. Allocation/associate statement fix clarification from Bill. Next ctsm tag to be integrated.
-
1040 (LUH2): deconflicted against latest fates tags. Troubleshooting a disturbance rate summation issue.
- HLM-side api updates have been deconflicted against refactor as well: CTSM#2076 (clm-fates luh2), E3SM#5760 (elm-fates luh2)
Latest issues and discussions
- Issues: nothing new/updated
- Discussions
Calibration progress update (Adrianna, Jessica, Rosie)
- Adrianna: CLM5 PPE
-
fff
has large impact on FATES GPP - Working on getting relative % change in parameter affect (as opposed to absolute change)
- Charlie: looking at spatial distribution would be interesting. He suspects it'll be split between snow/no snow areas
- Next steps:
- Bring in Ryan's two-stream branch to see the affect on the albedo bias
- Another PPE with just fates params and then maybe clm + fates
- Latin hyper-cube set (on for albedo tuning and then decision on next calibration steps)
- Charlie: What about elm/clm param divergence? Maybe to investigate: calibration of fates with just fates versus fates + hlm params?
-
- Jessie: ELM-FATES training next summer
- Geared towards ForestGeo community and potentially run at their sites. Inventory init since no met data available.
- Still need to talk with Stuart to see if we're going to hold it concurrently with existing planned ForestGeo workshop or be standalone.
- Adrianna: could we improve inventory initialization file (considering compatibility with ED)
- Upcoming FATES meetings (5 min)
- Weekly synthesis of the PR Status Board (5-10 min)
- Latest issues (5-10 min)
- Calibration progress update (5-10 min)
- FATES modeling meeting notes discussion (Ryan)
- Walk-on topics
-
Meetings
-
FATES modeling meeting: Thursday August 17 - TBD
- Rosie: Chaopeng Shen interested in presenting (https://bg.copernicus.org/articles/20/2671/2023/bg-20-2671-2023.pdf)
-
FATES modeling meeting: Thursday August 17 - TBD
-
Conference
-
Ecological Society of America (ESA): This week
- Jennifer and Marcos attending
-
Ecological Society of America (ESA): This week
-
Machine outages
- CISL: None scheduled
- NERSC: None sceduled
- OLCF: None scheduled
-
Integrated
-
New
- 1056 (longname fix): Reviewed.
-
Updated
- 1024 (fates refactor): Global SP mode tests are not b4b against baseline. Investigating.
- 999 (CLM BGC compatibility): Exact restart tests are not B4B. Investigating.
- 1055 (too much canopy): Needs discussion about changing tolerances
Latest issues and discussions
- Rosie: Status of NoComp ERS issue?
- Greg: still working on this. Has fallen down priority behind BGC and refactor PRs.
Calibration progress update (Adrianna, Jessica, Rosie)
- Adrianna: creating parameter files to be able to run PPE with FATES
- Charlie: Pft areas by landuse type file. Have this as a separate file our put into the LUH2 concatenated file.
- Could allow us to decouple somewhat from the surface data set. Would allow us to reword nocomp mode.
- Charlie to add Chongang on this meeting.
- Adrianna: there is a proposal to use ChatGPT to refactor clm fortran code to another language to faciliate ML work.
- Upcoming FATES meetings (5 min)
- Weekly synthesis of the PR Status Board (5-10 min)
- Latest issues (5-10 min)
- Calibration progress update (5-10 min)
- Walk-on topics
-
Meetings
- FATES modeling meeting: Thursday August 3 - Office Hours
-
Conference
-
Ecological Society of America (ESA): August 6 - August 11, 2023
- Jennifer and Marcos attending
-
Ecological Society of America (ESA): August 6 - August 11, 2023
-
Machine outages
- CISL: Cheyenne, July 31-August 4
- NERSC: Perlmutter, July 31 (short)
- OLCF: None planned
-
New
- 1055 (too much canopy): fixes #845. To be reviewed.
-
Updated
- 1032 (LUH2 data tool): Sam L reviewed. Minor additions requested, but approved.
- 1024 (fates refactor): SP mode tests are not b4b against baseline. Investigating.
- 999 (CLM BGC compatibility): Exact restart tests are not B4B. Investigating.
Latest issues and discussions
- No new issues or updates
Calibration progress update (Adrianna, Jessica, Rosie)
- Deferred
- Discussing Ryan CNP reviewer response and diurnal eddy flux comparison findings.
- Upcoming FATES meetings (5 min)
- Weekly synthesis of the PR Status Board (5-10 min)
- Latest issues (5-10 min)
- Calibration progress update (5-10 min)
- Walk-on topics
-
FATES modeling meeting: Thursday August 3 - TBD
- We will reach out to Zachary ang Chongang. They both presented RFA updates at the NGEE-T all hands today.
Integrated
- fates-docs#35 (crown damage documentation), E3SM#5802 (API25.5), 1053 (C3/C4 Ryan 1991 fix), 1021 (Tree recruitment)
Updated
-
1005 (seed dispersal): fixes to
SeedUpdate
and minor refactor. Need to deconflict E3SM#5555. - CTSM#1959 (FATES BGC update): two unresolved issues and then testing.
-
1032 (LUH2 data tool):
- Refactored to use
decode_times=False
-
ncks -v
failing onsecmb
andrange
. Possible insight into runtime error seen?- Possibly due to corrupted download? If not, Charlie suggests try converting to a different NetCDF file type.
- Refactored to use
Latest issues and discussions
Closed
-
fates-docs#36 (master to main)
- Reminder: See FATES discussion thread about how to update local repo after
master
tomain
update
- Reminder: See FATES discussion thread about how to update local repo after
Updated
- 845 (sp mode too much canopy): oldest/youngest patch area update correction seems to fix this. PR forthcoming.
Calibration progress update (Adrianna, Jessica, Rosie)
- Adrianna: Still working towards PPE run to assess impact of clm parameters on fates. Working on making PPE scripts a little more user friendly.
- If PPE is cheap, we could run with Ryan and Atkins each.
-
init_patches
refactor, bareground, and natpft/landuse mapping discussion- Charlie: created a
bc_in
variable for bareground fraction to remove indexing pfts by zero. He will put this into a refactor PR.
- Charlie: created a
- Upcoming FATES meetings (5 min)
- Weekly synthesis of the PR Status Board (5-10 min)
- Latest issues (5-10 min)
- Global variable refactor (5-10 min)
- Quarterly Highlights Request for E3SM-FATES (5 min)
- Calibration progress update (5-10 min)
- Walk-on topics
- FATES modeling meeting: Thursday July 21 - Adam Hanbury-Brown
- Distinguished Scientist Seminar Series, Sonia I. Seneviratne: Thursday, July 20th, 2023, 10-11:30 AM PDT
- Jennifer hosting
Integrated
New
- fates-docs#35 (crown damage documentation): under review, minor edits forthcoming
Latest issues and discussions
New
-
1051 (nocomp restart fails ers): nocomp failing with
Lm3
, but notLd90
- fates-docs#36 (master to main): master to main
Updated
-
845 (sp mode too much canopy): patch fusion and resizing during init questions
- Ryan: possibly zero LAI due to high resolution or crop?
- Try running single point at a failing lat-lon
- Could this be problematic based on how hlm handles single-point averaging for a specific lat-lon?
- Adrianna: we could also take a look at the failing lat-lon to see what the lai is at those cells
- mismatch between patch area and lai area?
Local branch: https://github.com/adrifoster/fates/tree/hlmparams_refactor/api
- Greg: we should point to a reference on
get
/set
being recommended object oriented practice that we can point to for the developer's style guide section
This request came from NGEE-Tropics project management (Sandy, Daniel)
- Ryan and Jen to crib from Charlie's E3SM presentation
- Highlighting our communication: E3SM Phase 3 meeting and E3SM-Infrastructure regular meeting participation
Calibration progress update (Adrianna, Jessica, Rosie)
- Jessie: vertical scaling scheme cases are running. Results forthcoming.
- Caught temperature sensitivity error. Give Rosie and Charlie a heads up on this. See PR#1053 for fix.
- Adrianna: pulling together the scripts for PPE. Goal is to generate clm parameters that affect fates-sp mode output.
- Discussion of elm vs clm parameter differences and potential impact
- Greg: schedule fates documentation triage meeting
- Greg: ask Charlie about fates-largescale-tools
- Jessie: new repository for fates tools to do large regional. Will likely have some overlap with calibration repository.
- Upcoming FATES meetings (5 min)
- Weekly synthesis of the PR Status Board (5-10 min)
- Latest issues (5-10 min)
- Calibration progress update (5-10 min)
- Walk-on topics
- FATES modeling meeting: Thursday July 21 - Adam Hanbury-Brown
- Distinguished Scientist Seminar Series, Sonia I. Seneviratne: Thursday, July 20th, 2023, 10-11:30 AM PDT
- Jennifer hosting
Integrated
- CTSM#2009 (paramfile/test updates): associated with pr958, tagged as ctsm5.1.dev130
Updated
- 999 (CLM BGC compatibility): testing complete. Erik conducting final ctsm-side review
-
1021 (Tree recruitment):
ncan
indexing issue fixed. Allocation tailoring forthcoming. - 1041 (EMA NPP fix): land developer debug tests all passed on summit. Testing on cheyenne and perlmutter forthcoming.
New
-
fates-doc#34 (reduced complexity modes): merged during the meeting.
- Related: Created new issue in fates-users-guide repo to add section explaining how to update the documentation.
- Related: Marcos and Jessie to work on add new sections.
Latest issues and discussions
New
- CTSMS#2047 (0.01 "magic" number): fates api is part of this
Calibration progress update (Adrianna, Jessica, Rosie)
- No major items to report
- Adrianna working on trying to get lai streams to work with fates (part of ctsm bug squashing week)
- Discussion of https://github.com/ESCOMP/CTSM/issues/1505
- Upcoming FATES meetings (5 min)
- Weekly synthesis of the PR Status Board (5-10 min)
- Latest issues (5-10 min)
- Calibration progress update (5-10 min)
- LUH2 update (5 min)
- Walk-on topics
-
FATES modeling meeting: Thursday July 06 - TBD
- Global carbon budget meeting is this day
- CTSM bug squash week: July 10-14
- Distinguished Scientist Seminar Series, Sonia I. Seneviratne: Thursday, July 20th, 2023, 10-11:30 AM PDT
- Jennifer hosting
Integrated
-
958 (drought deciduous), 1043 (tooling typo fix)
- CTSM#2009 (paramfile/test updates): associated with pr958, awaiting ctsm-side integration
Updated
- 999 (CLM BGC compatibility): Ryan testing
-
1021 (Tree recruitment): Deconflicted. Initial testing revealed initialization error. Greg working on solution.
- Look to see where we could add checks against regeneration_model to avoid allocation when not needed.
- 1041 (EMA NPP fix): Needs debug mode test. Land developer tests are all non-debug.
Latest issues and discussions
Closed
- 1046 (restart indexing): via #958 above
New
- 1047 (unit-longname discrepency): minor comment update. Incorporate into BGC or Tree recruitment
Calibration progress update (Adrianna, Jessica, Rosie)
- Jessie: Vertical scaling of respiration progress. Still need to look into low LAI issue (trimming, uptake)
- Rosie: freezing tolerance testing
- Rosie: adding list of biases to correct in minimum shippable fates document
- Rosie: will share current parameterization file that addresses many of these
- Ryan: should we consider adding a forest size structure approximation in SP mode (for calibration and otherwise)?
- Improving biomass representation and rooting depth for SP+Hydro.
- Charlie: this will be useful when Gordon's multilayer canopy atmospheric model comes in to clm.
- Upcoming FATES meetings (5 min)
- Weekly synthesis of the PR Status Board (5-10 min)
- Latest issues (5-10 min)
- Calibration progress update (5-10 min)
- Two-stream update (5 min)
- LUH2 update (5 min)
- Walk-on topics
-
FATES modeling meeting: Thursday July 06 - TBD
- Global carbon budget meeting is this day
- E3SM 2023 All-Hands: June 26-28, Denver Airport Convention Center
- CTSM bug squash week: July 10-14
Updates
- 958 (drought deciduous): Ryan ID'd possible ERS fix in issue #1046 (see below)
- 1021 (Tree recruitment): Testing forthcoming. Marcos to re-review Adam's updates.
- 1032 (LUH2 data tool): Testing on cheyenne forthcoming. Discussing integration into ctsm5.2 data pipeline.
- 1041 (EMA NPP fix): Initial testing looks good. This needs an ELM-side parameter file update. Suggestion to update E3SM#5699 to update param file and point to this fix is pending feedback.
New
- 1043 (tooling typo fix): Fixes typo in default tool arguments section
- 1045 (site refactor): Moves fates sites type into it's own module
Latest issues and discussions
Issues
- 1046 (restart indexing): potential fix to ERS issue in #958
- 1042 (site-level units/descriptions)
Discussions
Calibration progress update (Adrianna, Jessica, Rosie)
- Adrianna: Met with Katie Dagon and Daniel Kennedy to discuss PPE division of labor. Can we just split up by fates/clm or are there clm params fates relies on?
- CLM soil and ice albedo parameters would be important for fates PPE
- Note PPE will not be run on latest fates commit
- Rosie: calibration update
- Rosie will stop here current results given that Ryan's work and Adrianna's PPE are forthcoming.
- Carbon use efficiency (NPP/GPP) is very high in temperate and boreal post GPP calibration (using Ryan et al resp)
- LAI is looking low (consistent with what Jessie is seeing as well in the tropics)
- Charlie: Jessie to take up work synthesizing GEM data in conjuntction with Yadvinder Malhi to help solve this problem
- no update
- CTSM-FATES API ported from ELM-FATES successfully last week.
- LUH2 data tooling: discussing integration into ctsm5.2 data workflow
- Refactoring may be necessary for integration. Testing on cheyenne is next step.
- CTSM software meeting folks agreed to add xesmf to ctsm_pylib (see 6-22-2023 meeting notes)
- Adrianna: CESM workshop discussion of minimum shippable fates
- Check on updating fates to use LAI streams
- Getting 2-stream in for this is high priority
No meeting due to federal holiday
- Upcoming FATES meetings (5 min)
- Weekly synthesis of the PR Status Board (5-10 min)
- Latest issues (5-10 min)
- Calibration progress update (5-10 min)
- LUH2 update (5 min)
- Walk-on topics
-
FATES modeling meeting: Thursday June 22 - TBD
- Charlie to ask Zachary about presenting
- Charlie suggested making the fates_developers google group the distribution list for the software meeting
- CESM 2023 Workshop: June 12-14
- E3SM 2023 All-Hands: June 26-28, Denver Airport Convention Center
- CTSM bug squash week: July 10-14
Integrated
New
- 1041 (EMA NPP fix)
- 1040 (LUH2): coordinated e3sm PR forthcoming
Updates
- 958 (drought deciduous): ERS fix invesitgation still ongoing
- 1021 (Tree recruitment): Reviews underway. Ryan to submit PR to PR.
Latest issues and discussions
Issues
- 1039 (uninitialized ema_npp)
- 1014 (storage maintenance respiration): two followup questions added from Marcos
Discussions
Calibration progress update (Adrianna, Jessica, Rosie)
- Jessie: report back from demographic benchmarking model intercomparison project (D-BEN)
- Jessie: continuing to review LAI bias
- There has been an issue with the model crashing 8 months into a global run due to BalanceCheck
- Issue appears to have been a problem with
max_patch_per_col
(hlm-side). Updating this withmaxpatch_total
alleviates the issue.
- Issue appears to have been a problem with
- Land Sites Platform paper published: https://onlinelibrary.wiley.com/doi/10.1111/gcb.16808
- Roughness length issue:
- Rosie noted in an email that roughness length looks like its an order of mag different
- Jessie: fates is passing roughness length back to elm, but it might not be getting updated
- Might have been using bareground version of this (
z0mg
)
- Might have been using bareground version of this (
- Upcoming FATES meetings (5 min)
- Weekly synthesis of the PR Status Board (5-10 min)
- Latest issues (5-10 min)
- Calibration progress update (5-10 min)
- Two-stream progress update (5 min)
- LUH2 update (5 min)
- Walk-on topics
- FATES modeling meeting: Thursday June 8 - Xiulin Gao
- CESM 2023 Workshop: June 12-14
- E3SM 2023 All-Hands: June 26-28, Denver Airport Convention Center
- CTSM bug squash week: July 10-14
- Related: Add Chongang to this software meeting invite. Charlie to send.
Integrated
New
- 1034 (SolarRad Unit test): WIP. Refactor PR to come in first.
- 1036 (2-stream): Drafted
Updates
- 1032 (LUH2 data tool): Potetial cache issues only on specific machine/os?
- 958 (drought deciduous): Ryan to help look into non-b4b restarts
- 1008 (biomass partitioning): Pushing this to head of the queue
- 1005 (seed dispersal): Maybe push this ahead of #958, behind #1008
Latest issues and discussions
- No new issues of note
Calibration progress update (Adrianna, Jessica, Rosie)
- Calibrating for NEON sites (Adrianna)
- Questions: use default pft or specific NEON set? Joint calibration across all sites or individual site calibration?
- Marcos and Adrianna have been talking about creating specific pfts for the site similar to his work in tropics
- Making the LAI on surface data sets match site data
Two-stream progress update (Ryan)
- Previewed draft PR #1036
- Discussion of history output changes and unit test pr
- Run testing on Perlmutter on going
- Greg investigating indexing by age issue. Caused by disturbance rate being larger than one, but not sure why this isn't being scaled as expected.
- Publishing discussion: model description paper suggestion (Marcos)
- Convert technote?
No meeting due to federal holiday
- Upcoming FATES meetings (5 min)
- Weekly synthesis of the PR Status Board (5-10 min)
- Latest issues (5-10 min)
- Calibration progress update (5-10 min)
- Two-stream progress update (5 min)
- LUH2 update (5 min)
- Walk-on topics
- FATES modeling meeting: Thursday May 25 - Julia Maria Chacon-Labella
- JuliaHub call (Greg & Ryan): Friday May 26, 8:30 AM PDT
- JuliaHub reached out to Greg. Others welcome to attend if so desired.
-
CESM 2023 Workshop: July 12-14
- Charlie to attend in person
-
E3SM 2023 All-Hands: June 26-28, Denver Airport Convention Center
- Ryan, Charlie, Greg will be attending virtually. Jessica will be on site.
- CTSM bug squash week: July 10-14
Integrated
- E3SM#5604 (NP refactor): awaiting one more approval before integration
Updates
-
958 (drought deciduous): testing underway. Looking into non-b4b restarts.
- CTSM#2009 (paramfile/test updates): associated with pr958
New
- 1032 (luh2 data tooling): tooling to curate luh2 data for fates use
Latest issues and discussions
New
- 1029 (
init_cohort
andrecruitment
) - 1030 (refactor site type)
- 1031 (Parallel regridding)
- 1033 (Automated tool testing)
Calibration progress update (Adrianna, Jessica, Rosie)
- Adrianna: calibration cascade at NEON sites. She is working on updating surface data to have LAI reflect actual observations at the site.
- Marcos and Adrianna to talk about common approaches to this challenge. Comparison of ML techniques.
Two-stream progress update (Ryan)
- Currently grappling with two-stream resulting in lower than expected LAI. Presented diagnostic case without trimming and N-scaling of dark respiration.
- Potential: could we use observed data to help benchmark if the results are actually as expected (i.e. instead of comparing against Norman). Ryan and Adrianna to look into data at BCI and NEON, respectively. Marcos noted that Manaus has some old data.
- LAPACK seems to be pulling its weight wrt the 2-stream matrix inversions: speed difference between Norman and 2-stream doesn't seem very apparent.
- See LUH2 data tool PR above. Greg working on discrepency handling between where LUH2 data says there isn't data, but HLM says there is.
- Jessica has an intern coming in that will be partnering with Margeret Torn's intern to run fates at Blogit (sp?) Experimental Site
- Adrianna and Marocs' postdoc will try and meet up at the NEON meeting
- Upcoming FATES meetings (5 min)
- Weekly synthesis of the PR Status Board (5-10 min)
- Latest issues (5-10 min)
- Calibration progress update (5-10 min)
- Two-stream progress update (5 min)
- Walk-on topics
- FATES modeling meeting: May 25 - Julia Maria Chacon-Labella
- ESS-PI meeting: May 16-17
- E3SM All-Hands: June 26-28
Integrated
Updates
- 1024 (fates refactor): review ongoing
- 958 (drought deciduous): parameter patch xml file added. Associated parameter tool update forthcoming.
- E3SM#5604 (NP refactor): awaiting one more approval before integration
Latest issues and discussions
Calibration progress update (Adrianna, Jessica, Rosie)
Two-stream progress update (Ryan)
- Upcoming FATES meetings (5 min)
- Weekly synthesis of the PR Status Board (5-10 min)
- Latest issues (5-10 min)
- Calibration progress update (5-10 min)
- Two-stream progress update (1 min)
- LUH2 update (5 min)
- Walk-on topics
-
FATES modeling meeting: May 11 - TBD
- Consensus is to cancel this.
- ESS-PI meeting: May 16-17
New
-
1024 (fates refactor): priortize this to avoid conflicts
- Plan is to integrate all prior PRs before incorporating this one.
- Charlie previewed the conflicts: they are a few hours of work, but not daunting Updates
- E3SM#5604 (NP refactor): awaiting one more approval before integration
- 1008 (biomass partitioning): Rosie to review
Latest issues and discussions
New
- Photosynthesis being applied on mean pft-layer-level leaf areas, not cohort-actual Link
- Consensus is to move forward with Ryan's idea
- Need better protections for negative storage termination flux and negative germination flux to the litter pool Link
- Ryan to continue to investigate this and start a github discussion
Calibration progress update (Adrianna, Jessica, Rosie)
- Rosie: Improvements made in calibrating out the bias compared against CLM5.1. Working on the ILAMB comparison currently against CERES data. Started investigating GPP & ET calibration.
- Jennifer has a branch with daylength addition to FATES that ELM/CLM both have. Rosie noted this is something she had to remove from CLM to aid comparison.
- Which to work on first? What is the lowest hanging fruit? Daylength? Acclimation?
Two-stream progress update (Ryan)
- Deferred
- Deferred
- Upcoming FATES meetings (5 min)
- Weekly synthesis of the PR Status Board (5-10 min)
- Latest issues (5-10 min)
- Calibration progress update (5-10 min)
- Two-stream progress update (5 min)
- LUH2 update (5 min)
- Walk-on topics
-
FATES modeling meeting: May 11 - TBD
- We will decide on office hours or cancellation next week
- ESS-PI meeting: May 16-17
Updates
- 958 (drought deciduous): deconflicted and reviewed. To be tested.
- 1022 (fates_rad_leaf_xl): minor changes requested
-
1005 (seed dispersal)
- Greg and Yanlan responded to review comments. Issue/discussions to be generated based on suggestions.
- 1008 (biomass partitioning): Adam updated per suggestions.
- E3SM#5604 (NP refactor): Approved. Awaiting integration.
Latest issues and discussions
- No new or updated discussions or issues
Calibration progress update (Adrianna, Jessica, Rosie)
- Jessica: Working on carbon budget tracking through all organs. To be a PR soon.
- Assessing what TRENDY output requirements are already incorporated into fates.
- Adrianna: Continuing to work on unit testing. Continuing ahead with short term solution to read in parameter values. Not pursuing longer term fates pio option at this time.
- IO discussion recap
Two-stream progress update (Ryan)
- Marcos: discussion of clumping index methodology
- Current method is each cohort as its own scattering element
- Greg: SCORPIO divergence from PIO on variable dimension check is causing a read crash. Check in with the commit author about the definition of "extra dimensions".
- Related to Adrianna's IO exploration: ask about general history of divergence between PIO and SCORPIO
- Upcoming FATES meetings (5 min)
- Weekly synthesis of the PR Status Board (5-10 min)
- Latest issues (5-10 min)
- Calibration progress update (5-10 min)
- Two-stream progress update
- FATES history compatibility with MIPs
- Walk-on topics
- FATES modeling meeting: LUH2 updates
- ESS-PI meeting: May 16-17
Updated
- 958 (Deciduous phenology): crown damage overlap review status?
- 1005 (seed dispersal)
- 1008 (biomass partitioning)
- E3SM#5604 (NP refactor)
New
- 1021 (Tree recruitment): needs reviewers
- 1022 (fates_rad_leaf_xl)
- 1023 (Termination mortality): needs reviewers
Latest issues and discussions
Update
- 1014 (bleaf vs bstore_allom call): Charlie checked responded
Calibration progress update (Adrianna, Jessica, Rosie)
Two-stream progress update (Ryan)
FATES history compatibility with MIPs (Jessica)
- Upcoming FATES meetings (5 min)
- Weekly synthesis of the PR Status Board (5-10 min)
- Latest issues (5-10 min)
- Calibration progress update (5-10 min)
- Two-stream progress update
- Walk-on topics
- FATES modeling meeting: LUH2 updates
- ESS-PI meeting: May 16-17
Integrated
Updates
-
958 (Deciduous phenology)
- Elongation factors added as arguments to the allometry. Scientific testing looks very similar.
- Crown damage needed to be integrated. Jessie and Charlie to review.
- Ready to deconflict and regression test soon.
-
CTSM#1959 (FATES BGC update)
- Erik reviewed. Some changes requested. Also recommending review from Will.
- 1008 (biomass partition): Zachary agreed to review. Rosie confirmed review as well.
Latest issues and discussions
Issue
-
1014 (bleaf vs bstore_allom call)
- Discussion held. There will be followup. Marcos likely to take point on pull request.
Discussions
-
1007 (hydro output error): Tested re-removing. This is still an issue (#261).
- Ryan wants to priorize this.
-
1015 (modify param file help): Ryan providing support
- Ryan spun out part of this into a new discussion (#1020) about hydro calibration and noted that he asked for others to suggest additional answers.
- Adrianna: focus will be limited to cohort refactoring to avoid complicating the forthcoming LUH2 PR from Charlie.
- Type bound procedures doesn't increase the memory load appreciably.
- Rosie: albedo bias structure bewtween CERES and GEWEX looks different based on looking at impact of reducing rhonir and rhovis.
- Rosie is going to do some comparison to clm5 biases, but then move on from this due to shift to 2-stream
- Jessie: first round or results submitted to DBEN. Moving on to trying to get the transitions to be represented.
- Presentation slide deck - deffered
- Update: Marcos and Ryan math deriviation are consistent (and match previous efforts).
- Working on figuring out ways to make reductions in the size of necessary calculations.
- Q: is the current trimming method going to be compatible with 2-stream?
- Upcoming meetings (5 min)
- Weekly synthesis of the PR Status Board (5-10 min)
- Latest issues (5-10 min)
- Calibration progress update (5-10 min)
- Walk-on topics
- FATES modeling meeting: Thursday April 13 - Lingcheng Li
- ESS-PI meeting: May 16-17
- Jennifer and Charlie coordinating on a request from Beth Drewniak
Integrated
Updates
- 991 (Restart mem reduce): Ryan made updates and testing locally
- 1008 (biomass partitioning): Requested reviews from Zachary and Rosie
Latest issues and discussions
Discussions
-
1007 (hydro output error): LAI min (
ai_min
) has maybe re-surfaced?- We had tried to remove this, but manually reinstated with PR 739
- Original issue 261
- Greg/Ryan to test removing again.
- 1015 (modify param file help): help requests in using modify_fates_paramfile.py
Issues
-
1011 (Refactor disturbance)
- Charlie to start a new issue regarding the related
_localization
variables
- Charlie to start a new issue regarding the related
- 1014 (bleaf vs bstore_allom call)
Calibration progress update (5-10 min)
- Adrianna: refactoring work in progress. Testing for b4bness as work progresses.
- Question: should we refactor standalone procedures as a method?
- Ryan: what's the impact on memory?
- Greg: suggestion creating a discussion with a table of the function to method conversion
- Question: splitting
NaN
s out of zeroing routines? Yes.
- Question: should we refactor standalone procedures as a method?
- Jessie: FBG simulations running in sparse grid mode. Amsterdam simulations nearly done. Waiting on perlmutter coming back up and more allocation hours.
- Upcoming meetings (5 min)
- Weekly synthesis of the PR Status Board (5-10 min)
- Latest issues (5-10 min)
- Calibration progress update (5-10 min)
- Two-stream update (Ryan)
- Refactoring our core types
- Walk-on topics
-
FATES modeling meeting: Thursday April 13 - TBD
- We came up with a number of potential options. Adrianna confirmed with Jackie that Julia could reschedule.
Updated
- CTSM#1962 (soil moisture): Found nag compiler issue with latest fates tag. See 1009 below.
- 1005 (seed dispersal): fixes added to nearest neighbor indexing
- 958 (drought deciduous): Marcos updating per review suggestions and retesting locally
New
- 1009 (line continuation): Greg adding other b4b issue fixes. Needs to be integrated before CTSM#1962 can be retested.
-
1008 (biomass partitioning): needs reviewers
- Ryan to ask Zachary. Rosie will also review.
- Jennifer noted that this has been discussed from a science perspective in the CA fates meeting.
Latest issues and discussions
No updates
Calibration progress update (5-10 min)
- Jessie is seeing total balance check error cropping up again as was seen in issue 961
- Rosie noted that she has a post-doc working on implementing the CLIMA calibration method
- Adrianna asked if the team was comfortable with refactoring patch and cohort types into their own modules
- This would also include creating methods to the types (e.g.
init
) instead of having seperate subroutines - She will incrementally test to ensure b4b-ness
- Team approves
- This would also include creating methods to the types (e.g.
- Rosie: Caldararu Silvia contacted Rosie to talk about QUINCY model and FATES
Attendence was low so we deferred a number of these items to next week
- Upcoming meetings (5 min)
- Github RSA host key update (1 min)
- [-] Weekly synthesis of the PR Status Board (5-10 min)
- [-] Latest issues (5-10 min)
- [-] Calibration progress update (5-10 min)
- [-] Two-stream update (Ryan)
- Walk-on topics
Github replaced their RSA SSH host key Friday March 24. You may see the following warning:
@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@
@ WARNING: REMOTE HOST IDENTIFICATION HAS CHANGED! @
@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@
IT IS POSSIBLE THAT SOMEONE IS DOING SOMETHING NASTY!
Someone could be eavesdropping on you right now (man-in-the-middle attack)!
It is also possible that a host key has just been changed.
The fingerprint for the RSA key sent by the remote host is
SHA256:uNiVztksCsDhcc0u9e8BujQXVUpKZIDTMczCvj3tD2s.
Please contact your system administrator.
Add correct host key in ~/.ssh/known_hosts to get rid of this message.
Host key for github.com has changed and you have requested strict checking.
Host key verification failed.
See the github blog post for information and fix.
Discussion deferred
- FATES modeling meeting: Thursday March 30 - Carvalhais
Discussion deferred
Integrated
Updated
- 1005 (seed dispersal): xml patch file added. Reviewers found. Testing to start soon.
- CTSM#1962 (soil moisture): ctsm software team ok with integrating as in. To be tested.
Latest issues and discussions
Discussion deferred
-
1007 (hydro output error): LAI min (
ai_min
) has maybe re-surfaced?- We had tried to remove this, but manually reinstated with PR 739
- Original issue 261
Calibration progress update (5-10 min)
Discussion deferred
Discussion deferred
- Ryan to create a new E3SM branch to bring some nutrients updates so others don't have to keep tracking his branch.
- Jessie: Got the elm sparse grid working (on cori) and learned how to make custom sparse grids (Keith Oleson provided the scripts)
- Jennifer: soil organic carbon discussion. She will diagnose and create a new issue/discussion.
- Upcoming meetings (5 min)
- Weekly synthesis of the PR Status Board (5-10 min)
- Latest issues (5-10 min)
- Calibration progress update (5-10 min)
- Walk-on topics
- Integrate active crown fire - Jackie
-
FATES modeling meeting: Thursday March 30 - Carvalhais
- Chacon-Labella reschedule: Jackie to reach out to Julia to reschedule for April.
- Ideas for April meetings?
- Ryan suggested quick 10 minute presentations
- Jessie offered Atkin comparison update
- Adrianna to reach out to Will about NIWOT clm-fates comparisons
- Ryan suggested quick 10 minute presentations
Integrated
Updated
- 984 (acclimation): Testing complete. To be integrated today.
-
991 (SP restart memory): Ryan looking into
allocate_bcin
check necessity - 995 (NoComp large tree init): Discussed Jessie's questions. Consensus is to not add a new parameter.
- 999 (CLM BGC compatibility): Ryan reconciling unexpected ctsm DIFF
- CTSM#1962 (soil moisture init): discussion status?
New
-
1005 (seed dispersal): Yanlan opened recently. Greg to create e3sm-side PR soon.
- Ryan suggested Eva. Marcos recommended Rachel Ward. Jackie suggested Adrianna.
- 998 (issue triage): small PR to address recent issue triage
Latest issues and discussions
New
- 1000 (coeff of PAR question): converted to discussion 1006
- 1003 (logging and other disturbance): spun off of issue 961, which was closed by PR 996
- 1001 (leaf orientation index range): cap range + graceful "fail" + parameter long name update
Calibration progress update (5-10 min)
- Adrianna rehased the ensemble update from last Thursday fates meeting
- Jessie reviewed how elm and trendy handles radiation partitioning; they are very different!
- Adrianna helped Jessie determine how to modify the case scripts to make sure elm read in the multiple stream data correctly
- Aside: TRENDY is an intercomparison project
- Unit testing radiation (Adrianna)
- Adrianna has started working on unit testing framework for radiation. Discussion about how we might utilize it for other fates modules.
- Greg suggested reviewing pFunit as a potential framework to adopt: https://github.com/Goddard-Fortran-Ecosystem/pFUnit
- Adrianna has started working on unit testing framework for radiation. Discussion about how we might utilize it for other fates modules.
- Integrate active crown fire (Jackie)
- Jackie is leaving NCAR; date TBD (likely April). She is finalizing her manuscript. She is going to resurrect the integrate active crown fire PR.
- European CLM group started up by Rosie; first meeting well attended (44 people). She will share the references and links with us: https://github.com/ciceroOslo/CLM-CET
- Idea is to help create communication connections for Europeans working with CLM and CLM-FATES.
- Upcoming meetings (5 min)
- Weekly synthesis of the PR Status Board (5-10 min)
- Latest issues (5-10 min)
- Calibration progress update (5-10 min)
- Walk-on topics
-
FATES modeling meeting: Thursday March 16 - Chacon-Labella
- Confirm Adrianna to lead
Integrated
Updated
- E3SM#5369 (elm-fates fire): IBM specific complier issue resolved. Testing complete. Awaiting integration.
- Soil moisture increase PRs:
- 991 (reduced restart memory): Reviewed. Ryan to add clarifying comments. To be tested.
-
958 (drought deciduous phenology): Review in process. Check in with Chongang feedback?
- Ryan will reach out to Chongang if necessary, but doesn't appear like it will be.
-
984 (photosynth temp acclimation):
fates_leaf_photo_temp_acclim_thome_time
added to paramter file. Update xmlpatch and test.- New parameter variable is using years. Is there a strong feeling about whether or not to make the units consistent with its related parameter? No strong preference; keep as is.
- 996 (NaN logging bugfix): Review in process. Fixes are straight forward. Should be able to test soon.
-
999 (CLM BGC compatibility): Ryan testing; seeing some failures
- Ryan and Adrianna meeting to discuss tomorrow. Charlie would like to see some explanation for the differences.
Check in
- 836 (Hardening and frost mort): Ask Rosie to check in with Marius
Charlie, Ryan and Greg conducted another issue triage review last week with multiple updates and closures.
Updated
- 935 (longwave down issue): Sam Levis and Xiulin continue to discuss
- 707 (moss and lichen): Checking in with Hui on status.
Calibration progress update (5-10 min)
- Adrianna working on 300 member ensemble to address albedo bias
- Half failed due to various balance check failures.
- Reviewing the tests that failed, only
fates_rad_lead_xl
parameter appears to be relatively well correlated to failing tests.- Adrianna to test ensemble without varying this to see how much this alleviates failures.
- Review the logs to collect the balance check failure type
- Jessie successfully ran simulations with TRENDY data (per feedback from Daniel Kennedy)
- Combining the seperated diffuse and direct radiation in TRENDY data allowed elm to run. Dan said clm can handle this, but maybe elm can't?
- Charlie recommended reviewing output to compare the way the model seperates the two, versus the TRENDY data split.
- Combining the seperated diffuse and direct radiation in TRENDY data allowed elm to run. Dan said clm can handle this, but maybe elm can't?
- Jennifer: Started part-time, temporary position at a carbon market startup (similar to Polly). Company is Cultivo.
- Charlie: LUH2 python script prototype. xESMF issue feedback. Pushback is more using packages with deep dependencies.
- Ryan: Unit testing on the Norman scheme to see where that component breaks. He feels like this is something worth elevating in priority.
- Adrianna has been testing collapsing fates information into ctsm radiation scheme to see how this affects albedo and assess the differences.
- Adrianna relayed that Keith has concerns that Norman is unstable for certain input conditions.
- Upcoming meetings (5 min)
- Weekly synthesis of the PR Status Board (5-10 min)
- Latest issues (5-10 min)
- Calibration progress update (5-10 min)
- Demographic benchmarking project update (Jessie)
- Discussion on github reviews (Ryan)
-
FATES modeling meeting: Thursday March 16 - Chacon-Labella
- Adrianna to lead as Charlie and Jackie will be out (Jackie to ask Adrianna)
Update
-
931 (Atkin respiration model): Implemented variable name and C4 updates
- Ask Charlie if qualitative testing remains
- Test if its b4b again
- Jessie to run most recent changes to see how C4
- Updates to the naming of parameters has been executed
- Aside: parameter file update
- Jessie would like the allometry for the parameter file to be updated
- Consensus is that this should be its own thing
- Acclimation PR will have a new parameter that is hardcoded. Charlie would like to make it a new parameter.
- Jessie would like the allometry for the parameter file to be updated
- Aside: parameter file update
- Ask Charlie if qualitative testing remains
-
E3SM#5369 (elm-fates fire): Namelist defaults and build update. Final testing.
- New tests are forthcoming pending discussion with E3SM dev team. Greg is rebasing and working on some namelist cleanup
-
984 (Acclimation): will come after Atkin, Ryan will add XML parameter change content
- Charlie will convert hardcoded parameter to file and add code to read in the new variable
- 958 (Drought Phenology): Check in with Marcos, looks good, check in about grasses with Chonggang?
-
995 (Large tree init)
- Aside: Ryan wondering if this is recommended for beyond nocomp? Jessie and Charlie think this is a good question
To Do:
- Create PRs to E3SM and CTSM that use higher soil moisture in cold start for ALL fates, not just fates hydro
- Jessie: Where does the code exist in the CLM side? Charlie will create the ctsm-side PR. Jessie will create the E3SM PR.
- Charlie wondering if there should be further discussion of why the hlms are using this low hydro init value. He will tag relevant people in the ctsm pr.
- Jessie: Where does the code exist in the CLM side? Charlie will create the ctsm-side PR. Jessie will create the E3SM PR.
- New
- 994 (bareground establishment): Increasing soil moisture and starting with larger trees
- Updated
- 935 (longwave down issue): investigations restarting
-
845 (too much canopy): Adrianna and Jessie noting this is coming up in SP mode calibration work. This could also be causing issue #986.
- This seems to be popping up a lot; we probably should spend some more time investigating this.
Calibration progress update (5-10 min)
- Adrianna out today
- Jessie having difficulty using the TRENDY data. Crash timing depends on how the ordering of the file streams.
- Marcos wondering if TRENDY data has leap year and if that is causing difficulties in this case. He was seeing crashes in Feb.
- Potentially something with the metadata? Maybe it's reading the
time
variable metadata? Jessie renamed the filename to a different year, but the metadata is still the original year. - Jessie will post a discussion/issue.
- Jackie: hand-holding is good, but don't transcribe offline review for the reviewer. Ask them to do this so they get practice.A
- Scope of reviewer pool: Consensus is that we should include people outside of the core modeling team
- Rosie suggests that we should bring this up regularly at the modeling call and include early career researchers more.
- We should create a list of people to ask for review. Ryan and Rosie thinks this should be private to avoid perception of exclusivity.
- Include the "carrot" of review contributions in next DOI
- Charlie: Include both elm and clm communities to improve cross-pollination and exposure to the code.
- Can we find a link to an existing github review tutorial?
- Upcoming meetings (5 min)
- Weekly synthesis of the PR Status Board (5-10 min)
- Latest issues (5-10 min)
- Calibration progress update (5-10 min)
- Walk-on topics
- FATES modeling meeting: Thursday March 2 - Hawkins, Dagon, and Kennedy
Update
-
931 (Atkin respiration model):
- Proposal to update
fates_base_mr_20
and adjust Atkin parameter name - Patch tooling question: patch update split in two xml files versus combine file with nonsense value
- https://docs.google.com/presentation/d/1tnttlh9CGFG5fTRjyZf-ALXMKyVOF8OnFnQqmTyJmJk/edit?usp=sharing
- Proposal to update
- New
- Updated
Calibration progress update (5-10 min)
- Upcoming meetings (5 min)
- Weekly synthesis of the PR Status Board (5-10 min)
- Latest issues (5-10 min)
- Calibration progress update (5-10 min)
- CLM-FATES Dynamic Land Units meeting report back (5-10 min)
- Walk-on topics
-
FATES modeling meeting: Thursday February 16, TBD
- Consensus was to cancel this given all the meetings that have been happening recently.
- Integrated
- 888 (C-based harvest)
- 926 and 971 (history updates)
-
CTSM#1932 (NEON FATES)
- Adrianna noted that some of the sites don't get off the ground (similar to what Jessie is seeing with some ForestGEO sites). Could be due to the soil moisture.
- How to make initial conditions "season-proof" (Marcos)
- Jessie to start a discussion thread to collect ideas that came up at the meeting.
- Update
-
824 (deallocate): Issue due to deallocating
target
through associated pointer. Testing. - 931 (Atkin respiration model): Tested without param correction. B4B in the case.
- 958 (Drought deciduous): Stable. Prioritize this for review next. New features to come in later PR.
-
824 (deallocate): Issue due to deallocating
- New
- E3SM#5429 (C-based harvest): rebased replacement for original E3SM#5106: Approved and tested. Waiting for integration.
- 984 (Photosynth temp acclimation)
- New
- 983 (NAG compiler PRT2 issue): hanging on history write
- Updated
-
855 (SP mode help): novice seeking help
- Adrianna noted that they posted in a bunch of places. Keith pointed them to the CESMDiscussion board.
-
855 (SP mode help): novice seeking help
Calibration progress update (5-10 min)
- Adrianna, Jessie, and Rosie to start meeting regularly this week on Wednesdays
- GPP run disparity between Jessie (low) and Adrianna's (high) runs
- Run through the same ILAMB intercomparison
- Different forcing data sets (what is the E3SM default?). To do: create a common protocol document to make sure we are all working from the same settings, data, process, etc.
- Design document in progress
- Sharing proposals using FATES that are in the works
- MIMICS+ (Rosie): Ryan to share the development path document for FATES and MIMICS coupling
Canceled due to Land Model Working Group Meeting
- Upcoming meetings (5 min)
- Weekly synthesis of the PR Status Board (5-10 min)
- Latest issues (5-10 min)
- Calibration progress update (5-10 min)
- Walk-on topics
- FATES modeling meeting: Thursday February 2, Nick Smith
-
Land Model Working Group Meeting (Feb 6-8)
- LULCC agenda meeting today (1PM PST/2PM MST/4PM EST)
- Integrated
- Update
-
E3SM#5106 (c-based harvest)
- Testing. Ryan investigating phosphorous dynamics test failure for eca mode.
-
824 (deallocate)
- Ryan implemented diagnostic checks and potential fix. Greg testing
- 971 and 926: testing
- 910 (water balance): Greg to make changes and test
-
E3SM#5106 (c-based harvest)
- Closed
-
769 (leaf memory): Closed.
- To be split up into three seperate tasks. See issue 978 below
-
769 (leaf memory): Closed.
Calibration progress update (5-10 min)
- Upcoming meetings (5 min)
- Weekly synthesis of the PR Status Board (5-10 min)
- Latest issues (5-10 min)
- Calibration progress update (5-10 min)
- Walk-on topics
- FATES modeling meeting: Thursday February 2, Nick Smith
-
Land Model Working Group Meeting (Feb 6-8)
- Email thread on-going discussion
- Update
-
E3SM#5106 (c-based harvest)
- Holding for issue 973 fix. PR 976 drafted with fix.
-
769 (leaf memory)
- scientific testing ongoing, but there are concerns. Discussion set for next week.
-
931 (Atkins)
- Can we bring this in as a non-default option even given discussion around C4 grass handling?
- Yes, although Charlie wants to clean up first.
- Can we bring this in as a non-default option even given discussion around C4 grass handling?
-
851 (hydro stability)
- Marcos found what may have been a water-balance bug which was showing up in elm-fates
-
E3SM#5106 (c-based harvest)
- New
-
CTSM#1932 (NEON fates)
- Adrianna to announce the capability on the FATES github discussion announcments page
-
973 (tveg24 debug issue)
- fix: set
patchno
for fbg + nocomp runs (not just sp mode runs). See PR 976 above.
- fix: set
-
975 (nuopc issue): ctsm-side issue 1937 created.
- OP has it working, but the fix is a little odd.
- Greg to close this on the fates side and Erik to ask if this is ok to close on the ctsm side.
-
CTSM#1932 (NEON fates)
Calibration progress update (5-10 min)
- Discussion of ILAMB script progress
- Calibration team to set up time to bring Adrianna back up to speed (post-LMWG meeting) and meet weekly
- Given bias with Norman, 2-stream is the long term goal
- Most canned 2-stream algorithms assume stacked leaf-layers. Definitely possible, but needs to account for parallel leaf-layers.
- Ryan has python prototype that does this that we could port.
- We should also consider future proofing radiative transfer scheme (they are on to multi-streams)
- Upcoming meetings (5 min)
- Weekly synthesis of the PR Status Board (5-10 min)
- Latest issues (5-10 min)
- Calibration progress update (5-10 min)
- Will Weider discuss CTSM PR expectations and resource allocation
- Walk-on topics
- FATES modeling meeting: Thursday January 19, topic TBD
-
Land Model Working Group Meeting (Feb 6-8)
- Coordinating a clm-fates development discussion
- When Ryan is back we should coordinate a meeting to discuss LULCC
- Will also floated the idea of scoping out more fates training with cesm containers
- Josh Rady was interested in what containerization offerings where out there
- Coordinating a clm-fates development discussion
- E3SM Spring All-hands meeting: TBD
- Update
-
E3SM#5106 (c-based harvest)
- Testing revealed some additional API25 compatibility work was needed. Shijie pulled new PR from Ryan. Re-test today.
- 888 (c-based harvest)] companion deconflicted and up-to-date with API25
- 769 (leaf memory): scientific testing ongoing
-
E3SM#5106 (c-based harvest)
- New
- New
Calibration progress update (5-10 min)
- Getting Adrianna up to speed on recent work
- More broadly: re-onboarding Adrianna to the updates from the last few months
- Jessie continuing work on Atkins discepencies
- PR expectations discussion
- Rewording 2a so as to not suggest precluding turn FATES on by default
- 2b example is hillslope-hydrology
- Ryan noted that we should add a refactor/code cleanup/bug fixes/performance improvement item (1c?)
- (Marcos) 1a concerns about correcting bug or improving model to better align with data but making something else worse
- (Will) Agreed. Example LUNA fixes degraded the performance, but were necessary
- Resource allocation discussion
- Dedicated FATES resources at NCAR: Jackie and Adrianna
- How to handle concern of where to focus development (big-leaf versus fates)
- CESM3 and FATES
- Ideally there would be CESM FATES runs for TRENDY and CMIP
- LULCC is limiting factor -> resource allocation
- Ideally there would be CESM FATES runs for TRENDY and CMIP
No meeting due to lab holidays
2024 Software Engineering Meeting Notes
2023 Software Engineering Meeting Notes
2022 Software Engineering Meeting Notes
2021 Software Engineering Meeting Notes
FATES API and Host Land Model compatibility table
Relevant References page (User's Guide)
Moorcroft et al. 2001. Ecological Monographs, 74:557-586.