Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[pull] master from wrf-model:master#5

Merged
pull[bot] merged 1 commit into
MRE-Code-Hub:masterfrom
wrf-model:master
Jan 31, 2026
Merged

[pull] master from wrf-model:master#5
pull[bot] merged 1 commit into
MRE-Code-Hub:masterfrom
wrf-model:master

Conversation

@pull

@pull pull Bot commented Jan 31, 2026

Copy link
Copy Markdown

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

…WRITE statement (#2265)

TYPE: bug fix

KEYWORDS: time series, TS, format, write, wrf solar, solar_diagnostics

SOURCE: Massimo D'Isidoro (ENEA - Italian National Agency for New Technologies, Energy and Sustainable Economic Development)

DESCRIPTION OF CHANGES:
Problem:
When solar diagnostics is activated in namelist.input, the WRF time series output WRITE statement lists 55 variables but the Fortran FMT only specifies 54 fields. This causes the last variable (ts_swddnic2) to
be written in a separate record in the output files *.TS , leading to inconsistent time series data.

Solution:
The WRITE statement FMT has been corrected to match all 55 variables, ensuring that they are written on the same record in the TS file.

ISSUE: 
This fixes part of issue #2255

LIST OF MODIFIED FILES: 
M       share/wrf_timeseries.F

TESTS CONDUCTED: 
1. A WRF run with solar_diagnostics activated, compared to previous version to confirm the fix.
2. The regression tests have passed.

RELEASE NOTE: Corrected a bug in the WRF time series output, when solar_diagnostics is activated, where the last variable (ts_swddnic2) was written in a separate record. TS files now correctly includes all the 55 variables on the same record.
@pull pull Bot locked and limited conversation to collaborators Jan 31, 2026
@pull pull Bot added the ⤵️ pull label Jan 31, 2026
@pull pull Bot merged commit 1957ed7 into MRE-Code-Hub:master Jan 31, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Development

Successfully merging this pull request may close these issues.

1 participant