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

Missing variables in MPI messing up with soil conductivity #490

Closed
ccarouge opened this issue Nov 27, 2024 · 0 comments · Fixed by #491
Closed

Missing variables in MPI messing up with soil conductivity #490

ccarouge opened this issue Nov 27, 2024 · 0 comments · Fixed by #491

Comments

@ccarouge
Copy link
Member

When cable_user%soil_thermal_fix is True, we calculate the soil conductivity using this function:

FUNCTION total_soil_conductivity(ssnow,soil)

It is called from stempv:

IF (cable_user%soil_thermal_fix) THEN
ccnsw = total_soil_conductivity(ssnow,soil)
ELSE
ccnsw = old_soil_conductivity(ssnow,soil)
ENDIF

This new soil conductivity calculation is using the parameters soil%sand_vec, soil%ssat_vec, soil%watr, soil%cnsd_vec. These parameters were introduced for the groundwater scheme. It turns out these parameters were not fully implemented in main since the groundwater scheme isn't fully implemented. Initialisations and MPI implementations are faulty.

It could be these parameters or other parameters for the ground water are currently used in the code elsewhere. In this case, fixing this issue would have more impact than the soil conductivity alone. But I do not intend search to see if that is the case.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant