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

mixed precision sat_vapor_pressure #1095

Merged
merged 74 commits into from
Mar 29, 2023
Merged

Conversation

mlee03
Copy link
Contributor

@mlee03 mlee03 commented Dec 20, 2022

Description
In this Pull Request, sat_vapor_pres has been edited to as part of the mixed-mode project in FMS. The changes here include removal of all unlimited polymorphic variables (i.e class(*) ) in this module.

How Has This Been Tested?
make and make check passes with Intel and GCC

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • Any dependent changes have been merged and published in downstream modules
  • New check tests, if applicable, are included
  • make distcheck passes

@mlee03 mlee03 requested a review from mcallic2 December 20, 2022 15:18
sat_vapor_pres/Makefile.am Outdated Show resolved Hide resolved
sat_vapor_pres/sat_vapor_pres.F90 Outdated Show resolved Hide resolved
sat_vapor_pres/include/sat_vapor_pres.inc Outdated Show resolved Hide resolved
sat_vapor_pres/include/sat_vapor_pres.inc Outdated Show resolved Hide resolved
sat_vapor_pres/include/sat_vapor_pres.inc Outdated Show resolved Hide resolved
sat_vapor_pres/include/sat_vapor_pres.inc Outdated Show resolved Hide resolved
sat_vapor_pres/include/sat_vapor_pres.inc Outdated Show resolved Hide resolved
sat_vapor_pres/include/sat_vapor_pres_k.inc Outdated Show resolved Hide resolved
sat_vapor_pres/include/sat_vapor_pres_k.inc Outdated Show resolved Hide resolved
sat_vapor_pres/include/sat_vapor_pres_k.inc Outdated Show resolved Hide resolved
@mlee03
Copy link
Contributor Author

mlee03 commented Jan 26, 2023

@rem1776 @mcallic2 ready for review!

@mlee03
Copy link
Contributor Author

mlee03 commented Jan 26, 2023

Closing this PR, this PR contains mixedmode include files for each FMS directory. Will open a separate PR that will not include the include file so that it can go into main.

x = -9.09718*(TBASI/tem(i)-1.0) - 3.56654*log10(TBASI/tem(i)) &
+0.876793*(1.0-tem(i)/TBASI) + log10(ESBASI)
esice =10.**(x)
x = -9.09718_lkind*(TBASI/tem(i)-one) &
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

More funky indentation.

endif
if (present (esat)) then
esat = -999.
esat = -999.0_lkind
endif
endif ! (nbad = 0)


Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Extra newline.

@rem1776 rem1776 merged commit 7b3906c into NOAA-GFDL:mixedmode Mar 29, 2023
rem1776 pushed a commit to rem1776/FMS that referenced this pull request May 25, 2023
rem1776 added a commit that referenced this pull request Jun 2, 2023
…re, and fms_mod (#1239)

* feat: mixed precision axis_utils2 (#1104)

* feat: mixed precision fms_mod (#1147)

* feat: horiz interp mixed precision (#1067)

* mixed precision sat_vapor_pressure  (#1095)

* feat: add mixed precision axis_utils unit tests (#1172)

* fix: move type definitions to before first usage to fix nvhpc bug (#1187)

* fix: change allocatable type for intel errors (#1221)

Co-authored-by: Caitlyn McAllister <65364559+mcallic2@users.noreply.github.com>
Co-authored-by: Jesse Lentz <42011922+J-Lentz@users.noreply.github.com>
Co-authored-by: MiKyung Lee <58964324+mlee03@users.noreply.github.com>
rem1776 added a commit that referenced this pull request Jun 16, 2023
…re, and fms_mod (#1239) (#1258)

* feat: mixed precision axis_utils2 (#1104)

* feat: mixed precision fms_mod (#1147)

* feat: horiz interp mixed precision (#1067)

* mixed precision sat_vapor_pressure  (#1095)

* feat: add mixed precision axis_utils unit tests (#1172)

* fix: move type definitions to before first usage to fix nvhpc bug (#1187)

* fix: change allocatable type for intel errors (#1221)

Co-authored-by: Caitlyn McAllister <65364559+mcallic2@users.noreply.github.com>
Co-authored-by: Jesse Lentz <42011922+J-Lentz@users.noreply.github.com>
Co-authored-by: MiKyung Lee <58964324+mlee03@users.noreply.github.com>
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.

6 participants