Skip to content
Failed

Changes

Summary

  1. Threadpublic (#1133) (details)
  2. Threadpublic (#1133) (details)
  3. First commit to make compatible with latest clubb changes. (details)
  4. More updates to make work with clubb change. (details)
  5. Hopefully final update to make work the clubb's threadpublic branch. Using 1 mpi proc we have achieved BFBness with or without silhs. (details)
  6. Removing testing changes (details)
Commit 69a0500429869014c93503d18cff3ccfd516dc6d by noreply
Threadpublic (#1133)

* Initial threadprivate removals and simple changes.

* model flags update, incomplete

* Some updates, almost working

* Added saturation_formula and l_host_applies_sfc_fluxes to clubb_config_flags

* Accompanying changes to flags lists

* Changes to corrrelation array setup

* Adding more flag checking

* Forgot to set l_host_applies_sfc_fluxes in initialize_clubb_config_flags_type and commit adding l_implemented to the argument loit

* Adding saturation parameters to api

* Most of the way through passing sclr_dim, hydromet_dim, edsclr_dim, and sclr_tol through arguemnt lists. Moved first_write breaking silhs stats aparently, and bit changing due to the way we were writing sclr_tol for cases that have sclr_dim=0 (arm different, gabls2 BFB).

* Removing accidentally commited files.

* Fixed silhs output, all cases BFB but sclr_tol is saved differently to stats files in cases where sclr_dim = 0, so this is bc also bc:fb20458d5212b007c79c17613201af637bbc3d7e since that introduced the difference.

* Changing stats logicals back to false, only set to true for testing. So technically BIT_CHANGING

* Starting array_vars changes.

* No more global scalar indices

* Starting to remove hydromet indices

* Finished the annoying part with the microphysics interfaces.

* No more global variables in array_index. All cases BFB relative to last commit.

* No more global pdf_dim, Ncnp2_on_Ncnm2 now set via argument list, and added temporary hydromet_dim_glob because I thought it would fix a bug but it didn't, it might in the future though, so I left it in for now, it will be removed soon.

* Added nan checking before overriding default value for Ncnp2_on_Ncnm2

* Finished hydromet_dim argument listing and removed global versions of it as well as sclr_dim, edsclr_dim, and sclr_tol.

* open_netcdf_for_writing does nothing if nvars=0 for a stats file, which is understandable, but now that we call first_write immediately after we need that also to check for nvar=0

* Adding better error messages to stats_init

* The nvidia compiler does not like zero size arrays.

* Adding defaults for scalar arrays values in clubb driver, and adding an error check after calling stats_init.

* Reverting accidental changes to clubb_thread_test

* Changing name of scarr/scalar_array to sclr_idx and hydromet_array to hm_metadata

* Adding new types to _api

* Fixing erroneous name changes.

* Fixing bug introduced in 65987c8a77496f7bb10bfd28717345b1f63de1b7

* Removing print

* QuadTune: updates setup in order to read in 30x30
regional files automatically.

For #910.

* Updating the code to set the bottom of the thermodynamic level grid for
diffusion at thermodynamic level 2 -- the first level above the surface.

This is being done in preparation for ghostless gridding.

* Commit to enforce a fixed-point boundary condition of wp3=0 at
thermodynamic level k = 2, the first above-ground thermodynamic level.

This is being done in preparation for ghostless gridding.

This commit is bc.

Results are much more similar to the previous results than using
using zero-flux as the lower boundary condition for wp3.

* With wp3 now being set to a fixed-point boundary condition value
of 0 at the current thermodynamic level 2 (first t-lev above the
surface), it is not necessary to loop over level 2 within the code
for each wp3 term.

This commit is BIT_CHANGING, but only for the wp3 budget terms
(e.g. wp3_ta) in the zt output file at level k = 2. It doesn't change
the actual answer. All other variables are bit-for-bit identical.

* Removing omp parallel test line

* Removing parameter declarion around l_quintic_poly_interp. Having this be a parameter causes bit changes when using -O2

---------

Co-authored-by: vlarson <vlarson@uwm.edu>
Co-authored-by: bmg929 <bmg2@uwm.edu>
The file was modifiedindex_mapping.F90 (diff)
The file was modifiednumerical_check.F90 (diff)
The file was modifiedclip_explicit.F90 (diff)
The file was modifiedsetup_clubb_pdf_params.F90 (diff)
The file was modifiedstats_zm_module.F90 (diff)
The file was modifiedstats_clubb_utilities.F90 (diff)
The file was modifiedprecipitation_fraction.F90 (diff)
The file was modifieddiagnose_correlations_module.F90 (diff)
The file was modifiedparameters_model.F90 (diff)
The file was modifiedsponge_layer_damping.F90 (diff)
The file was modifiedstat_file_module.F90 (diff)
The file was modifiedadvance_xp2_xpyp_module.F90 (diff)
The file was modifiedarray_index.F90 (diff)
The file was modifiedoutput_netcdf.F90 (diff)
The file was modifiedsfc_varnce_module.F90 (diff)
The file was modifiedclubb_api_module.F90 (diff)
The file was modifiedfill_holes.F90 (diff)
The file was modifiedadg1_adg2_3d_luhar_pdf.F90 (diff)
The file was modifiedadvance_xp3_module.F90 (diff)
The file was modifiedadvance_windm_edsclrm_module.F90 (diff)
The file was modifiedadvance_clubb_core_module.F90 (diff)
The file was modifiederror_code.F90 (diff)
The file was modifiedstats_zt_module.F90 (diff)
The file was modifiedmodel_flags.F90 (diff)
The file was modifiedpdf_closure_module.F90 (diff)
The file was modifiedadvance_xm_wpxp_module.F90 (diff)
The file was modifiednew_hybrid_pdf_main.F90 (diff)
The file was modifiedcorr_varnce_module.F90 (diff)
The file was modifiedmixing_length.F90 (diff)
The file was modifiedsaturation.F90 (diff)
The file was modifiedadvance_helper_module.F90 (diff)
Commit 14c8636ab5c46f3f20204fae38b1eeb765bc804c by noreply
Threadpublic (#1133)

* Initial threadprivate removals and simple changes.

* model flags update, incomplete

* Some updates, almost working

* Added saturation_formula and l_host_applies_sfc_fluxes to clubb_config_flags

* Accompanying changes to flags lists

* Changes to corrrelation array setup

* Adding more flag checking

* Forgot to set l_host_applies_sfc_fluxes in initialize_clubb_config_flags_type and commit adding l_implemented to the argument loit

* Adding saturation parameters to api

* Most of the way through passing sclr_dim, hydromet_dim, edsclr_dim, and sclr_tol through arguemnt lists. Moved first_write breaking silhs stats aparently, and bit changing due to the way we were writing sclr_tol for cases that have sclr_dim=0 (arm different, gabls2 BFB).

* Removing accidentally commited files.

* Fixed silhs output, all cases BFB but sclr_tol is saved differently to stats files in cases where sclr_dim = 0, so this is bc also bc:fb20458d5212b007c79c17613201af637bbc3d7e since that introduced the difference.

* Changing stats logicals back to false, only set to true for testing. So technically BIT_CHANGING

* Starting array_vars changes.

* No more global scalar indices

* Starting to remove hydromet indices

* Finished the annoying part with the microphysics interfaces.

* No more global variables in array_index. All cases BFB relative to last commit.

* No more global pdf_dim, Ncnp2_on_Ncnm2 now set via argument list, and added temporary hydromet_dim_glob because I thought it would fix a bug but it didn't, it might in the future though, so I left it in for now, it will be removed soon.

* Added nan checking before overriding default value for Ncnp2_on_Ncnm2

* Finished hydromet_dim argument listing and removed global versions of it as well as sclr_dim, edsclr_dim, and sclr_tol.

* open_netcdf_for_writing does nothing if nvars=0 for a stats file, which is understandable, but now that we call first_write immediately after we need that also to check for nvar=0

* Adding better error messages to stats_init

* The nvidia compiler does not like zero size arrays.

* Adding defaults for scalar arrays values in clubb driver, and adding an error check after calling stats_init.

* Reverting accidental changes to clubb_thread_test

* Changing name of scarr/scalar_array to sclr_idx and hydromet_array to hm_metadata

* Adding new types to _api

* Fixing erroneous name changes.

* Fixing bug introduced in 65987c8a77496f7bb10bfd28717345b1f63de1b7

* Removing print

* QuadTune: updates setup in order to read in 30x30
regional files automatically.

For #910.

* Updating the code to set the bottom of the thermodynamic level grid for
diffusion at thermodynamic level 2 -- the first level above the surface.

This is being done in preparation for ghostless gridding.

* Commit to enforce a fixed-point boundary condition of wp3=0 at
thermodynamic level k = 2, the first above-ground thermodynamic level.

This is being done in preparation for ghostless gridding.

This commit is bc.

Results are much more similar to the previous results than using
using zero-flux as the lower boundary condition for wp3.

* With wp3 now being set to a fixed-point boundary condition value
of 0 at the current thermodynamic level 2 (first t-lev above the
surface), it is not necessary to loop over level 2 within the code
for each wp3 term.

This commit is BIT_CHANGING, but only for the wp3 budget terms
(e.g. wp3_ta) in the zt output file at level k = 2. It doesn't change
the actual answer. All other variables are bit-for-bit identical.

* Removing omp parallel test line

* Removing parameter declarion around l_quintic_poly_interp. Having this be a parameter causes bit changes when using -O2

---------

Co-authored-by: vlarson <vlarson@uwm.edu>
Co-authored-by: bmg929 <bmg2@uwm.edu>
The file was modifiedsilhs_api_module.F90 (diff)
The file was modifiedsilhs_importance_sample_module.F90 (diff)
The file was modifiedlatin_hypercube_driver_module.F90 (diff)
The file was modifiedoutput_2D_samples_module.F90 (diff)
The file was modifiedtransform_to_pdf_module.F90 (diff)
Commit 4c2910912901810f3aab85dfc70ef384bb7b4555 by 37674341+huebleruwm
First commit to make compatible with latest clubb changes.
The file was modifiedsrc/physics/cam/clubb_intr.F90 (diff)
Commit 0ab05498ab58e8e66b45ba43afc92ced443f1452 by 37674341+huebleruwm
More updates to make work with clubb change.
The file was modifiedsrc/physics/cam/clubb_intr.F90 (diff)
The file was modifiedsrc/physics/cam/physpkg.F90 (diff)
The file was modifiedsrc/physics/cam/subcol_SILHS.F90 (diff)
Commit 8db57f1ea3df8f66286d9e38bdd373f9d0ef43bf by 37674341+huebleruwm
Hopefully final update to make work the clubb's threadpublic branch. Using 1 mpi proc we have achieved BFBness with or without silhs.
The file was modifiedsrc/physics/cam/clubb_intr.F90 (diff)
The file was modifiedsrc/physics/cam/physpkg.F90 (diff)