-
Notifications
You must be signed in to change notification settings - Fork 383
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
floating invalid with F20TRC5AV1C-04P2 and -bgc bgc -nutrient_comp_pathway eca
#3088
Comments
I tested this again with master of Nov 6th and I see same issue. |
Hi Noel,
I think Jinyun's right, his PR
https://github.com/E3SM-Project/E3SM/pull/3303/files fixed the nan issue
for col_plant_pdemand_vr, should also fix the floating invalid issue you
got in line 2415 AllocationMod.F90.
dsolutionp_dt(c,j) = gross_pmin_vr(c,j)
-potential_immob_p_vr(c,j)
- &
col_plant_pdemand_vr(c,j) + biochem_pmin_vr_col(c,j)
+ &
primp_to_labilep_vr_col(c,j) + pdep_to_sminp(c) *
ndep_prof(c,j)
…On Wed, Dec 4, 2019 at 3:35 PM noel ***@***.***> wrote:
I tested this again with master of Nov 6th and I see same issue.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
<#3088?email_source=notifications&email_token=AC3AHXQCATMXBRQ5B5CTY4TQXA5FVA5CNFSM4IGUVWWKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEF64V5I#issuecomment-561892085>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AC3AHXQFHDKZGZNKODT4PZLQXA5FVANCNFSM4IGUVWWA>
.
--
Research Scientist
Climate Sciences Department
Lawrence Berkeley National Laboratory
1 Cyclotron Rd, Berkeley, CA, USA 94720
Senior Fellow
Berkeley Institute of Data Science
University of California, Berkeley
190 Doe Library, Berkeley, CA 94720
|
Hi Noel, Could you please try qzhu-lbl/lnd/plant_pdemand_init_fixnan see if the error is still there? |
I tried the same experiment again and it failed in the same way with master of July 7th.
|
With master of Nov19th, I tried this again. I don't see the same error, but instead, the same error as I found here #2044
I don't know if there were changes that allowed code to run further (and hit this error), or if this new error is complicating things here. |
It looks like the compset no longer available, but I tried the following: and I get yet a different error:
|
I'm going to close the issue as it's no longer the same error as original and not clear it's even supported. |
ensure we trigger tests on eamxx submods
This testing is on cori-knl using a master of Jul 23rd.
This test passes:
SMS_PT_D_Ln5.ne30_ne30.F20TRC5AV1C-04P2
However, if I do the following in the casedir:
And rebuild, I get the following error:
I also tested with ony
-bgc bgc
, which completed, but it doesn't let me build with only-nutrient_comp_pathway eca
(ie I needed both). Can someone confirm if this is a valid thing to test? That is, a valid way to configure? Note that I started with the following options:-bgc bgc -nutrient cnp -nutrient_comp_pathway eca -soil_decomp century -methane -nitrif_denitrif
which fails in the same way as above -- then narrowed down which options were causing the error.The text was updated successfully, but these errors were encountered: