-
Notifications
You must be signed in to change notification settings - Fork 52
Consolidate base directory for diagnostics (obs, mapping and mask files) into one config option #500
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
Consolidate base directory for diagnostics (obs, mapping and mask files) into one config option #500
Changes from all commits
11e638f
9c796c4
6f827b4
06d559e
f2b710a
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -30,6 +30,17 @@ parallelTaskCount = 4 | |
# handle 12 simultaneous processes, one for each monthly climatology. | ||
ncclimoParallelMode = bck | ||
|
||
[diagnostics] | ||
## config options related to observations, mapping files and region files used | ||
## by MPAS-Analysis in diagnostics computations. | ||
|
||
# The base path to the diagnostics directory. Typically, this will be a shared | ||
# directory on each E3SM supported machine (see the example config files for | ||
# its location). For other machines, this would be the directory pointed to | ||
# when running "download_analysis_data.py" to get the public observations, | ||
# mapping files and region files. | ||
baseDirectory = /lus/theta-fs0/projects/ClimateEnergy_2/diagnostics | ||
|
||
[input] | ||
## options related to reading in the results to be analyzed | ||
|
||
|
@@ -39,11 +50,6 @@ baseDirectory = /lus/theta-fs0/projects/OceanClimate_2/mpeterse/20171031.tenYear | |
# names of ocean and sea ice meshes (e.g. oEC60to30v3, oQU240v3, oRRS30to10v3, etc.) | ||
mpasMeshName = oEC60to30v3wLI | ||
|
||
# Directory for mapping files (if they have been generated already). If mapping | ||
# files needed by the analysis are not found here, they will be generated and | ||
# placed in the output mappingSubdirectory | ||
mappingDirectory = /lus/theta-fs0/projects/ClimateEnergy_2/diagnostics/mpas_analysis/maps | ||
|
||
# names of namelist and streams files, either a path relative to baseDirectory | ||
# or an absolute path. | ||
oceanNamelistFileName = mpas-o_in | ||
|
@@ -112,29 +118,6 @@ endYear = 9999 | |
startYear = 1 | ||
endYear = 9999 | ||
|
||
[oceanObservations] | ||
## options related to ocean observations with which the results will be compared | ||
|
||
# directory where ocean observations are stored | ||
baseDirectory = /lus/theta-fs0/projects/ClimateEnergy_2/diagnostics/observations/Ocean | ||
|
||
[seaIceObservations] | ||
## options related to sea ice observations with which the results will be | ||
## compared | ||
|
||
# directory where sea ice observations are stored | ||
baseDirectory = /lus/theta-fs0/projects/ClimateEnergy_2/diagnostics/observations/SeaIce | ||
|
||
[timeSeriesSeaIceAreaVol] | ||
## options related to plotting time series of sea ice area and volume | ||
|
||
# plot on polar plot | ||
polarPlot = False | ||
|
||
[regions] | ||
# Directory containing mask files for ocean basins and ice shelves | ||
regionMaskDirectory = /lus/theta-fs0/projects/ClimateEnergy_2/diagnostics/mpas_analysis/region_masks | ||
|
||
[streamfunctionMOC] | ||
## options related to plotting the streamfunction of the meridional overturning | ||
## circulation (MOC) | ||
|
@@ -145,34 +128,4 @@ regionMaskDirectory = /lus/theta-fs0/projects/ClimateEnergy_2/diagnostics/mpas_a | |
# in its calculation, whereas the postprocessing script does. | ||
# NOTE: this is a temporary option that will be removed once the online | ||
# MOC takes into account the bolus velocity when GM is on. | ||
usePostprocessingScript = True | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I will put the MOC section back. That was removed by mistake. |
||
|
||
[climatologyMapSoseTemperature] | ||
## options related to plotting climatology maps of Antarctic | ||
## potential temperature at various levels, including the sea floor against | ||
## reference model results and SOSE reanalysis data | ||
|
||
# Times for comparison times (Jan, Feb, Mar, Apr, May, Jun, Jul, Aug, Sep, Oct, | ||
# Nov, Dec, JFM, AMJ, JAS, OND, ANN) | ||
seasons = ['JFM', 'JAS', 'ANN'] | ||
|
||
# list of depths in meters (positive up) at which to analyze, 'top' for the | ||
# sea surface, 'bot' for the sea floor | ||
depths = ['top', -200, -400, -600, -800, 'bot'] | ||
|
||
[climatologyMapSoseSalinity] | ||
## options related to plotting climatology maps of Antarctic | ||
## salinity at various levels, including the sea floor against | ||
## reference model results and SOSE reanalysis data | ||
|
||
# Times for comparison times (Jan, Feb, Mar, Apr, May, Jun, Jul, Aug, Sep, Oct, | ||
# Nov, Dec, JFM, AMJ, JAS, OND, ANN) | ||
seasons = ['JFM', 'JAS', 'ANN'] | ||
|
||
# list of depths in meters (positive up) at which to analyze, 'top' for the | ||
# sea surface, 'bot' for the sea floor | ||
depths = ['top', -200, -400, -600, -800, 'bot'] | ||
|
||
[timeSeriesAntarcticMelt] | ||
# a list of ice shelves to plot | ||
#iceShelvesToPlot = ['Antarctica', 'Peninsula', 'West Antarctica', 'East Antarctica', 'Larsen_C', 'Filchner', 'Ronne', 'Filchner-Ronne', 'Brunt_Stancomb', 'Fimbul', 'Amery', 'Totten', 'Ross_West', 'Ross_East', 'Ross', 'Getz', 'Thwaites', 'Pine_Island', 'Abbot', 'George_VI'] | ||
usePostprocessingScript = True |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -30,6 +30,17 @@ parallelTaskCount = 1 | |
# handle 12 simultaneous processes, one for each monthly climatology. | ||
ncclimoParallelMode = serial | ||
|
||
[diagnostics] | ||
## config options related to observations, mapping files and region files used | ||
## by MPAS-Analysis in diagnostics computations. | ||
|
||
# The base path to the diagnostics directory. Typically, this will be a shared | ||
# directory on each E3SM supported machine (see the example config files for | ||
# its location). For other machines, this would be the directory pointed to | ||
# when running "download_analysis_data.py" to get the public observations, | ||
# mapping files and region files. | ||
baseDirectory = /lus/theta-fs0/projects/ClimateEnergy_2/diagnostics | ||
|
||
[input] | ||
## options related to reading in the results to be analyzed | ||
|
||
|
@@ -39,11 +50,6 @@ baseDirectory = /lus/theta-fs0/projects/ClimateEnergy_2/azamatm/E3SM_simulations | |
# names of ocean and sea ice meshes (e.g. oEC60to30v3, oQU240v3, oRRS30to10v3, etc.) | ||
mpasMeshName = oRRS18to6v3 | ||
|
||
# Directory for mapping files (if they have been generated already). If mapping | ||
# files needed by the analysis are not found here, they will be generated and | ||
# placed in the output mappingSubdirectory | ||
mappingDirectory = /lus/theta-fs0/projects/ClimateEnergy_2/diagnostics/mpas_analysis/maps | ||
|
||
# names of namelist and streams files, either a path relative to baseDirectory | ||
# or an absolute path. | ||
oceanNamelistFileName = mpas-o_in | ||
|
@@ -111,52 +117,3 @@ endYear = 10 | |
# For valid statistics, index times should include at least 30 years | ||
startYear = 1 | ||
endYear = 9999 | ||
|
||
[oceanObservations] | ||
## options related to ocean observations with which the results will be compared | ||
|
||
# directory where ocean observations are stored | ||
baseDirectory = /lus/theta-fs0/projects/ClimateEnergy_2/diagnostics/observations/Ocean | ||
|
||
[seaIceObservations] | ||
## options related to sea ice observations with which the results will be | ||
## compared | ||
|
||
# directory where sea ice observations are stored | ||
baseDirectory = /lus/theta-fs0/projects/ClimateEnergy_2/diagnostics/observations/SeaIce | ||
|
||
[timeSeriesSeaIceAreaVol] | ||
## options related to plotting time series of sea ice area and volume | ||
|
||
# plot on polar plot | ||
polarPlot = False | ||
|
||
[regions] | ||
# Directory containing mask files for ocean basins and ice shelves | ||
regionMaskDirectory = /lus/theta-fs0/projects/ClimateEnergy_2/diagnostics/mpas_analysis/region_masks | ||
|
||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. here is another example of section (below this comment) that seems unrelated to this PR and that is being removed. Not sure if it was intentional or not. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I really think we don't want this stuff in the example config files, so, yes, it was intentional. |
||
[climatologyMapSoseTemperature] | ||
## options related to plotting climatology maps of Antarctic | ||
## potential temperature at various levels, including the sea floor against | ||
## reference model results and SOSE reanalysis data | ||
|
||
# Times for comparison times (Jan, Feb, Mar, Apr, May, Jun, Jul, Aug, Sep, Oct, | ||
# Nov, Dec, JFM, AMJ, JAS, OND, ANN) | ||
seasons = ['JFM', 'JAS', 'ANN'] | ||
|
||
# list of depths in meters (positive up) at which to analyze, 'top' for the | ||
# sea surface, 'bot' for the sea floor | ||
depths = ['top', -200, -400, -600, -800, 'bot'] | ||
|
||
[climatologyMapSoseSalinity] | ||
## options related to plotting climatology maps of Antarctic | ||
## salinity at various levels, including the sea floor against | ||
## reference model results and SOSE reanalysis data | ||
|
||
# Times for comparison times (Jan, Feb, Mar, Apr, May, Jun, Jul, Aug, Sep, Oct, | ||
# Nov, Dec, JFM, AMJ, JAS, OND, ANN) | ||
seasons = ['JFM', 'JAS', 'ANN'] | ||
|
||
# list of depths in meters (positive up) at which to analyze, 'top' for the | ||
# sea surface, 'bot' for the sea floor | ||
depths = ['top', -200, -400, -600, -800, 'bot'] |
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.
why is the section below removed from this config file (same goes for other sections in other config files)?
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.
That's the whole point of this PR. People shouldn't need to point to these directories manually now that the directory structure is standardized.
Are you thinking that people would need to make their own region masks and point to those directories? I guess that's conceivable, particularly for non-standard grids.
My thought was that we want the example config files to be relatively simple. I guess you prefer to keep them relatively verbose and to point out to users that they could point the
mappingSubdirectory
and/orregionMaskSubdirectory
as an absolute path?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.
No, in this case I only meant the section below that has other (unrelated) things in it.
I would only keep config.example more verbose.
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. The way comments work on GitHub, all I see are the previous 3 lines to your comment so that wasn't clear. It could be that i need to put the MOC section back in but I don't think the rest of this is helpful. It just makes it a lot harder to change default options because all of the example config files have to be changed to match. I think the examples should be relatively lean.
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.
ok, agree that only the MOC section should be put back in for this specific config files, while the others are redundant/arbitrary.