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

Feature soildensity #215

Merged
merged 4 commits into from
Aug 3, 2022
Merged

Feature soildensity #215

merged 4 commits into from
Aug 3, 2022

Conversation

dschlaep
Copy link
Member

@dschlaep dschlaep commented Aug 2, 2022

- close #209 (Integrate SOILWAT2's soil density descriptor)

- see DrylandEcology/SOILWAT2#324 (SOILWAT2 submodule points to this devel branch for now)

- new behavior: user specifies whether input soil density represents matric or bulk density; code then determines both matric and bulk density values from inputs

- Class `swSite` gains new slot `"SoilDensityInputType"` and associated methods `swSite_SoilDensityInputType()` (#209).
"SoilDensityInputType" encodes if soil density inputs represent matric soil or bulk soil values.
@dschlaep dschlaep self-assigned this Aug 2, 2022
@codecov
Copy link

codecov bot commented Aug 2, 2022

Codecov Report

Merging #215 (8cd21b1) into release/devel_v6.0.0 (c32a221) will decrease coverage by 0.04%.
The diff coverage is 55.00%.

@@                   Coverage Diff                    @@
##           release/devel_v6.0.0     #215      +/-   ##
========================================================
- Coverage                 54.56%   54.52%   -0.05%     
========================================================
  Files                        42       42              
  Lines                      6478     6491      +13     
========================================================
+ Hits                       3535     3539       +4     
- Misses                     2943     2952       +9     
Impacted Files Coverage Δ
R/A_swGenericMethods.R 32.64% <0.00%> (-0.28%) ⬇️
R/F_swSite.R 25.60% <0.00%> (-0.85%) ⬇️
R/K_swContainer.R 21.27% <0.00%> (-0.18%) ⬇️
src/rSW_Site.c 98.04% <100.00%> (+0.03%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

…ensity type)

SOILWAT2 commit ac323024c142840211c6e41d84083e7f4211ef3b (Merge pull request #324 from DrylandEcology/feature_soildensity_type)
@dschlaep dschlaep merged commit 6e5504b into release/devel_v6.0.0 Aug 3, 2022
@dschlaep dschlaep deleted the feature_soildensity branch August 3, 2022 18:35
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 this pull request may close these issues.

2 participants