Skip to content
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

Just isomip src #263

Merged
merged 17 commits into from
May 10, 2016
Merged

Conversation

gustavo-marques
Copy link
Collaborator

This branch has the basic source codes for the ISOMIP test case. The model compiles, but this test cases does not run properly as of now. If possible, It would be good to add these new/modified files to the main branch so that future changes will also be applied to the ISOMIP case.

gustavo-marques and others added 17 commits March 25, 2016 15:16
Modified ISOMIP files so they are compatible with those changes. In particular, dev/master had changes to the vertical grid structures.
The code now compiles.
Remerged it in and removed the conflicts.
…unt test case (ALE/RHO).

Replaced NIMEM_ with SZI_ in ISOMIP_initialization.F90 and MOM_state_initialization.F90
Added GV argument to the ISOMIP calls in MOM_state_initialization.F90
Included a few parameters (NZ_ZERO_SUF, NZ_ZERO_BOT) that allow the user to set the thickness to zero (near the surface and bottom, respectively).
@adcroft adcroft merged commit 116c76f into mom-ocean:dev/master May 10, 2016
@gustavo-marques gustavo-marques deleted the just_isomip_src branch March 30, 2017 20:27
NoraLoose added a commit to NoraLoose/MOM6 that referenced this pull request Dec 3, 2022
iangrooms pushed a commit to iangrooms/MOM6 that referenced this pull request Jan 29, 2024
* Enhancements for adding land block elimination to NUOPC cap:
 - Add sum_across_PEs_int4_2d to the sum_across_PEs interface
 - Allow mask_table file to be placed in run directory (now,
the first dir that is looked at).

* Enhance NUOPC cap to support MOM_mask_table.

- Determine masked blocks.
- Evenly distribute eliminated cells.
- Fill ESMF gindex array accordingly.
- During Export phase, set fields of eliminated cells to zero.

* set %label in register_netcdf_field and register_netcdf_axis

* first working version of an automated mask table generator

* While determining masked blocks, take reentrancy and tripolar stitch into account

* apply tripolar stitch fix in auto mask_table generation

* add AUTO_IO_LAYOUT_FAC parameter to control IO_LOAYUT when AUTO_MASKTABLE is on

* Miscellaneous auto masking fixes to address reviews:

- Dimensionalize topographic depth variables used to determine cell masks in auto masktable routine.
- Raise error if the user provided PE layout is inconsistent with auto masktable generation.
- Save the masktable parameter description to a string variable to avoid repetition.
- Fix typos, whitespaces, use modern array syntax.

* Disable FPEs in MacOS testing

Due to poor handling of floating point in HDF5 1.14.3, it is currently
not possible to use floating point exceptions (FPEs) whenever this
version is present.

The GitHub Actions CI nodes would randomly select either 1.14.2 or
1.14.3, and would raise an FPE error if 1.14.3 was selected.
Additionally, the homebrew installation does not provide a clean method
for selecting a different version of HDF5.

Thus, for now we disable FPEs in the MacOS testing, and hope to catch
any legitimate FP errors in the Ubuntu version.  We will restore these
tests as soon as this has been fixed in an easily-accessible version of
HDF5.

As part of this PR, I have also moved the FCFLAGS configuration to the
platform specific Actions files, allowing for independent compiler
configuration for each platform.

---------

Co-authored-by: Marshall Ward <marshall.ward@noaa.gov>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants