-
Notifications
You must be signed in to change notification settings - Fork 282
334 rho sigma diagnostic regridding #348
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
Merged
adcroft
merged 66 commits into
mom-ocean:dev/master
from
nichannah:334-rho-sigma-diagnostic-regridding
Dec 8, 2016
Merged
Changes from all commits
Commits
Show all changes
66 commits
Select commit
Hold shift + click to select a range
e401fdf
Diagnostic regridding for zstar, sigma and rho vertical coordinates. …
df6cd17
Correct available_diags for vertically remapped diagnostics. #334
nichannah 5240529
Extend vertical remapping to handle SLIGHT and HYCOM1. #334
nichannah c3ce7bc
Use conversion=GV%H_to_kg_m2 for umo,vmo diagnostics
adcroft b0a60b8
Re-factor of MOM_diag_mediator + cell_methods fix
adcroft 54b5724
Re-factor of diag_mediator: removed functions is_u_axes, is_v_axes, ...
adcroft 11141a2
Implemented z-coordinate diagnostics of interface variables
adcroft bfd21a4
Implemented remapped diagnostics of vertically extrinsic fields
adcroft 3a7efe5
Labeled some diagnostics as vertically integrated by layer
adcroft ccdc9b6
Doxygenized assert()
adcroft b77344b
Clarification: renamed vector of diag_remap to diag_remap_cs
adcroft 4804926
Added axes_groups for each diagnostic coordinate
adcroft 9338ee5
Clarification: renamed diag_type%axes to diag_type%axes_ids
adcroft d393a08
Re-introduced name "Z*" for ZSTAR coordinate mode
adcroft 0816589
Merge branch 'user/aja/diag_interp_and_extrinsic' into nicjhan-334-rh…
adcroft 47c82b8
Fixed reintegrate_column() for missing data cases
adcroft 1f0a1d7
Renamed old diag-to-z diagnostics module
adcroft 73207b5
Merge pull request #1 from adcroft/user/aja/diag_interp_and_extrinsic
f059eb5
Remove unused code and add/improve comments after aja/diag_interp_and…
3ee3dac
Merge branch 'dev/master' into 334-rho-sigma-diagnostic-regridding
3bd0add
Better diagnostic debugging and fixed subroutine name for correctness…
b1b2381
Improved checking for vertical grid definition input, also layer var …
fc6942f
Don't pass temporary axes variable to register_diag_field(). #334.
604540b
Compiler error: fix dummy argument intent. #334
7bb14ed
Whitespace and formatting fixes. #334
1057172
Whitespace and comment fixes. #334.
42670b4
Change names of several old z coord diags to be the same as regular d…
593de0a
Add some masking for vertically remapped diags based on on bathymetry…
2072bfe
Merge dev/master fixing conflicts. #334.
fdb4caf
Fix merge mistake. #334
8fb94a2
Variable renaming, whitespace and documentation. #334
32586a4
Diagnostic vertical remapping performance improvement. Only activate …
caf702d
Whitespace changes. #334
b701b8b
Fix masking for vertically remapped diagnostics. #334
e4e167a
Merge branch 'dev/master' into 334-rho-sigma-diagnostic-regridding
bd1028e
Allow DIAG_REMAP_Z_GRID_DEF to use old protocol
adcroft 4493464
Reduce references to external parameter vertical_coord_strings
adcroft 9fac48a
Reduced available coords for diags
adcroft 78fdc73
Removed out-of-date error checking in MOM_diag_to_Z
adcroft 9fb5f33
Added arbitrary number of diagnostic coordinates, NUM_DIAG_COORDS
adcroft e6385f7
Updated ALE_initRegridding API
adcroft c6db899
Separated ALE_initRegridding into re-usable parts
adcroft d0cdda0
Hide unused coordinate config parameters, pt 1
adcroft d4361f3
Hide interp_scheme when not needed, pt 2
adcroft 7608654
Overloaded regrid_consts:state_dependent()
adcroft 0640a80
Added extract_integer() to MOM_string_functions
adcroft 599044d
Re-factor of regrid initialization, pt 1
adcroft 1bc03d8
Moved reading of regridding parameters into MOM_regridding.F90
adcroft a323495
Changed default DIAG_COORD_Z grid to be WOA09 for deep configurations
adcroft 0d8783a
Fix conflict with dimension names in diag_manager
adcroft 043345e
Extended "UNIFORM" syntax to UNIFORM[:N[,dz]]
adcroft 8acebf1
Cleaned up diag_remap_configure_axes() arguments
adcroft b726404
Minor re-factor to insert new layer in register_diag hierarchy
adcroft c71fe83
Fixed masking for _z remapped diagnostics
adcroft 6fae7f7
Added _xyave to all remapped diagnostics
adcroft 58b26a0
Remove support for diagnostic horizontal averaging on Z interfaces. #334
nichannah c32946c
Fix subroutine name in error string. #334
nichannah 3aa0e78
Fix "edges" attribute for dimension variable in z-diag output
adcroft ff858c7
Corrected variable names (changed extrinsic to extensive)
adcroft 5bd2cfc
Implemented xy-averaged diagnostics for interface data
adcroft fd533dc
Added draft of Diagnostics page
adcroft 90d31ed
Moved to flat (no subdirs) mode for doxygen
adcroft b1ed87f
Merge branch 'dev/master' into nicjhan-334-rho-sigma-diagnostic-regri…
adcroft 360fb36
Moved umo,vmo to MOM.F90 and based on uhtr,vhtr
adcroft c838594
Re-enable support for diagnostic horizonal averaging on Z interfaces …
nichannah a4031da
Fix variable intent compilation error in dev/master. #334
nichannah 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
This file contains hidden or 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
Large diffs are not rendered by default.
Oops, something went wrong.
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains hidden or 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 hidden or 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 hidden or 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.
I'm curious about why are you changed the declared string lengths to not match the string? I think it's a harmless change but was there a failure somewhere?
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.
I think I did that because I put these into an array below. My understanding is that Fortran doesn't like having variable length strings in an array. I'll add a comment.
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.
I see that now. I'm surprised that
character(len=*), dimension(n), parameter =works - I would have thought the*is at odds with theparameter...