ufs-weather-model icon indicating copy to clipboard operation
ufs-weather-model copied to clipboard

Published 20 hours ago • verified publisher icon ufs-community

extremely verbose write statement in FV3

Open SamuelTrahanNOAA opened this issue 10 months ago • 2 comments

Description

This write statement produces many megabytes of output for some configurations. It is enough to have a significant impact on the runtime:

write(0,*) ' history_type_store_data3D kinds ', kind_phys, kind(work), lbound(work), ubound(work), size(work)

To Reproduce:

Run FV3 with the write component. Witness the write statement

SamuelTrahanNOAA avatar Mar 27 '24 13:03 SamuelTrahanNOAA

I already removed it in this PR:

https://github.com/NOAA-EMC/fv3atm/pull/803/files#diff-fbce29fff88e35fff84fde16c5b1f1a366a0940468defddf41b28a940cb9833d

DusanJovic-NOAA avatar Mar 27 '24 13:03 DusanJovic-NOAA

That PR is still a draft.

Our fix is identical (delete the line) so it doesn't matter which one is merged first.

SamuelTrahanNOAA avatar Mar 27 '24 13:03 SamuelTrahanNOAA

This was actually fixed in an earlier pull request, not 2201. Probably the pull request Dusan points to, two messages ago (https://github.com/NOAA-EMC/fv3atm/pull/803/) but I can't be certain.

SamuelTrahanNOAA avatar Apr 15 '24 13:04 SamuelTrahanNOAA