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

CAM fix for scorpio #3157

Merged
merged 1 commit into from
Aug 28, 2019
Merged

CAM fix for scorpio #3157

merged 1 commit into from
Aug 28, 2019

Conversation

dqwu
Copy link
Contributor

@dqwu dqwu commented Aug 27, 2019

This PR includes a fix related to scorpio file id usage in CAM.

PR #2771 has fixed similar issues for other
shr_scam_getCloseLatLon calls.

Fixes #2963

[BFB]

With this fix, module procedure shr_scam_getCloseLatLonPIO is
called instead of shr_scam_getCloseLatLonNC.

For scorpio, the ncid returned to the user is not the actual ncid
returned by the underlying netcdf/pnetcdf library. This opaque
pio_ncid is unusable from outside scorpio, such as nf90_inquire
calls in shr_scam_getCloseLatLonNC.
@jayeshkrishna jayeshkrishna assigned singhbalwinder and unassigned dqwu Aug 27, 2019
singhbalwinder added a commit that referenced this pull request Aug 27, 2019
CAM fix for scorpio

This PR includes a fix related to scorpio file id usage in CAM.

PR #2771 has fixed similar issues for other
shr_scam_getCloseLatLon calls.

Fixes #2963

[BFB]

* dqwu/scorpio_cam_fix:
  Updating a call to interface shr_scam_getCloseLatLon
@singhbalwinder
Copy link
Contributor

Merged to next

singhbalwinder added a commit that referenced this pull request Aug 28, 2019
CAM fix for scorpio

This PR includes a fix related to scorpio file id usage in CAM.

PR #2771 has fixed similar issues for other
shr_scam_getCloseLatLon calls.

Fixes #2963

[BFB]

* dqwu/scorpio_cam_fix:
  Updating a call to interface shr_scam_getCloseLatLon
@singhbalwinder singhbalwinder merged commit 1501d8f into master Aug 28, 2019
@dqwu dqwu deleted the dqwu/scorpio_cam_fix branch August 29, 2019 16:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

SMS_R_Ld5.ne4_ne4.FSCM5A97 test fails with PIO2
3 participants