-
Notifications
You must be signed in to change notification settings - Fork 92
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
two-stream - b4b Norman and no history changes #1141
Merged
Merged
Changes from 98 commits
Commits
Show all changes
114 commits
Select commit
Hold shift + click to select a range
1a7fdb1
Added two-stream code and common globals
rgknox 282d8ca
Refactor some fates radiation code. New shared memory, and renamed dr…
rgknox 013e07f
two-stream changes, updated photosynthesis calls to protect against n…
rgknox a11d4ae
cleaning up two-stream calls and comments
rgknox b786308
Added fsun to history diagnostics,testing different fsun formulations…
rgknox 882e02a
added type definition
rgknox 53ee4a2
All sorts of two-stream and general radiation clean up
rgknox e7b58da
Updated error logging in twostream to use logging file units and erro…
rgknox 6ccedd5
removed cold init argument for radiation driver start
rgknox 783639b
bug fix related to two-stream radiation updates
rgknox 8e34f39
Various updates and fixes to two-stream, including error checking and…
rgknox 5211412
Fix to the restart order issue
rgknox 5b2129a
moved nlevsclass calculation earlier in subroutine
rgknox 28eab39
Added site level scratch array for the 2stream solver
rgknox 22b6b36
Updated unit tests to handle scratch arrays in twostream
rgknox ae56d63
Merge up two-stream part 1
rgknox 3e959a8
merge resolution between two-stream and patch/cohort refactor
rgknox 6782b4a
removing layer height profile, for now
rgknox fe3c57f
Merge branch 'main' into two-stream-clean-api26-v2
rgknox 7dc7c47
conflict resolutions for two-stream
rgknox 6d59ea5
add readme to init bld directory
rgknox 3a6b2dc
Updates to two-stream functional unit tests
rgknox fd0d926
Connected the radiation model switch in the parameter file to the in-…
rgknox e2aac37
Merge branch 'two-stream-clean-api26-v2' into two-stream-clean-api26
rgknox d7396b1
Updated radiation model checks to allow for norman and two-stream. Up…
rgknox bc03f65
Started a new radiation unit test that is used to target specific can…
rgknox 174f292
Updates to two-stream error reporting, checking the linear solution, …
rgknox b282476
Merge branch 'two-stream-clean' of github.com:rgknox/fates into two-s…
rgknox e10595c
Applied higher threshold to two-stream closure check, added forced cl…
rgknox 5217289
Set albedo correction to true in fates two-stream
rgknox f44848b
Merge branch 'two-stream-clean-albedo' into two-stream-clean
rgknox 713bbad
small updates to two-stream unit tests
rgknox 3c6dee2
Added error tracking in two-stream
rgknox 3966cc8
Updates to error tracking of two-stream
rgknox 1f6eb83
Update biogeochem/FatesAllometryMod.F90
rgknox 25923bc
Update radiation/TwoStreamMLPEMod.F90
rgknox 169a8bd
Updates to functional unit test code
rgknox b74b021
Merge branch 'two-stream-clean-errcheck' into two-stream-clean
rgknox 4d8bebc
Updates to the history file for twostream
rgknox b3370b9
Cleaning two-stream and radiation drivers to get correct history diag…
rgknox 460645d
more cleaning of radiation history
rgknox bd33520
updates to the radiation history diagnostics
rgknox 1a3ae69
Changed the avmu radiation parameter to a constant remembered in the …
rgknox 156a17a
Added isnan() functions instead of comparing to self
rgknox a26146d
Removed old optional two-stream derivations and now using only B1 and…
rgknox a9260d0
Added checks on scattering coefficient
rgknox 772d57e
Merge branch 'two-stream-clean' into two-stream-clean-v3solution
rgknox df2dee1
fixed typo
rgknox 8ff6444
Merge branch 'two-stream-clean-v3solution' into two-stream-clean
rgknox 391a4f6
merge resolution and reverting of inventory init to not dump output
rgknox c3da9e6
Updated messaging on the param check failure if the fates_rad_model i…
rgknox 789c36a
Updated fail message on rad parameter bounds
rgknox 7f14138
Addressing reviewer comments
rgknox 1aa2ead
removed unused alias in fateshistoryinterface
rgknox dbb2cfe
fixes to lingering merge conflict resolutions
rgknox 83f8156
Adding in protections to avoid Kb==a in two-stream
rgknox 904c0e4
Merge branch 'two-stream-clean' of github.com:rgknox/fates into two-s…
rgknox 7112368
clean up comments and old text in two-stream code
rgknox 6ca1ccb
Conflict resolution two-stream and api30
rgknox 2e140be
small tweaks to error tracking in fates two-stream
rgknox 9f6f5c2
cleaning up conservation error diagnostics
rgknox 3fa96a3
Tweaking the correction to beam optical depth, needs provisions for n…
rgknox cb6b677
Initializing two-stream error with the ignore flag
rgknox 24760aa
Tweaks to area squeezing in two-stream
rgknox d35b9b1
bug fix in area squeezing during two-stream
rgknox 0d7b490
Merge two-stream with api31
rgknox 31ed39b
cleaned up canopy mask logic
rgknox 38c3f1f
Added in code to preserve leaf to cohort scaling in norman mode, will…
rgknox 33c83cb
trivial merge resolution between two-stream and inventory uppdates
rgknox 46ecf39
altnerate reverted form of leaf layer calculation
rgknox 624293b
testing out b4b corrections
rgknox 5f4a8b6
Reverted incorrect nscalar
rgknox 33cb3ae
Merge branch 'two-stream-clean' into two-stream-clean-b4b
rgknox b88f636
More b4b attempts at two-stream
rgknox 3c0051e
debugging logic ends
rgknox c71dadb
more b4b attempts at two-stream
rgknox e0c564f
Merge branch 'two-stream-clean-b4b' into two-stream-clean-b4b2
rgknox 1d64107
making switches in b4b check consistent
rgknox d4ac49b
reverting history code in two-stream to match main for b4b
rgknox da54174
attempts to revert history code in 2str to get b4b
rgknox 1df494a
small b4b changes to two-stream
rgknox 56aa325
Creating seperate module for norman rad
rgknox 8cc57c9
fixed some typos
rgknox 7abc732
cleaning up photosynthesis code w/two-stream
rgknox 40f1a2a
Merge branch 'two-stream-clean-b4b-hist' into two-stream-clean-b4b2
rgknox a3754a1
Reverted history code to be as similar to base as possible
rgknox a972ff3
more two-stream b4b work
rgknox c34773a
reinstated ed_laisun_z and ed_laisha_z for b4b tests
rgknox 0c1e890
reverting canopy mask for b4b
rgknox 89aecd1
reworking towards b4b
rgknox c6da2fe
b4b stuff
rgknox f6b879e
b4b stuff for two-stream
rgknox 140e6ff
b4b for two-stream
rgknox 36105c3
more b4b stuff
rgknox 312593f
revert history to match main
rgknox 36be028
merge two-stream with lu1
rgknox 62a37f9
removed solver error (redundant with conservation error)
rgknox 893d8cf
merge updates on two-stream with luh2, as well as consolidation of ra…
rgknox b743547
two-stream cleaning up
rgknox 0366819
adding text around preserve_b4b
rgknox 5e4c6df
removed redundant declaration
rgknox f416029
Updating scipy module importing to work for both new and old conventions
rgknox 9fa0153
added try clause to scipy module loading
rgknox 64eb770
deleted unnecessary comment
rgknox 6935a40
removed unncecessary comment
rgknox 5daf737
set intent in on veg_tempk
rgknox 1642919
removed unused global declarations in history
rgknox 200f024
removing hlm_numSWb use in code, it is redundant
rgknox f6832c3
two-stream responding to reviewer comments
rgknox d127f51
resolving more reviewer comments
rgknox 0079a85
Fixing zero protections on two-stream zenith calcualtions
rgknox 9c6f822
Merge branch 'main' into two-stream-clean-b4b-nohist
rgknox af2c3b9
Fixes to nan checks in fates two-stream
rgknox 7b2cdbe
fixed try statement on module importing in python scipy
rgknox File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
What's the background here on the
layer_height_method
? Will one method be choosen later or be converted to a formal parameter (i.e. moved into the parameter file)? A think a few code comments here explaining the rationale for the hard coded value would be helpful.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 constant depth assumption is the default, and is equivalent to the way we currently do things in the canopy summarization code. I added the alternative so that we have the option to explore it in the future. I have not done a thorough investigation of it yet though.