Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use of C13/C14 timeseries for non-transient control cases gives incorrect input when run long enough #592

Open
ekluzek opened this issue Dec 10, 2018 · 2 comments
Assignees
Labels
blocked: dependency Wait to work on this until dependency is resolved bug something is working incorrectly science Enhancement to or bug impacting science

Comments

@ekluzek
Copy link
Collaborator

ekluzek commented Dec 10, 2018

Brief summary of bug

Some 1850 control simulations with Carbon isotopes turned on the C13/C14 time-series files. Any of these simulations that ran for more than 1850 would have used the wrong values for the carbon isotopes.

General bug information

CTSM version you are using: any version

Does this bug cause significantly incorrect results in the model's science? Yes (for C13/C14)

Configurations affected:

1850 or 2000 compsets that have CLM4.5/CLM5.0 and isotopes with the time-series on

non-transient simulations that use the C13/C14 time-series as follows...

use_c13_timeseries = .true.
use_c14_bombspike = .true.

will be using constant 1850 conditions until you run for more than 1850 years, when it will start running through the time-series.

Details of bug

Note the reason for this is due to #182, where the hardcoded mechanism for reading in isotopes was fragile and allowed little control. Resolving that issue can resolve this problem as well as make it able to work with 2000 control, which we can NOT do at this time. For 2000 conditions isotopes will be running with 1850 conditions, and if you turn the time-series on as above, you'll have to run for 2000 years before you are running with 2000 conditions, and then it will fly by until you reach the end of the file.

Note Esther Brady pointed this limitation out on Nov/21st, and we realized it was a bug at that time.

@mingjieshi
Copy link

Hi Erik,

I checked the time-series of delta C14 respirations (e.g., delta_C14ER = ((c14ER/ER)/1e-12 - 1) * 1e3) over Arctic (the historical runs are at /gpfs/fs1/p/cgd/tss/people/oleson/CLM_LAND_ONLY_RELEASE/CLM5/clm50_r270_1deg_GSWP3V1_iso_newpopd_hist/lnd/proc/tseries/month_1/), and found earlier peaks (during 1935-1945) than I expected (1960s). Do you think if this bug was the reason for the early peak? Thanks!

Mingjie

@billsacks billsacks added tag: bug - impacts science bug something is working incorrectly and removed type: bug - impacts science labels May 24, 2019
@ekluzek ekluzek removed this from the cesm2.1.1 milestone May 21, 2020
@billsacks billsacks added the blocked: dependency Wait to work on this until dependency is resolved label Dec 9, 2021
@billsacks
Copy link
Member

Blocked: depends on #182 (in principle we could do this without #182 but #182 makes this significantly easier).

@samsrabin samsrabin added science Enhancement to or bug impacting science and removed bug - impacts science labels Aug 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked: dependency Wait to work on this until dependency is resolved bug something is working incorrectly science Enhancement to or bug impacting science
Projects
None yet
Development

No branches or pull requests

4 participants