Do not create two CO2 tracers#1581
Conversation
- Only create one total CO2 tracer when a CO2 chemistry is running and co2_cycle is on - When sharing total CO2 between co2_cycle and chemistry, do not create duplicate CO2 diagnostic fields
|
While I tried to make this change relatively robust, I think there are a couple of weak points.
|
|
At a meeting about this, we decided that the diagnostic name for CO2 should be However, I think it we would end up with two output fields for the same quantity. Currently, we already output all 3D constituent fields in I think the same issue would affect the surface flux since those are also output using Did I miss something or forget a detail? @klindsay28, @fvitt, @jimmielin, @nusbaume |
|
I can think of a couple of ways to implement the change above, however, we are not going to do this for NorESM (as among other things, it complicates post-processing and analysis) so I leave it up to someone from the CAM group to pick this up. Some ideas for implementing
|
Probably should error out if CO2 is not transported. CO2 is a long-lived tracer and may never happen. |
Is this only if |
Yes only if co2_cycle is active. |
Based on discussions with Keith Lindsay: In CESM1 and CESM2, the addition of _BGC%BDRD just added some extra diagnostic fields that could be useful for understanding the carbon cycle. However, with the proposed changes in ESCOMP/CAM#1581 to get co2_cycle working with recent versions of CAM, _BGC%BDRD would change answers, and so wouldn't strictly be a concentration-driven configuration (if I understand correctly). Therefore, at least for now, we have decided to have no _BGC specifier on the concentration-driven compsets. We may revisit this choice later.
|
@lkemmons and @tilmes |
I am not sure I do not fully understand the problem, but this sounds reasonable. You should ask Ben Gaubert - he has done more with CO2. |
|
FYI - At today's CSEG meeting today, it was pointed out that this PR needs to come in for the emission driven runs. We are targeting it for the next CAM CESM alpha tag. Please try to have reviews done soon - there is one CESM alpha tag in front of the CAM one (so maybe next week will be our turn)? |
Addresses #1580