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

Iofix #30

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Iofix #30

wants to merge 2 commits into from

Conversation

ngrivault
Copy link

iofix.F:
Format change for Stations and Outputs (ASCII mode)
Land masking fixed (Cartesian case)

mod_parallel_field_io.F:
WRITE format fix to Matlab

      Land masking fixed (Cartesian case)
mod_parallel_field_io.F: WRITE format fix to Matlab
@malej
Copy link
Collaborator

malej commented May 7, 2019

@fengyanshi ... did you have a look at this for station output yet.... I recall discussing but did you merge it yet?

@fengyanshi
Copy link
Owner

I found a bug in his updated code
write(FORMAT_LEN(9:13),'(A5)') 'E16.6'
probably should use (A7) instead of (A5). I didn't merge the code. I did the suggested format manually and will push to github later when I finish the slender case.

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.

3 participants