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

sat_vapor_pres does not output location of bad temperatures (unless it's on a root pe) #1596

Open
wfcooke opened this issue Oct 9, 2024 · 1 comment · May be fixed by #1619
Open

sat_vapor_pres does not output location of bad temperatures (unless it's on a root pe) #1596

wfcooke opened this issue Oct 9, 2024 · 1 comment · May be fixed by #1619
Assignees

Comments

@wfcooke
Copy link

wfcooke commented Oct 9, 2024

If I set
show_all_bad_values = true
in sat_vapor_pres_nml I expect to see reports of locations of bad temperatures in my stdout. I do not.

The issue is that the output line
e.g.

write(iunit,'(a,e10.3,a,i4,a,i6)') 'Bad temperature=',temp(i),' at i=',i,' pe=',mpp_pe()

uses stdout as the output unit.
stdout is only defined for root pe

If I use stderr instead at

stdoutunit = stdout()

I get the output I need.
For readability stdoutunit should probably be changed to stderrunit (and set to stderr) in the sat_vapor_pres codeset.

@rem1776 rem1776 self-assigned this Oct 10, 2024
@rem1776 rem1776 linked a pull request Dec 3, 2024 that will close this issue
8 tasks
@rem1776
Copy link
Contributor

rem1776 commented Dec 3, 2024

@wfcooke Thanks for bringing this up, just submitted PR #1619 which should fix this issue for the next release. With the update it will change the output formatting slightly, from:

Bad temperature= 0.100E+03 at i= 1 j= 1 k= 1 pe= 1

to
WARNING from PE 1: Bad temperature= 0.100E+03 at i= 1 j= 1 k= 1

Which I hope won't cause any issues, but if it does just let me know.

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 a pull request may close this issue.

2 participants