-
Notifications
You must be signed in to change notification settings - Fork 5
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 soilwat2 with co2 #51
Conversation
- SOILWAT2 commit DrylandEcology/SOILWAT2@2063e38643b7ad49c e0eade239e8093886c7c123 cleaned up struct 'SW_VEGPROD': * defined struct 'VegTypeOut' and 'CoverType'
- update makefile
- close DrylandEcology/SOILWAT2#99 - CO2 flags are turned off - this commit compiles and executes the testing example
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The PR compiles properly. I tested with CO2 off and found no difference in the output (bmassavg.csv) or debugging output files between this PR and the master branch. Turning off the CO2 features within STEPWAT2 is fine for now. I will add an issue to STEPWAT2 to incorporate additional changes to make the CO2 code functional with STEPWAT2. For instance, we will need to add the carbon.in file to /Stepwat_Inputs/Input/sxw/Input.
Let's wait with merging the pull request into master until the corresponding soilwat_co2_effects branch of SOIILWAT2 is merged into master. |
- added input for runon fraction
Adjust code so that the CO2-enabled version of SOILWAT2 can be used as submodule (currently tracking the soilwat_co2_effects branch of SOILWAT2).
This PR compiles and the testing example runs well and produces output similar to the output generated by #50