New notebooks #190
Merged
New notebooks #190
Conversation
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
Updated mtn_wave_tests_1km to restore missing graphics.
Conflicts: docs/examples/BCMoist.ipynb
Conflicts: docs/examples/README.md
|
BTW when merged could the commits be squashed? Each edit to README.md in particular resulted in a new commit. |
bensonr
reviewed
May 10, 2022
|
Hi, Rusty. Users could conceivably simply compile the Python-wrapped
library themselves (instructions are in fv_eta.F90) so if it doesn't belong
there we can remove it.
Thanks,
Lucas
…On Tue, May 10, 2022 at 3:09 PM Rusty Benson ***@***.***> wrote:
***@***.**** commented on this pull request.
I know this is a small file, but does
fv_eta.cpython-39-x86_64-linux-gnu.so need to be stored in the repo?
—
Reply to this email directly, view it on GitHub
<#190 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AMUQRVBNKKTZ6ZLTABYIU4TVJKX45ANCNFSM5VSEBSJQ>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
|
@lharris4 - I think we'd have a better git experience without the python compiled object in the repository. |
|
OK. I have a few updates I want to make anyway so hold off on the PR.
Thanks,
Lucas
…On Wed, May 11, 2022 at 9:56 AM Rusty Benson ***@***.***> wrote:
@lharris4 <https://github.com/lharris4> - I think we'd have a better git
experience without the python compiled object in the repository.
—
Reply to this email directly, view it on GitHub
<#190 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AMUQRVGUCQDLJ2D6BZMBMW3VJO373ANCNFSM5VSEBSJQ>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
|
OK, I have made some updates, deleted the binary, and added instructions for re-creating the binary containing the wrapped fv_eta module. |
laurenchilutti
approved these changes
Jun 15, 2022
|
Hi, Lauren. Thanks for taking care of this.
Lucas
…On Wed, Jun 15, 2022 at 10:15 AM laurenchilutti ***@***.***> wrote:
***@***.**** approved this pull request.
—
Reply to this email directly, view it on GitHub
<#190 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AMUQRVAU2UAFBHKZMBPSYVLVPHQJFANCNFSM5VSEBSJQ>
.
You are receiving this because you modified the open/close state.Message
ID: ***@***.***
com>
|
laurenchilutti
added a commit
that referenced
this pull request
Oct 13, 2022
* updated fv_regional_bc.F90 to read levsp from GFS_ctl file (#152) * Updating a continue statement in fv_nudge (#153) * update in fv_nudge to fix 666 label logic * fix logic errors (#138) * update to external_ic::remap_scale to skip remapping non-existent IC tracers (#159) * Fix nudge logic (#157) * fix logic errors * fix answer changes * adding back a line that was mistakenly deleted in a previous commit (#166) * Release 042022 (#184) * Merge of updates from GFDL Weather and Climate Dynamics Division (202204): *add license header to missing files and fix typo in header *updates needed for fv3_gfsphysics to have access to bounded_domain *remove obsoleted driver/SHiELD files *updating to fix bug where long_name and units attributes were not being captured in the RESTARTS *remove unused function fv_diagnostics::max_vorticity_hy1 *remove avec timer remnants *adding ability to specify prefix and directory when reading and writing restarts *remove old style namelist read in favor of read from internal character variable *Added option for a mean wind *The planetary radius and rotation rate are now re-scalable by a namelist parameter (small_earth_scale) instead of using exclusively the hard-coded FMS constant. *fv_mapz: Cleanup and added helpful annotations to avoid getting lost so easily * remove duplicate code and fix lstatus on all grids depending on gfs_data and gfs_data.tile1 * New idealized tests *Makes the non-hydrostatic restart variables optional for reads to allow hydrostatic ICs *Fix the hydrostatic TE remapping; Add GMAO cubic for TE remapping, which is used if kord_tm=0 and remap_te=.true. *Add a TE remapping option (kord_tm=0) *Addressing GNU Warnings *Add the L75 vertical config from HAFS * clean up fms_mp_mod and remove mp_bcst * cherry pick 5193c6b from dev/emc * Attempt at integrating fixes on top of dev/emc branch. (#173) * remove outdated GFDL_tools/fv_diag_column.F90 which exists as the result of an improper merge. The correct file is tools/fv_diag_column.F90 (#191) * various io fixes (#192) * fixes io performance issues by making everyone a reader when io_layout=1,1 adds capability to use FMS feature to ignore data integrity checksums in restarts * rename enforce_rst_cksum to ignore_rst_cksum and change the default value for compatibility * fix index error (#196) * New notebooks (#190) * Moving source files into source directory * Added advection notebook * Fixed subplot spacing * New 3D case notebooks * New idealized tests Updated mtn_wave_tests_1km to restore missing graphics. * first try at held-suarez * Updated H-S superrotation notebook * New level placement tool in an interactive note * Minor updates to notebooks; deleted fv_eta binary. * Regional decomposition test fix (when nrows_blend > 0) (#194) (#200) * Regional bc blend changes to extend into interior halos and overlap on corners. Still not working for u and v. dyn_core.F90 : Fix from Jun Wang to correct sync of u,v fv_regional_bc.F90 : add check for nrows_blend > tile size; fix error when nrows_blend=1 Conflicts (not taken during chery-pick): driver/SHiELD/atmosphere.F90 model/fv_sg.F90 Co-authored-by: Ted Mansell <[email protected]> * Implementing CI (#207) * CI Parallelworks update (#211) * Update call to read_input_nml and remove unnecessary code. (#161) * Removing use of INTERNAL_FILE_NML and cleaning up read_namelist_test_cases to remove unused argument * deleting duplicate call to read_namelist_test_case_nml in fv_control * removing commented code in fv_control Co-authored-by: Rusty Benson <[email protected]> Co-authored-by: menzel-gfdl <[email protected]> Co-authored-by: Rusty Benson <[email protected]> Co-authored-by: MatthewPyle-NOAA <[email protected]> Co-authored-by: lharris4 <[email protected]> Co-authored-by: Ted Mansell <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Description
Two new Python notebooks:
FV3_level_transmogrifier, which shows different FV3 standard levels and allows inspection for discontinuities. Directory has the dependent files including the standard atmosphere and a Linux-Intel binary for fv_eta_mod. This allows construction of new vertical level setups that are free of noise and stability issues.How Has This Been Tested?
Not code, just Python notebooks.
Checklist:
Please check all whether they apply or not