-
Notifications
You must be signed in to change notification settings - Fork 135
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
mixed precision mosaic2 #1074
mixed precision mosaic2 #1074
Conversation
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.
Do you think we should do just one .fh per include file? I feel like having r4 and r8 copies isn't really necessary and we'll end up with tons of include files for directories with many modules.
mosaic2/include/grid2.fh
Outdated
!* License along with FMS. If not, see <http://www.gnu.org/licenses/>. | ||
!*********************************************************************** | ||
|
||
!> @file |
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.
have you tried generating the doc site for this btw? I would think this file needs @addtogroup
but since you put those in the .inc instead it might be fine.
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'll add this to the *.fh files
mosaic2/include/grid2.inc
Outdated
end select | ||
end select | ||
call get_grid_size_for_one_tile(component, tile, nlon, nlat) | ||
valid_types = .true. |
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.
you can get rid of the valid_types variable, it was only added as a easy way to error out if the types don't match up.
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.
good catch, this one must have slipped
@rem1776, some of the fh files could get very long. It would be more organized and easier for future development if they were in separate files, that's my take on it... |
* mixed precision column_diagnostics (#1076) * mixed precision mosaic2 (#1074) * mixed precision topography unit tests (#1364) * mixed precision tridiagonal (#1363) * mixed precision: add test for coupler_types and data_override changes (#1361) * Mixed precision `amip_interp_mod` (#1365) BREAKING CHANGE: Several variables and arrays which were previously public have been made private. These variables and arrays are not used in ice_amip or ocean_amip. amip_interp_type(r4_kind) and amip_interp_type(r8_kind) should be used for r4_kind and r8_kind precision, respectively. amip_interp_type without a kind parameter defaults to r8_kind. Co-authored-by: rem1776 <Ryan.Mulhall@noaa.gov> Co-authored-by: mlee03 <Mikyung.Lee@lscamd50-d.gfdl.noaa.gov> Co-authored-by: Caitlyn McAllister <65364559+mcallic2@users.noreply.github.com> Co-authored-by: Jesse Lentz <42011922+J-Lentz@users.noreply.github.com>
Description
In this Pull Request,
mosaic2
has been edited to as part of the mixed-mode project in FMSHow Has This Been Tested?
make and make check passes with
Intel
andGCC
Checklist:
make distcheck
passes