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

Feature/tclune/#81 compute coords locally #83

Merged
merged 6 commits into from
Nov 6, 2023

Conversation

tclune
Copy link

@tclune tclune commented Oct 26, 2023

This PR introduces new algorithms for computing lat lon corners of cubed sphere. These new routines use far less memory, as no global domain arrays are involved. They are also arguably more accurate, and run faster not that the latter are significant concerns.

These have not been validated in GEOS, and the default behavior is to use the older coordinates for backward compatibility. This can be toggled by seting compute_coords_locally = .true. in the fv_core_nml namelist.

tclune and others added 5 commits October 9, 2023 15:12
So far only tested in ESMF CubedSphere constructor.   Committing now
to transfer to a test environment.
Same routines for computing new coordinates now in both packages.
@tclune tclune added Non 0-diff The changes in this pull request are non-zero-diff Needs Lead Approval Change requires science lead approval Skip Changelog labels Oct 26, 2023
@github-actions
Copy link

Label error. Requires exactly 0 of: Contingent - DNA, Needs Lead Approval, Contingent -- Do Not Approve. Found: Non 0-diff, Needs Lead Approval, Skip Changelog

3 similar comments
@github-actions
Copy link

Label error. Requires exactly 0 of: Contingent - DNA, Needs Lead Approval, Contingent -- Do Not Approve. Found: Non 0-diff, Needs Lead Approval, Skip Changelog

@github-actions
Copy link

Label error. Requires exactly 0 of: Contingent - DNA, Needs Lead Approval, Contingent -- Do Not Approve. Found: Non 0-diff, Needs Lead Approval, Skip Changelog

@github-actions
Copy link

Label error. Requires exactly 0 of: Contingent - DNA, Needs Lead Approval, Contingent -- Do Not Approve. Found: Non 0-diff, Needs Lead Approval, Skip Changelog

@tclune tclune added 0 diff The changes in this pull request have verified to be zero-diff with the target branch. and removed Non 0-diff The changes in this pull request are non-zero-diff labels Oct 26, 2023
@github-actions
Copy link

Label error. Requires exactly 0 of: Contingent - DNA, Needs Lead Approval, Contingent -- Do Not Approve. Found: 0 diff, Needs Lead Approval, Skip Changelog

1 similar comment
@github-actions
Copy link

Label error. Requires exactly 0 of: Contingent - DNA, Needs Lead Approval, Contingent -- Do Not Approve. Found: 0 diff, Needs Lead Approval, Skip Changelog

@mathomp4 mathomp4 requested a review from wmputman October 26, 2023 17:17
@tclune tclune changed the base branch from geos/main to geos/develop October 26, 2023 17:17
@mathomp4 mathomp4 self-requested a review October 26, 2023 17:18
@github-actions
Copy link

Label error. Requires exactly 0 of: Contingent - DNA, Needs Lead Approval, Contingent -- Do Not Approve. Found: 0 diff, Needs Lead Approval, Skip Changelog

@mathomp4
Copy link
Member

I confirm that this along with GEOS-ESM/FVdycoreCubed_GridComp#258 is zero-diff (when not activated, of course)

mathomp4
mathomp4 previously approved these changes Nov 1, 2023
Copy link

github-actions bot commented Nov 2, 2023

Label error. Requires exactly 0 of: Contingent - DNA, Needs Lead Approval, Contingent -- Do Not Approve. Found: 0 diff, Needs Lead Approval, Skip Changelog

@mathomp4 mathomp4 merged commit 1e92044 into geos/develop Nov 6, 2023
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0 diff The changes in this pull request have verified to be zero-diff with the target branch. Needs Lead Approval Change requires science lead approval Skip Changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants